Re: [PATCH 1/2] ARM: dts: imx6ull-dhcor-maveo-box: Remove unused SPI device

[Marek Vasut]

On 10/22/26 12:58 PM, Christoph Niedermaier wrote:

Hello Christoph,

> > > > > > > --- a/arch/arm/boot/dts/nxp/imx/imx6ull-dhcor-maveo-box.dts
> > > > > > > +++ b/arch/arm/boot/dts/nxp/imx/imx6ull-dhcor-maveo-box.dts
> > > > > > > @@ -60,14 +60,6 @@ &ecspi4 {
> > > > > > >      pinctrl-0 = <&pinctrl_ecspi4>;
> > > > > > >      pinctrl-names = "default";
> > > > > > >      status = "okay";
> > > > > >
> > > > > > Drop "status = "okay"" too , since this controller will be unused ?
> > > > >
> > > > > you can instantiate a new device on the SPI bus from userspace. up to
> > > > > you, but if I was a user I would appreciate to have it enabled.
> > > > Good point -- Christoph, is this the (use)case here , can the user do that?
> > >
> > > I don't think that's the case, since there isn't currently an SPI device
> > > connected to this interface. Nevertheless, I'd be interested to know how
> > > to instantiate a new device from user space.
> >
> > Documentation/spi/instantiating-devices.rst
>
> This is a relatively new document, I haven't seen it yet. Thank you very much.
>
> I'm now leaning toward leaving the interface set to “okay” so that the existing
> SPI pins on X10 connector can easily accessed in the future.

If you connect anything more complex than a simple device, you would 
anyway need a DT overlay for this.

> @Marek: If you insist, I can also remove the status line.
Either way is fine by me, but consider the above.