Messages in this thread | | | From | Vadim Pasternak <> | Subject | RE: [patch v1 1/1] leds: verify vendor and change license in mlxcpld driver | Date | Fri, 4 Nov 2016 10:18:09 +0000 |
| |
Hi Jacek,
Thank you very much. Vadim.
> -----Original Message----- > From: Jacek Anaszewski [mailto:j.anaszewski@samsung.com] > Sent: Friday, November 04, 2016 10:38 AM > To: Vadim Pasternak <vadimp@mellanox.com>; rpurdie@rpsys.net > Cc: linux-leds@vger.kernel.org; linux-kernel@vger.kernel.org; jiri@resnulli.us > 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@mellanox.com> > > --- > > 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@mellanox.com>"); > > 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
| |