From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pavel Machek Subject: Re: [PATCH 02/24] leds: core: Add support for composing LED class device names Date: Tue, 13 Nov 2018 22:50:40 +0100 Message-ID: <20181113215040.GA2235@amd> References: <20181111120234.GA28794@amd> <20181111201605.GA20160@amd> <57b77d4e-39a0-aaf2-5952-c2c25dc58593@gmail.com> <20181112103513.GB5695@amd> <662c02ba-455b-e2bf-a5e2-ae3933161894@gmail.com> <20181112190554.GA13959@amd> <20181112220616.GB11942@amd> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="k1lZvvs/B4yU6o8G" Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Geert Uytterhoeven Cc: Jacek Anaszewski , linux-leds@vger.kernel.org, "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Linux Kernel Mailing List , Rob Herring , Baolin Wang , Daniel Mack , dmurphy@ti.com, Linus Walleij , oleg@kaa.org.ua, Sakari Ailus , simon@lineageos.org, xiaotong.lu@spreadtrum.com List-Id: devicetree@vger.kernel.org --k1lZvvs/B4yU6o8G Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue 2018-11-13 22:38:49, Geert Uytterhoeven wrote: > Hi Pavel, >=20 > On Mon, Nov 12, 2018 at 11:07 PM Pavel Machek wrote: > > Not really, I'm afraid. Hard drives have no red LEDs on them (and the > > LED would not be visible, anyway) so the "device" symlink in such case > > would point to some kind of i2c LED controller. >=20 > I've seen hard drives with LEDs on the PCB. Me too. > External USB enclosures with hard drives typically have LEDs, too. Yes, but typically neither are OS controlled. I'm thinking home NAS system; it may have per-disk LEDs and they'll probably be on some kind of dedicated LED controller. Anyway, my point is that "device that controls this LED" is often different from "device this LED is about". Regards, Pavel --=20 (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blo= g.html --k1lZvvs/B4yU6o8G Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iEYEARECAAYFAlvrRzAACgkQMOfwapXb+vK3gwCfQZiQIfJVltzytk33zq1aIbwR ZiAAoKDyWVBNyG9TS9jhfQevyqvfWUHX =UwZm -----END PGP SIGNATURE----- --k1lZvvs/B4yU6o8G--