Re: [RFC/PATCH] Per-device parameter support (11/16)
From: Rusty Russell (rusty_at_rustcorp.com.au)
Date: 10/25/04
- Previous message: Adam Belay: "Re: [PATCH] PnP Fixes for 2.6.10-rc1"
- In reply to: Tejun Heo: "[RFC/PATCH] Per-device parameter support (11/16)"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
To: Tejun Heo <tj@home-tj.org> Date: Mon, 25 Oct 2004 14:57:29 +1000
On Sat, 2004-10-23 at 13:30 +0900, Tejun Heo wrote:
> dp_11_module_param_arr.diff
>
> This is the 11st patch of 16 patches for devparam.
>
> The unsigned int * @nump of module_param_array is changed back to
> unsigned int @num, and new sets of macros named module_param_arr*()
> are added. These new macros don't take the num argument. This change
> is made for two reasons
>
> 1. To be consistent with devparam macros. In devparam, we'll be
> using field name of struct elements, so we won't be able to use
> pointer argument.
> 2. It's more consistent with other moduleparam macros.
>
> This patch only modifies moduleparam.h and doesn't modify the users
> of the modified macros. The next patch takes care of that. This and
> the next patch (dp_12_module_param_arr_apply.diff) are optional.
It's finely balanced, but the module_param macro only skips the "&"
because it turns it into a name. module_param_named() should probably
be changed to take a ptr too. But you'll get a warning if you use the
interface wrong, so it's not a huge issue.
The balance is tipped here I think because the massive number of
functions we're starting to sprout into.
Rusty.
-- A bad analogy is like a leaky screwdriver -- Richard Braakman - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
- Previous message: Adam Belay: "Re: [PATCH] PnP Fixes for 2.6.10-rc1"
- In reply to: Tejun Heo: "[RFC/PATCH] Per-device parameter support (11/16)"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Relevant Pages
|
|