On Thu, Apr 26, 2018 at 11:13 AM, Mario Six <mario....@gdsys.cc> wrote: > Hi Jagan, > > On Thu, Apr 26, 2018 at 7:34 AM, Jagan Teki <jagannadh.t...@gmail.com> wrote: >> On Thu, Apr 19, 2018 at 6:06 PM, Mario Six <mario....@gdsys.cc> wrote: >>> Support DM in the MPC8xxx SPI driver, and remove the legacy SPI >>> interface. >>> >>> Signed-off-by: Mario Six <mario....@gdsys.cc> >>> --- >>> drivers/spi/mpc8xxx_spi.c | 144 >>> ++++++++++++++++++++++++++++++++++------------ >>> 1 file changed, 107 insertions(+), 37 deletions(-) >>> --- >>> >>> Changes v1 -> v2: >>> * Removed legacy layer >>> * Sorted includes >>> * Now return -ETIMEDOUT on timeout during transfer >>> >>> --- >>> >>> diff --git a/drivers/spi/mpc8xxx_spi.c b/drivers/spi/mpc8xxx_spi.c >>> index 4aa5db8821..9c9dd67ec2 100644 >>> --- a/drivers/spi/mpc8xxx_spi.c >>> +++ b/drivers/spi/mpc8xxx_spi.c >>> @@ -6,10 +6,12 @@ >>> */ >>> >>> #include <common.h> >>> - >>> +#include <dm.h> >>> +#include <errno.h> >>> #include <malloc.h> >>> #include <spi.h> >>> #include <asm/mpc8xxx_spi.h> >>> +#include <asm-generic/gpio.h> >> >> [snip] >> >>> +}; >>> + >>> +static const struct udevice_id mpc8xxx_spi_ids[] = { >>> + { .compatible = "fsl,spi" }, >> >> Look like new binding right? don't we have suitable binding for this >> driver? or what is the equivalent driver for this in Linux? >> > > I took that compatible string from the Linux kernel, actually. See for example > > https://raw.githubusercontent.com/torvalds/linux/master/arch/powerpc/boot/dts/mpc8379_mds.dts > > There are more specific ones, like "fsl,mpc8610-spi", but "fsl,spi" is > the most generic.
So this driver is same as drivers/spi/spi-fsl-spi.c from Linux, is it? _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot