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

Jean Delvare khali at linux-fr.org
Wed Oct 8 14:01:58 CEST 2008


Hi Darrick,

On Mon, 06 Oct 2008 18:19:09 -0700, Darrick J. Wong wrote:
> 
> Signed-off-by: Darrick J. Wong <djwong at 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




More information about the lm-sensors mailing list