linux-kernel - Re: [PATCH v4 00/18] Introduce a led trigger for CPU activity and consolidate LED driver in ARM

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: <CACRpkda6-8dxSbF12N-Q32cOpogu-ZZmTApnvNBa03iOcEeqaQ@mail.gmail.com>
Date:	Mon, 29 Aug 2011 13:00:42 +0200
From:	Linus Walleij <linus.walleij@...aro.org>
To:	Bryan Wu <bryan.wu@...onical.com>,
	Richard Purdie <rpurdie@...ys.net>
Cc:	linux@....linux.org.uk, tony@...mide.com, jochen@...am.de,
	linux@...im.org.za, nicolas.pitre@...aro.org, arnd@...db.de,
	linux-arm-kernel@...ts.infradead.org, jamie@...ieiles.com,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v4 00/18] Introduce a led trigger for CPU activity and
 consolidate LED driver in ARM

On Fri, Aug 26, 2011 at 11:03 AM, Bryan Wu <bryan.wu@...onical.com> wrote:

> Based on Linus Walleij's ARM LED consolidation work, this patchset introduce a
> new generic led trigger for CPU not only for ARM but also for others.

I think Russell was pretty clear that he saw problems inside the "new LEDs"
subsystem (drivers/leds) that need be fixed before we migrate this support
to use the LED class devices.

Can you see if you can address his comment on the earlier realview patch,
i.e. this:
http://marc.info/?l=linux-arm-kernel&m=131237280009775&w=2

rpurdie: your comments on the state of the LEDs class?

If I get time I will attempt to fix it, no clue when that will be though.

Thanks,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/