[PATCH 01/10] dt-bindings: i3c: Add YAML schema for Cadence I3C controller

[Maniyam, Dinesh]

> -----Original Message-----
> From: Tom Rini <trini at konsulko.com>
> Sent: Wednesday, 19 February 2025 12:29 am
> To: Frank Li <Frank.li at nxp.com>
> Cc: Maniyam, Dinesh <dinesh.maniyam at intel.com>; u-boot at lists.denx.de;
> Marek <marex at denx.de>; Simon Goldschmidt
> <simon.k.r.goldschmidt at gmail.com>; Simon Glass <sjg at chromium.org>; Ilias
> Apalodimas <ilias.apalodimas at linaro.org>; Heinrich Schuchardt
> <xypron.glpk at gmx.de>; Jerome Forissier <jerome.forissier at linaro.org>; Mattijs
> Korpershoek <mkorpershoek at baylibre.com>; Ibai Erkiaga <ibai.erkiaga-
> elorza at amd.com>; Michal Simek <michal.simek at amd.com>; Dmitry Rokosov
> <ddrokosov at salutedevices.com>; Jonas Karlman <jonas at kwiboo.se>; Sebastian
> Reichel <sebastian.reichel at collabora.com>; Wang Jie <dave.wang at rock-
> chips.com>; Meng, Tingting <tingting.meng at intel.com>; Chee, Tien Fong
> <tien.fong.chee at intel.com>; Hea, Kok Kiang <kok.kiang.hea at intel.com>; Ng,
> Boon Khai <boon.khai.ng at intel.com>; Yuslaimi, Alif Zakuan
> <alif.zakuan.yuslaimi at intel.com>; Zamri, Muhammad Hazim Izzat
> <muhammad.hazim.izzat.zamri at intel.com>; Lim, Jit Loon
> <jit.loon.lim at intel.com>; Tang, Sieu Mun <sieu.mun.tang at intel.com>
> Subject: Re: [PATCH 01/10] dt-bindings: i3c: Add YAML schema for Cadence I3C
> controller
> 
> On Tue, Feb 18, 2025 at 11:24:13AM -0500, Frank Li wrote:
> > On Tue, Feb 18, 2025 at 10:56:56AM +0800, dinesh.maniyam at intel.com wrote:
> > > From: Dinesh Maniyam <dinesh.maniyam at intel.com>
> > >
> > > Add a YAML device tree binding schema for the Cadence I3C
> > > controller,
> >
> > Cadence or snps? compatible string said snps.
> >
> > > defining required and optional properties for proper integration and
> > > validation with dt-schema.
> > >
> > > Signed-off-by: Dinesh Maniyam <dinesh.maniyam at intel.com>
> > > ---
> > >  .../i3c/cdns,i3c-master.yaml                  |  51 +++++
> > >  doc/device-tree-bindings/i3c/i3c.yaml         | 186 ++++++++++++++++++
> > >  2 files changed, 237 insertions(+)
> > >  create mode 100644
> > > doc/device-tree-bindings/i3c/cdns,i3c-master.yaml
> >
> > file name should be one of compatible string.
> >
> > >  create mode 100644 doc/device-tree-bindings/i3c/i3c.yaml
> >
> > Where doc/device-tree-bindings/i3c?
> >
> > in kernel, and already have i3c.yaml
> > Documentation/devicetree/bindings/i3c/
> >
> > stop here, fix file path firsly.
> 
> We don't need this in U-Boot at all because we already have it:
> dts/upstream/Bindings/i3c/cdns,i3c-master.yaml
> 
> --
> Tom

Will drop this commit, since the files in this commit are redundant.

Dinesh