linux-kernel - Re: [patch] 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 for Android: free password hash cracker in your pocket

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

Date:   Thu, 15 Sep 2016 16:53:58 +0200
From:   Jacek Anaszewski <j.anaszewski@...sung.com>
To:     Pavel Machek <pavel@....cz>, vadimp@...lanox.com
Cc:     rpurdie@...ys.net, linux-leds@...r.kernel.org,
        linux-kernel@...r.kernel.org, jiri@...nulli.us,
        michaelsh@...lanox.com
Subject: Re: [patch] leds: add driver for Mellanox systems leds

Hi Pavel,

On 09/15/2016 02:52 PM, Pavel Machek wrote:
> Hi!
>
>> +++ b/MAINTAINERS
>> @@ -7655,6 +7655,14 @@ W:	http://www.mellanox.com
>>  Q:	http://patchwork.ozlabs.org/project/netdev/list/
>>  F:	drivers/net/ethernet/mellanox/mlxsw/
>>
>> +MELLANOX MLXCPLD LED DRIVER
>> +M:	Vadim Pasternak <vadimp@...lanox.com>
>> +L:	linux-leds@...r.kernel.org
>> +S:	Supported
>> +W:	http://www.mellanox.com
>
> Unless you have web page specificly about the LED hardware, I don't
> think W: tag is useful here.
>

I raised also similar objections during review. Eventually I left
the entry after skimming through the MAINTAINERS and noticing few
other similar examples.

In fact, the link to the main page doesn't convey any better information
than can be obtained after searching google. I'm dropping the entry
then.

-- 
Best regards,
Jacek Anaszewski