> -----Original Message-----
> From: Tom Rini <[email protected]>
> Sent: Wednesday, 19 February 2025 12:29 am
> To: Frank Li <[email protected]>
> Cc: Maniyam, Dinesh <[email protected]>; [email protected];
> Marek <[email protected]>; Simon Goldschmidt
> <[email protected]>; Simon Glass <[email protected]>; Ilias
> Apalodimas <[email protected]>; Heinrich Schuchardt
> <[email protected]>; Jerome Forissier <[email protected]>; Mattijs
> Korpershoek <[email protected]>; Ibai Erkiaga <ibai.erkiaga-
> [email protected]>; Michal Simek <[email protected]>; Dmitry Rokosov
> <[email protected]>; Jonas Karlman <[email protected]>; Sebastian
> Reichel <[email protected]>; Wang Jie <dave.wang@rock-
> chips.com>; Meng, Tingting <[email protected]>; Chee, Tien Fong
> <[email protected]>; Hea, Kok Kiang <[email protected]>; Ng,
> Boon Khai <[email protected]>; Yuslaimi, Alif Zakuan
> <[email protected]>; Zamri, Muhammad Hazim Izzat
> <[email protected]>; Lim, Jit Loon
> <[email protected]>; Tang, Sieu Mun <[email protected]>
> 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, [email protected] wrote:
> > > From: Dinesh Maniyam <[email protected]>
> > >
> > > 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 <[email protected]>
> > > ---
> > > .../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
