Re: [lm-sensors] [PATCH 09/12] ibmpex: Automatically load on IBM systems via DMI

[Jean Delvare <khali@linux-fr.org>]

Hi Darrick,

On Mon, 06 Oct 2008 18:19:09 -0700, Darrick J. Wong wrote:
> 
> Signed-off-by: Darrick J. Wong <djwong@us.ibm.com>
> ---
> 
>  drivers/hwmon/ibmpex.c |    6 ++++++
>  1 files changed, 6 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/hwmon/ibmpex.c b/drivers/hwmon/ibmpex.c
> index 4e9b19c..6e3a2c4 100644
> --- a/drivers/hwmon/ibmpex.c
> +++ b/drivers/hwmon/ibmpex.c
> @@ -608,3 +608,9 @@ MODULE_LICENSE("GPL");
>  
>  module_init(ibmpex_init);
>  module_exit(ibmpex_exit);
> +
> +MODULE_ALIAS("dmi:bvnIBM:*:pnIBMSystemx3650*");
> +MODULE_ALIAS("dmi:bvnIBM:*:pnIBMSystemx3550*");
> +MODULE_ALIAS("dmi:bvnIBM:*:pnIBMSystemx3655*");
> +MODULE_ALIAS("dmi:bvnIBM:*:pnIBMSystemx3755*");
> +MODULE_ALIAS("dmi:bvnIBM:*:pnIBMSystemx3350*");

Is there no way to have an alias at the IPMI level? That is, alias on
the device itself rather than the systems it is in? This would avoid the
foreseeable requirement to add entries every time new machines are
released.

But if there is no such way then I am fine applying this patch (except
that I'd sort the entries). I also suspect that the last "*" should
actually be ":*"?

-- 
Jean Delvare