[PATCH 2/5] fu540: prci: use common reset indexes defined in binding header

Sagar Kadam sagar.kadam at sifive.com
Wed Jun 24 09:05:34 CEST 2020


Hi,

> -----Original Message-----
> From: Bin Meng <bmeng.cn at gmail.com>
> Sent: Wednesday, June 24, 2020 8:33 AM
> To: Sagar Kadam <sagar.kadam at sifive.com>
> Cc: U-Boot Mailing List <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>; Jagan Teki <jagan at amarulasolutions.com>;
> Simon Glass <sjg at chromium.org>; twoerner at gmail.com; Patrick Wildt
> <patrick at blueri.se>; Fabio Estevam <festevam at gmail.com>; Weijie Gao
> <weijie.gao at mediatek.com>; Eugeniy Paltsev
> <Eugeniy.Paltsev at synopsys.com>
> Subject: Re: [PATCH 2/5] fu540: prci: use common reset indexes defined in
> binding header
> 
> [External Email] Do not click links or attachments unless you recognize the
> sender and know the content is safe
> 
> On Wed, Jun 24, 2020 at 11:01 AM Bin Meng <bmeng.cn at gmail.com>
> wrote:
> >
> > On Mon, Jun 22, 2020 at 8:28 PM Sagar Shrikant Kadam
> > <sagar.kadam at sifive.com> wrote:
> > >
> > > Indexes of reset signals available in PRCI driver are also defined
> > > in include/dt-bindings/clock/sifive-fu540-prci.h.
> > > So use those instead of defining new ones again within the
> > > fu540-prci driver.
> > >
> > > Signed-off-by: Sagar Shrikant Kadam <sagar.kadam at sifive.com>
> > > Reviewed-by: Pragnesh Patel <Pragnesh.patel at sifive.com>[A
> 
> Forget to mention, please fix the [A in the next version
> 
I will update fix for  "[A".
> > > ---
> > >  drivers/clk/sifive/fu540-prci.c | 16 ++++++----------
> > >  1 file changed, 6 insertions(+), 10 deletions(-)
> > >
> >
> > Reviewed-by: Bin Meng <bin.meng at windriver.com>

Thanks & BR,
Sagar


More information about the U-Boot mailing list