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.
> Jagan.
>
Best regards,
Mario
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
