Re: [PATCH v5 1/7] dt-bindings: connector: Add mikrobus-connector

From: Michael Walle
Date: Thu Jun 27 2024 - 13:19:25 EST


Hi,

Could you give us a DT snippet of how this should look like with a
board?

On Thu Jun 27, 2024 at 6:26 PM CEST, Ayush Singh wrote:
> + board:
> + description: board attached to mikrobus connector
> + $ref: /schemas/types.yaml#/definitions/phandle-array

Shouldn't this be a subnode of the connector?

i.e.

connector {
compatible = "mikrobus-connector";

// phandles to the parent controllers

spi {
temp-sensor@0 {
compatible = "maxim,max31855k";
reg = <0>;
};
};

i2c {
..
};
};

I don't think you can introduce a new
compatible = "maxim,max31855k", "mikrobus,spi";
if there is already a binding for "maxim,max31855k". But I might be
wrong. Why is this compatible needed at all?

Also, the mikrobus-connector driver could translate the chipselects.

-michael