From mboxrd@z Thu Jan  1 00:00:00 1970
From: Miquel Raynal <miquel.raynal@bootlin.com>
Date: Mon, 28 Jan 2019 10:22:55 +0100
Subject: [U-Boot] [PATCH 3/5] mtd: rawnand: denali: add reset handling
In-Reply-To: <CAAh8qsyyF2L-1WEW=aRJJm7nd=xEKHxM1ho3J1OyD0SJZJCCUQ@mail.gmail.com>
References: <20190125203051.10943-1-simon.k.r.goldschmidt@gmail.com>
 <20190125203051.10943-4-simon.k.r.goldschmidt@gmail.com>
 <20190128100852.461c2573@xps13>
 <CAAh8qsyyF2L-1WEW=aRJJm7nd=xEKHxM1ho3J1OyD0SJZJCCUQ@mail.gmail.com>
Message-ID: <20190128102255.08c90c32@xps13>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
To: u-boot@lists.denx.de

Hi Simon,

Simon Goldschmidt <simon.k.r.goldschmidt@gmail.com> wrote on Mon, 28
Jan 2019 10:16:28 +0100:

> Hi Miquel,
>=20
> On Mon, Jan 28, 2019 at 10:08 AM Miquel Raynal
> <miquel.raynal@bootlin.com> wrote:
> >
> > Hi Simon,
> >
> > Simon Goldschmidt <simon.k.r.goldschmidt@gmail.com> wrote on Fri, 25
> > Jan 2019 21:30:49 +0100:
> > =20
> > > This adds reset handling to the devicetree-enabled denalid nand drive=
r. =20
> >
> > "Denali NAND driver" =20
>=20
> Right.
>=20
> > =20
> > >
> > > For backwards compatibility, only a warning is printed when failing to
> > > get reset handles.
> > >
> > > Signed-off-by: Simon Goldschmidt <simon.k.r.goldschmidt@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 =3D (void *)dev_get_driver_data(dev);
> > > @@ -131,6 +133,12 @@ static int denali_dt_probe(struct udevice *dev)
> > >               denali->clk_x_rate =3D 200000000;
> > >       }
> > >
> > > +     ret =3D reset_get_bulk(bus, &reset_bulk);
> > > +     if (ret)
> > > +             dev_warn(bus, "Cant' get reset: %d\n", ret);
> > > +     else
> > > +             reset_deassert_bulk(&reset_bulk); =20
> >
> > Where are reset_get_bulk/reset_deassert_bulk() defined? =20
>=20
> 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.


Thanks,
Miqu=C3=A8l