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

[Sagar Kadam]

> -----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"

BR,
Sagar

> and include it wherever required.  Please let me know if this sounds okay.
> 
> Thanks & BR,
> Sagar Kadam
> 
> > Jagan.