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

[Christoph Niedermaier]

From: Marek Vasut <marex@xxxxxxxxxxxx>
Sent: Thursday, October 22, 2026 1:34 PM
> 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.

I will remove the okay status in version 2.

Regards
Christoph