<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Tue, 3 Dec 2024 at 19:51, Simon Glass <<a href="mailto:sjg@chromium.org">sjg@chromium.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Tom,<br>
<br>
On Tue, 3 Dec 2024 at 12:48, Tom Rini <<a href="mailto:trini@konsulko.com" target="_blank">trini@konsulko.com</a>> wrote:<br>
><br>
> On Thu, Nov 07, 2024 at 07:33:51PM +0000, Peter Robinson wrote:<br>
> > Hi Simon,<br>
> ><br>
> > > On Tue, 5 Nov 2024 at 07:18, Peter Robinson <<a href="mailto:pbrobinson@gmail.com" target="_blank">pbrobinson@gmail.com</a>> wrote:<br>
> > > ><br>
> > > > On Sun, 3 Nov 2024 at 00:34, Simon Glass <<a href="mailto:sjg@chromium.org" target="_blank">sjg@chromium.org</a>> wrote:<br>
> > > > ><br>
> > > > > There has been an LED framework in U-Boot which uses driver model for<br>
> > > > > about 9 years now. Recent work is underway to improve it and provide<br>
> > > > > more features. It is probably a good time to drop the old code, which<br>
> > > > > is only used by 5 boards:<br>
> > > ><br>
> > > > I don't believe, from what I can tell, they are feature comparable, at<br>
> > > > the very least I have not been able to get the Pinephone working with<br>
> > > > this so as it stands I still don't think this patch set is ready yet.<br>
> > ><br>
> > > I don't have that hardware, nor the other 4, so cannot do anything<br>
> > > with this feedback.<br>
> ><br>
> > Don't you have any HW that has a LED on it that you can substitute to<br>
> > see what it does?<br>
> ><br>
> > > Can you please be clear what you are asking me to do?<br>
> ><br>
> > Either produce patches that work on the the pinephone, or docs I, or<br>
> > other developers, can use to implement the functionality.<br>
> ><br>
> > Currently on the Pinephone the green LED lights up in the TPL/SPL<br>
> > (very early before ATF) stage and is lit up right through the the<br>
> > various FW stages, with your patch set I get no LED what so ever.<br>
><br>
> Please note that needing to confirm that we have equivalent<br>
> functionality between old and new frameworks (and<br>
> <a href="https://lore.kernel.org/all/20241110115054.2555-1-ansuelsmth@gmail.com/" rel="noreferrer" target="_blank">https://lore.kernel.org/all/20241110115054.2555-1-ansuelsmth@gmail.com/</a><br>
> might cover that) is why this series isn't ready for -next at this time.<br>
<br>
Yes, I'm not sure if Peter saw that, so I sent him the link.<br></blockquote><div><br></div><div>I have seen it, I have not had the chance to dig out my pinephone to test it again because I was traveling and had competing priorities on my time (and I do this as a hobby).</div><div><br></div><div>But also I think we have a little time on this, the new functionality only landed recently and we've had a LOT of deprecated functionality hang around for a lot longer than that. I think we should get this right rather than jam it through.</div><div><br></div><div>Peter<br></div></div></div>