On 25/04/2024 11:15, Niko Pasaloukos wrote: > Add device tree bindings for the Blaize BLZP1600 CB2 > development board (carrier board), which uses the > BLZP1600 SoM. > > Reviewed-by: James Cowgill <james.cowgill@xxxxxxxxxx> > Reviewed-by: Matt Redfearn <matt.redfearn@xxxxxxxxxx> > Reviewed-by: Neil Jones <neil.jones@xxxxxxxxxx> > Signed-off-by: Nikolaos Pasaloukos <nikolaos.pasaloukos@xxxxxxxxxx> > --- > .../devicetree/bindings/arm/blaize.yaml | 40 +++++++++++++++++++ > 1 file changed, 40 insertions(+) > create mode 100644 Documentation/devicetree/bindings/arm/blaize.yaml > > diff --git a/Documentation/devicetree/bindings/arm/blaize.yaml b/Documentation/devicetree/bindings/arm/blaize.yaml > new file mode 100644 > index 000000000000..8034aeb7a2b4 > --- /dev/null > +++ b/Documentation/devicetree/bindings/arm/blaize.yaml > @@ -0,0 +1,40 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/arm/blaize.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Blaize Platforms > + > +maintainers: > + - James Cowgill <james.cowgill@xxxxxxxxxx> > + - Matt Redfearn <matt.redfearn@xxxxxxxxxx> > + - Neil Jones <neil.jones@xxxxxxxxxx> > + - Nikolaos Pasaloukos <nikolaos.pasaloukos@xxxxxxxxxx> > + > +description: | > + Blaize Platforms using SoCs designed by Blaize Inc. > + > + The products currently based on the BLZP1600 SoC: > + > + - BLZP1600-SoM: SoM module > + - BLZP1600-CB2: Development board CB2 based on BLZP1600-SoM > + > + The compatible property should follow the format: > + > + compatible = "blaize,blzp1600-cb2", "blaize,blzp1600"; This is a friendly reminder during the review process. It seems my or other reviewer's previous comments were not fully addressed. Maybe the feedback got lost between the quotes, maybe you just forgot to apply it. Please go back to the previous discussion and either implement all requested changes or keep discussing them. Thank you. Best regards, Krzysztof