lkml.org 
[lkml]   [2016]   [Nov]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [patch v1 1/1] leds: verify vendor and change license in mlxcpld driver
Date
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

\
 
 \ /
  Last update: 2016-11-04 18:49    [W:0.508 / U:0.716 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site