[PATCH v2 2/6] dt-bindings: trivial-devices: add arduino spi mcu interface

Riccardo Mereu posted 6 patches 2 months, 3 weeks ago
There is a newer version of this series
[PATCH v2 2/6] dt-bindings: trivial-devices: add arduino spi mcu interface
Posted by Riccardo Mereu 2 months, 3 weeks ago
Add spidev item in order to describe in Arduino boards one of the
interfaces between microprocessor and microcontroller.

Signed-off-by: Riccardo Mereu <r.mereu@arduino.cc>
---
 Documentation/devicetree/bindings/trivial-devices.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
index 2eff6f274302..d0f7dbf15d6f 100644
--- a/Documentation/devicetree/bindings/trivial-devices.yaml
+++ b/Documentation/devicetree/bindings/trivial-devices.yaml
@@ -59,6 +59,8 @@ properties:
           - adi,lt7182s
             # AMS iAQ-Core VOC Sensor
           - ams,iaq-core
+            # Arduino microcontroller interface over SPI on UnoQ board
+          - arduino,unoq-mcu
             # Temperature monitoring of Astera Labs PT5161L PCIe retimer
           - asteralabs,pt5161l
             # i2c h/w elliptic curve crypto module
-- 
2.51.2
Re: [PATCH v2 2/6] dt-bindings: trivial-devices: add arduino spi mcu interface
Posted by Krzysztof Kozlowski 2 months, 3 weeks ago
On 14/11/2025 13:18, Riccardo Mereu wrote:
> Add spidev item in order to describe in Arduino boards one of the

Please drop "spidev item", because there is no such hardware as spidev.
It's Linuxism and its appearance in bindings only creates confusion for
further submissions (which are rejected...).

Please explain here the hardware, which in this context is the
connection between Arduino and Linux. You received comments that this is
not a trivial device, so please bring the hardware description which
will prove it is indeed a trivial device. Why don't you need supplies
for example?

Placeholders for spidev are not allowed, so you have entire commit msg
to convince us that you are doing something else.

I already asked for that so I won't be repeating again at v3.


Best regards,
Krzysztof
Re: [PATCH v2 2/6] dt-bindings: trivial-devices: add arduino spi mcu interface
Posted by Krzysztof Kozlowski 2 months, 3 weeks ago
On 14/11/2025 13:58, Krzysztof Kozlowski wrote:
> On 14/11/2025 13:18, Riccardo Mereu wrote:
>> Add spidev item in order to describe in Arduino boards one of the
> 
> Please drop "spidev item", because there is no such hardware as spidev.
> It's Linuxism and its appearance in bindings only creates confusion for
> further submissions (which are rejected...).
> 
> Please explain here the hardware, which in this context is the
> connection between Arduino and Linux. You received comments that this is
> not a trivial device, so please bring the hardware description which
> will prove it is indeed a trivial device. Why don't you need supplies
> for example?

... plus some hardware details on that MCU, e.g. its processor or to
which SPI controller it is wired on the board.

> 
> Placeholders for spidev are not allowed, so you have entire commit msg
> to convince us that you are doing something else.
> 
> I already asked for that so I won't be repeating again at v3.
> 
> 
> Best regards,
> Krzysztof


Best regards,
Krzysztof