linux-kernel - RE: [patch v4] leds: add driver for Mellanox systems LEDs

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <AM5PR0501MB20975BE9666D425C624A2A1AA2FB0@AM5PR0501MB2097.eurprd05.prod.outlook.com>
Date:   Thu, 8 Sep 2016 10:39:06 +0000
From:   Vadim Pasternak <vadimp@...lanox.com>
To:     Jacek Anaszewski <j.anaszewski@...sung.com>,
        "rpurdie@...ys.net" <rpurdie@...ys.net>
CC:     "dave@...blig.org" <dave@...blig.org>,
        "linux-leds@...r.kernel.org" <linux-leds@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "jiri@...nulli.us" <jiri@...nulli.us>,
        Michael Shych <michaelsh@...lanox.com>
Subject: RE: [patch v4] leds: add driver for Mellanox systems LEDs



> -----Original Message-----
> From: Jacek Anaszewski [mailto:j.anaszewski@...sung.com]
> Sent: Thursday, September 08, 2016 1:24 PM
> To: Vadim Pasternak <vadimp@...lanox.com>; rpurdie@...ys.net
> Cc: dave@...blig.org; linux-leds@...r.kernel.org; linux-kernel@...r.kernel.org;
> jiri@...nulli.us; Michael Shych <michaelsh@...lanox.com>
> Subject: Re: [patch v4] leds: add driver for Mellanox systems LEDs
> 
> On 09/08/2016 12:20 PM, Jacek Anaszewski wrote:
> > On 09/08/2016 09:25 AM, vadimp@...lanox.com wrote:
> >> From: Vadim Pasternak <vadimp@...lanox.com>
> >
> >> diff --git a/Documentation/leds/leds-mlxcpld.txt
> >> b/Documentation/leds/leds-mlxcpld.txt
> >> new file mode 100644
> >> index 0000000..7b4c2a3
> >> --- /dev/null
> >> +++ b/Documentation/leds/leds-mlxcpld.txt
> >> @@ -0,0 +1,110 @@
> >> +ver LEDs-mlxcpld
> >
> > I noticed this only now - is it a stray line or you intended it was
> > something more meaningful?
> 
> OK, I changed it to:
> 
> Kernel driver for Mellanox systems LEDs
> =======================================
> 
> followed by an empty line.
> 
> >> +==========================
> >> +Driver for Mellanox systems LEDs.
> >> +Provide system LED support for the nex Mellanox systems:
> >> +"msx6710", "msx6720", "msb7700", "msn2700", "msx1410", "msn2410",
> >> +"msb7800", "msn2740", "msn2100".
> >
> >
OK.
Thank you very much for reviews.

When we'll have some docs available at WEB, I'll update leds-mlxcpld.txt with it.

Best regards,
Vadim.
> 
> 
> --
> Best regards,
> Jacek Anaszewski