Hello Stephen, On Mon, 20 Mar 2023 14:27:56 -0700 Stephen Boyd <sboyd@xxxxxxxxxx> wrote: > Quoting Sean Anderson (2023-01-24 08:23:45) > > On 1/24/23 03:28, Geert Uytterhoeven wrote: > > > Hi Luca, > > > > > > On Tue, Jan 24, 2023 at 9:12 AM Luca Ceresoli <luca.ceresoli@xxxxxxxxxxx> wrote: > > >> On Thu, 19 Jan 2023 14:27:43 -0500 > > >> Sean Anderson <sean.anderson@xxxxxxxx> wrote: > > >> > On 1/11/23 10:55, Geert Uytterhoeven wrote: > > > > > >> I'm wondering whether Geert has a practical example of a situation > > >> where it is better to have these properties optional. > > > > > > My issue was that these properties were introduced long after the > > > initial bindings, hence pre-existing DTS does not have them. > > > Yes, we can add them, but then we have to read out the OTP-programmed > > > settings first. If that's the way to go, I can look into that, though... > > > > FWIW I think there's no need to update existing bindings which don't > > have this property. The required aspect is mainly a reminder for new > > device trees. > > > > Is there any resolution on this thread? I'm dropping this patch from my > queue. IIRC Geert kind of accepted the idea that these properties should stay required. Which is a bit annoying but it's the safest option, so unless there are new complaints with solid use cases for making them optionalm, I think it's OK to drop the patch. Luca -- Luca Ceresoli, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
