Re: [PATCH 5/6] ARM: dts: imx6dl-yapp43: Add support for new HW revision of the IOTA board

From: Shawn Guo
Date: Mon Mar 13 2023 - 20:42:57 EST


On Mon, Mar 13, 2023 at 05:42:21PM +0100, Michal Vokáč wrote:
> > > + eeprom@57 {
> > > + compatible = "atmel,24c128";
> > > + reg = <0x57>;
> > > + pagesize = <64>;
> > > + status = "okay";
> >
> > The "okay" status is only needed to flip a "disabled" one.
>
> OK, I will remove the status "okay" from all device nodes defined
> by this dtsi that should be enabled by default.
>
> I assume that using status "okay" (or "disabled") for referenced
> nodes coming from included SoC dtsi files is fine even if these
> nodes already have the required status at the moment.

Yeah, that would be fine.

Shawn

> Otherwise I see no guarantee that the node will have my required
> status in the future as well. The status in the included files
> may be changed by someone else for whatever reason.