From mboxrd@z Thu Jan  1 00:00:00 1970
From: Miquel Raynal <miquel.raynal@bootlin.com>
Date: Wed, 11 Jul 2018 15:55:58 +0200
Subject: [U-Boot] [RFC PATCH 10/20] spi: Extend the core to ease
 integration of SPI memory controllers
In-Reply-To: <CAD6G_RSB4f1OBhkjDkG9S44cU+q75yB+3c-t-SFwVQzGjeCocg@mail.gmail.com>
References: <20180606153040.21397-1-miquel.raynal@bootlin.com>
 <20180606153040.21397-11-miquel.raynal@bootlin.com>
 <CAD6G_RSB4f1OBhkjDkG9S44cU+q75yB+3c-t-SFwVQzGjeCocg@mail.gmail.com>
Message-ID: <20180711155558.4be24a56@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 Jagan,

Jagan Teki <jagannadh.teki@gmail.com> wrote on Fri, 6 Jul 2018 17:02:22
+0530:

> On Wed, Jun 6, 2018 at 9:00 PM, Miquel Raynal <miquel.raynal@bootlin.com>=
 wrote:
> > From: Boris Brezillon <boris.brezillon@bootlin.com>
> >
> > Some controllers are exposing high-level interfaces to access various
> > kind of SPI memories. Unfortunately they do not fit in the current
> > spi_controller model and usually have drivers placed in
> > drivers/mtd/spi-nor which are only supporting SPI NORs and not SPI
> > memories in general.
> >
> > This is an attempt at defining a SPI memory interface which works for
> > all kinds of SPI memories (NORs, NANDs, SRAMs).
> >
> > Signed-off-by: Boris Brezillon <boris.brezillon@bootlin.com>
> > Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
> > ---
> >  drivers/spi/Kconfig   |   7 +
> >  drivers/spi/Makefile  |   1 +
> >  drivers/spi/spi-mem.c | 500 ++++++++++++++++++++++++++++++++++++++++++=
++++++++
> >  include/spi-mem.h     | 258 ++++++++++++++++++++++++++
> >  include/spi.h         |  11 ++
> >  5 files changed, 777 insertions(+)
> >  create mode 100644 drivers/spi/spi-mem.c
> >  create mode 100644 include/spi-mem.h
> >
> > diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
> > index 235a8c7d73..0ee371b2d9 100644
> > --- a/drivers/spi/Kconfig
> > +++ b/drivers/spi/Kconfig
> > @@ -15,6 +15,13 @@ config DM_SPI
> >
> >  if DM_SPI
> >
> > +config SPI_MEM
> > +       bool "SPI memory extension"
> > +       help
> > +         Enable this option if you want to enable the SPI memory exten=
sion.
> > +         This extension is meant to simplify interaction with SPI memo=
ries
> > +         by providing an high-level interface to send memory-like comm=
ands.
> > +
> >  config ALTERA_SPI
> >         bool "Altera SPI driver"
> >         help
> > diff --git a/drivers/spi/Makefile b/drivers/spi/Makefile
> > index 4b6000fd9a..982529a0e6 100644
> > --- a/drivers/spi/Makefile
> > +++ b/drivers/spi/Makefile
> > @@ -10,6 +10,7 @@ ifdef CONFIG_DM_SPI
> >  obj-y +=3D spi-uclass.o
> >  obj-$(CONFIG_SANDBOX) +=3D spi-emul-uclass.o
> >  obj-$(CONFIG_SOFT_SPI) +=3D soft_spi.o
> > +obj-$(CONFIG_SPI_MEM) +=3D spi-mem.o
> >  else
> >  obj-y +=3D spi.o
> >  obj-$(CONFIG_SOFT_SPI) +=3D soft_spi_legacy.o
> > diff --git a/drivers/spi/spi-mem.c b/drivers/spi/spi-mem.c
> > new file mode 100644
> > index 0000000000..1aabe56819
> > --- /dev/null
> > +++ b/drivers/spi/spi-mem.c
> > @@ -0,0 +1,500 @@
> > +// SPDX-License-Identifier: GPL-2.0+
> > +/*
> > + * Copyright (C) 2018 Exceet Electronics GmbH
> > + * Copyright (C) 2018 Bootlin
> > + *
> > + * Author: Boris Brezillon <boris.brezillon@bootlin.com>
> > + */
> > +
> > +#ifndef __UBOOT__
> > +#include <linux/dmaengine.h>
> > +#include <linux/pm_runtime.h>
> > +#include "internals.h"
> > +#else
> > +#include <spi.h>
> > +#include <spi-mem.h>
> > +#endif
> > +
> > +#ifndef __UBOOT__ =20
>=20
> I would like remove Linux stuff atleast on this file becuase it's
> difficult for me to read or review the code and also driver/spi have
> fully u-boot dm stuff. I know it's easy for Linux sync but for this we
> can do manual sync what ever need. I'm on something what from Linux.

I'm not sure this is a wise idea.

And I don't understand how "driver/spi have fully u-boot dm stuff" is
related in any manner to this request.

But okay, if you commit to backport the changes/fixes on spi-mem from
Linux yourself, let's remove them.

Thanks,
Miqu=C3=A8l