[PATCH v2 1/5] dt-bindings: prci: add indexes for reset signals available in prci

[Jagan Teki]

On Fri, Jul 3, 2020 at 11:52 AM Sagar Kadam <sagar.kadam at sifive.com> wrote:
>
>
> > -----Original Message-----
> > From: U-Boot <u-boot-bounces at lists.denx.de> On Behalf Of Sagar Kadam
> > Sent: Monday, June 29, 2020 9:37 PM
> > To: Jagan Teki <jagan at amarulasolutions.com>
> > Cc: U-Boot-Denx <u-boot at lists.denx.de>; Rick Chen <rick at andestech.com>;
> > Paul Walmsley ( Sifive) <paul.walmsley at sifive.com>; Palmer Dabbelt
> > <palmer at dabbelt.com>; Anup Patel <anup.patel at wdc.com>; Atish Patra
> > <atish.patra at wdc.com>; Lukasz Majewski <lukma at denx.de>; Pragnesh
> > Patel <pragnesh.patel at sifive.com>; bin.meng at windriver.com; Simon Glass
> > <sjg at chromium.org>; Trevor Woerner <twoerner at gmail.com>; Eugeniy
> > Paltsev <Eugeniy.Paltsev at synopsys.com>; Patrick Wildt
> > <patrick at blueri.se>; Weijie Gao <weijie.gao at mediatek.com>; Fabio
> > Estevam <festevam at gmail.com>
> > Subject: RE: [PATCH v2 1/5] dt-bindings: prci: add indexes for reset signals
> > available in prci
> >
> > Hello Jagan,
> > > -----Original Message-----
> > > From: Jagan Teki <jagan at amarulasolutions.com>
> > > Sent: Monday, June 29, 2020 9:00 PM
> > > To: Sagar Kadam <sagar.kadam at sifive.com>
> > > Cc: U-Boot-Denx <u-boot at lists.denx.de>; Rick Chen
> > > <rick at andestech.com>; Paul Walmsley ( Sifive)
> > > <paul.walmsley at sifive.com>; Palmer Dabbelt <palmer at dabbelt.com>;
> > Anup
> > > Patel <anup.patel at wdc.com>; Atish Patra <atish.patra at wdc.com>; Lukasz
> > > Majewski <lukma at denx.de>; Pragnesh Patel
> > <pragnesh.patel at sifive.com>;
> > > bin.meng at windriver.com; Simon Glass <sjg at chromium.org>; Trevor
> > Woerner
> > > <twoerner at gmail.com>; Eugeniy Paltsev
> > <Eugeniy.Paltsev at synopsys.com>;
> > > Patrick Wildt <patrick at blueri.se>; Weijie Gao
> > > <weijie.gao at mediatek.com>; Fabio Estevam <festevam at gmail.com>
> > > Subject: Re: [PATCH v2 1/5] dt-bindings: prci: add indexes for reset
> > > signals available in prci
> > >
> > > [External Email] Do not click links or attachments unless you
> > > recognize the sender and know the content is safe
> > >
> > > On Fri, Jun 26, 2020 at 8:51 AM Sagar Kadam <sagar.kadam at sifive.com>
> > > wrote:
> > > >
> > > > Hi Jagan,
> > > >
> > > > > -----Original Message-----
> > > > > From: Jagan Teki <jagan at amarulasolutions.com>
> > > > > Sent: Thursday, June 25, 2020 11:13 PM
> > > > > To: Sagar Kadam <sagar.kadam at sifive.com>
> > > > > Cc: U-Boot-Denx <u-boot at lists.denx.de>; Rick Chen
> > > <rick at andestech.com>;
> > > > > Paul Walmsley ( Sifive) <paul.walmsley at sifive.com>; Palmer Dabbelt
> > > > > <palmer at dabbelt.com>; Anup Patel <anup.patel at wdc.com>; Atish
> > > Patra
> > > > > <atish.patra at wdc.com>; Lukasz Majewski <lukma at denx.de>;
> > Pragnesh
> > > > > Patel <pragnesh.patel at sifive.com>; bin.meng at windriver.com; Simon
> > > Glass
> > > > > <sjg at chromium.org>; Trevor Woerner <twoerner at gmail.com>;
> > Eugeniy
> > > > > Paltsev <Eugeniy.Paltsev at synopsys.com>; Patrick Wildt
> > > > > <patrick at blueri.se>; Weijie Gao <weijie.gao at mediatek.com>; Fabio
> > > > > Estevam <festevam at gmail.com>
> > > > > Subject: Re: [PATCH v2 1/5] dt-bindings: prci: add indexes for
> > > > > reset
> > > signals
> > > > > available in prci
> > > > >
> > > > > [External Email] Do not click links or attachments unless you
> > > > > recognize
> > > the
> > > > > sender and know the content is safe
> > > > >
> > > > > On Thu, Jun 25, 2020 at 5:56 PM Sagar Shrikant Kadam
> > > > > <sagar.kadam at sifive.com> wrote:
> > > > > >
> > > > > > Add bit indexes for reset signals within the PRCI module on
> > > > > > FU540-C000 SoC.
> > > > > > The DDR and ethernet sub-system's have reset signals indicated
> > > > > > by these reset indexes.
> > > > > >
> > > > > > Signed-off-by: Sagar Shrikant Kadam <sagar.kadam at sifive.com>
> > > > > > Reviewed-by: Pragnesh Patel <Pragnesh.patel at sifive.com>
> > > > > > Reviewed-by: Bin Meng <bin.meng at windriver.com>
> > > > > > ---
> > > > > >  include/dt-bindings/clock/sifive-fu540-prci.h | 8 ++++++++
> > > > > >  1 file changed, 8 insertions(+)
> > > > > >
> > > > > > diff --git a/include/dt-bindings/clock/sifive-fu540-prci.h
> > > > > > b/include/dt-
> > > > > bindings/clock/sifive-fu540-prci.h
> > > > > > index 6a0b70a..1c03b09 100644
> > > > > > --- a/include/dt-bindings/clock/sifive-fu540-prci.h
> > > > > > +++ b/include/dt-bindings/clock/sifive-fu540-prci.h
> > > > > > @@ -15,4 +15,12 @@
> > > > > >  #define PRCI_CLK_GEMGXLPLL            2
> > > > > >  #define PRCI_CLK_TLCLK                3
> > > > > >
> > > > > > +/* Reset bit indexes to be used by driver */
> > > > > > +#define PRCI_RST_DDR_CTRL_N    0
> > > > > > +#define PRCI_RST_DDR_AXI_N     1
> > > > > > +#define PRCI_RST_DDR_AHB_N     2
> > > > > > +#define PRCI_RST_DDR_PHY_N     3
> > > > > > +/* bit 4 is reserved bit */
> > > > > > +#define PRCI_RST_RSVD_N                4
> > > > > > +#define PRCI_RST_GEMGXL_N      5
> > > > > >  #endif
> > > > >
> > > > > Do these bindings are synced from Linux? If Yes better to sync
> > > > > with a particular commit or tag rather than patch.
> > > > >
> > > >
> > > > No, these reset bindings are not synced from Linux.
> > >
> > > This is synced file from Linux, better to inline with Linux files
> > > always, if these bindings are not related to Linux then maintain it in
> > > a separate file or support it in Linux first if they do require for
> > > Linux.
> > >
> > Ohh. Sorry I thought you were asking if reset-bindings are from Linux.
> > Yes this file is synced from Linux but these reset-bindings are not related to
> > Linux. So I can split it and place reset bindings into another file:
> > "include/dt-bindings/reset/sifive-fu540-reset.h"
>
> It will be "include/dt-bindings/reset/sifive-fu540-prci.h"

What if it would be the same directory structure and file name with
-u-boot.h extension. This way we can identify these are u-boot related
defines.

Jagan.