[U-Boot] [PATCH 3/5] mtd: rawnand: denali: add reset handling

[Simon Goldschmidt]

Hi Miquel,

On Mon, Jan 28, 2019 at 10:22 AM Miquel Raynal
<miquel.raynal at bootlin.com> wrote:
>
> Hi Simon,
>
> Simon Goldschmidt <simon.k.r.goldschmidt at gmail.com> wrote on Mon, 28
> Jan 2019 10:16:28 +0100:
>
> > Hi Miquel,
> >
> > On Mon, Jan 28, 2019 at 10:08 AM Miquel Raynal
> > <miquel.raynal at bootlin.com> wrote:
> > >
> > > Hi Simon,
> > >
> > > Simon Goldschmidt <simon.k.r.goldschmidt at gmail.com> wrote on Fri, 25
> > > Jan 2019 21:30:49 +0100:
> > >
> > > > This adds reset handling to the devicetree-enabled denalid nand driver.
> > >
> > > "Denali NAND driver"
> >
> > Right.
> >
> > >
> > > >
> > > > For backwards compatibility, only a warning is printed when failing to
> > > > get reset handles.
> > > >
> > > > Signed-off-by: Simon Goldschmidt <simon.k.r.goldschmidt at gmail.com>
> > > > ---
> > > >
> > > >  drivers/mtd/nand/raw/denali_dt.c | 8 ++++++++
> > > >  1 file changed, 8 insertions(+)
> > > >
> > > > diff --git a/drivers/mtd/nand/raw/denali_dt.c b/drivers/mtd/nand/raw/denali_dt.c
> > > > index d384b974df..21c4540637 100644
> > > > --- a/drivers/mtd/nand/raw/denali_dt.c
> > > > +++ b/drivers/mtd/nand/raw/denali_dt.c
> > > > @@ -9,6 +9,7 @@
> > > >  #include <linux/io.h>
> > > >  #include <linux/ioport.h>
> > > >  #include <linux/printk.h>
> > > > +#include <reset.h>
> > > >
> > > >  #include "denali.h"
> > > >
> > > > @@ -64,6 +65,7 @@ static int denali_dt_probe(struct udevice *dev)
> > > >       const struct denali_dt_data *data;
> > > >       struct clk clk, clk_x, clk_ecc;
> > > >       struct resource res;
> > > > +     struct reset_ctl_bulk reset_bulk;
> > > >       int ret;
> > > >
> > > >       data = (void *)dev_get_driver_data(dev);
> > > > @@ -131,6 +133,12 @@ static int denali_dt_probe(struct udevice *dev)
> > > >               denali->clk_x_rate = 200000000;
> > > >       }
> > > >
> > > > +     ret = reset_get_bulk(bus, &reset_bulk);
> > > > +     if (ret)
> > > > +             dev_warn(bus, "Cant' get reset: %d\n", ret);
> > > > +     else
> > > > +             reset_deassert_bulk(&reset_bulk);
> > >
> > > Where are reset_get_bulk/reset_deassert_bulk() defined?
> >
> > In reset.h. That's why I added this include (see above)...
>
> Yes but I guess it is something that will be merged in 5.1? Maybe I
> will wait for the next release to take this one.

You mean Linux 5.1, right? This patch is for U-Boot, not Linux.

Regards,
Simon