[PATCH v3 00/25] led: Remove old status-LED code

[Tom Rini]

On Thu, Nov 07, 2024 at 07:33:51PM +0000, Peter Robinson wrote:
> Hi Simon,
> 
> > On Tue, 5 Nov 2024 at 07:18, Peter Robinson <pbrobinson at gmail.com> wrote:
> > >
> > > On Sun, 3 Nov 2024 at 00:34, Simon Glass <sjg at chromium.org> wrote:
> > > >
> > > > There has been an LED framework in U-Boot which uses driver model for
> > > > about 9 years now. Recent work is underway to improve it and provide
> > > > more features. It is probably a good time to drop the old code, which
> > > > is only used by 5 boards:
> > >
> > > I don't believe, from what I can tell, they are feature comparable, at
> > > the very least I have not been able to get the Pinephone working with
> > > this so as it stands I still don't think this patch set is ready yet.
> >
> > I don't have that hardware, nor the other 4, so cannot do anything
> > with this feedback.
> 
> Don't you have any HW that has a LED on it that you can substitute to
> see what it does?
> 
> > Can you please be clear what you are asking me to do?
> 
> Either produce patches that work on the the pinephone, or docs I, or
> other developers, can use to implement the functionality.
> 
> Currently on the Pinephone the green LED lights up in the TPL/SPL
> (very early before ATF) stage and is lit up right through the the
> various FW stages, with your patch set I get no LED what so ever.

Please note that needing to confirm that we have equivalent
functionality between old and new frameworks (and
https://lore.kernel.org/all/20241110115054.2555-1-ansuelsmth@gmail.com/
might cover that) is why this series isn't ready for -next at this time.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20241203/ad900268/attachment.sig>