Hi Jacek, Thank you very much. Vadim. > -----Original Message----- > From: Jacek Anaszewski [mailto:j.anaszewski@xxxxxxxxxxx] > Sent: Friday, November 04, 2016 10:38 AM > To: Vadim Pasternak <vadimp@xxxxxxxxxxxx>; rpurdie@xxxxxxxxx > Cc: linux-leds@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; jiri@xxxxxxxxxxx > Subject: Re: [patch v1 1/1] leds: verify vendor and change license in mlxcpld > driver > > Hi Vadim, > > On 11/03/2016 08:17 PM, Vadim Pasternak wrote: > > Verify that vendor is Mellanox as the first step of initialization. > > If it is not - return ENODEV. > > Change module license from "GPL v2" to "Dual BSD/GPL". > > > > Signed-off-by: Vadim Pasternak <vadimp@xxxxxxxxxxxx> > > --- > > drivers/leds/leds-mlxcpld.c | 5 ++++- > > 1 file changed, 4 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/leds/leds-mlxcpld.c b/drivers/leds/leds-mlxcpld.c > > index 197ab9b..281482e 100644 > > --- a/drivers/leds/leds-mlxcpld.c > > +++ b/drivers/leds/leds-mlxcpld.c > > @@ -400,6 +400,9 @@ static int __init mlxcpld_led_init(void) > > struct platform_device *pdev; > > int err; > > > > + if (!dmi_match(DMI_CHASSIS_VENDOR, "Mellanox Technologies Ltd.")) > > + return -ENODEV; > > + > > pdev = platform_device_register_simple(KBUILD_MODNAME, -1, NULL, > 0); > > if (IS_ERR(pdev)) { > > pr_err("Device allocation failed\n"); @@ -426,5 +429,5 @@ > > module_exit(mlxcpld_led_exit); > > > > MODULE_AUTHOR("Vadim Pasternak <vadimp@xxxxxxxxxxxx>"); > > MODULE_DESCRIPTION("Mellanox board LED driver"); - > MODULE_LICENSE("GPL > > v2"); > > +MODULE_LICENSE("Dual BSD/GPL"); > > MODULE_ALIAS("platform:leds_mlxcpld"); > > > > Thanks for the patch. Applied. > > -- > Best regards, > Jacek Anaszewski -- To unsubscribe from this list: send the line "unsubscribe linux-leds" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
