On Thu, Oct 26, 2023 at 11:47:34AM +0300, Elad Nachman wrote: > From: Elad Nachman <enachman@xxxxxxxxxxx> > > Add dt bindings for AC5X RD COM Express Type 7 carrier board. > This board will Accept a CN9131 COM Express Type 7 CPU module. > > Signed-off-by: Elad Nachman <enachman@xxxxxxxxxxx> > --- > .../devicetree/bindings/arm/marvell/armada-7k-8k.yaml | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml b/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml > index 52d78521e412..71bc94047d1b 100644 > --- a/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml > +++ b/Documentation/devicetree/bindings/arm/marvell/armada-7k-8k.yaml > @@ -60,4 +60,12 @@ properties: > - const: marvell,armada-ap807-quad > - const: marvell,armada-ap807 > > + - description: AC5X RD COM Express Carrier for Armada CN9131 SoC with one external CP This description is i think technically wrong. You have the combination of the carrier and the module. > + items: > + - const: marvell,ac5x_rd_carrier > + - const: marvell,cn9131 > + - const: marvell,cn9130 > + - const: marvell,armada-ap807-quad > + - const: marvell,armada-ap807 Andrew