Re: [patch] leds: add driver for Mellanox systems leds

From	Jacek Anaszewski <j.anaszewski@samsung.com>
Newsgroups	linux.kernel
Subject	Re: [patch] leds: add driver for Mellanox systems leds
Date	2016-09-15 17:00 +0200
Message-ID	<shGiR-8jb-11@gated-at.bofh.it> (permalink)
References	<seJmW-7z9-25@gated-at.bofh.it> <shGiR-8jb-13@gated-at.bofh.it> <shEqJ-79k-7@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

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@mellanox.com>
>> +L:	linux-leds@vger.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

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [patch] leds: add driver for Mellanox systems leds Pavel Machek <pavel@ucw.cz> - 2016-09-15 15:00 +0200
  Re: [patch] leds: add driver for Mellanox systems leds Jacek Anaszewski <j.anaszewski@samsung.com> - 2016-09-15 17:00 +0200

csiph-web