[PATCH v2 1/3] spi: xilinx: Remove bitbang and register with spi core

Ricardo Ribalda Delgado ricardo.ribalda at gmail.com
Mon Jun 13 08:14:18 PDT 2016


Hi Naga


There is some functionality removed by this patch:

- Auto Detection of fifo size.
- Non irq mode (which is a great improvement on some systems)


Also some comments have been removed for apparently no reason:

> -       u32 cs_inactive;        /* Level of the CS pins when inactive*/
> +       int buffer_size;
> +       u32 cs_inactive;
>

And the rename of some vars increases the patch size:
-       struct xilinx_spi *xspi = spi_master_get_devdata(spi->master);
+       struct xilinx_spi *xqspi = spi_master_get_devdata(qspi->master);

I am not a big fan of code generation via macros:
#define XSPI_FIFO_READ(size, type)
Interpreting kernel oops or greping the code is much more difficult.
Why not using function parameters instead?

Could you add me in cc next time? Although I am not in the credit file
I have authored a good bunch of the file and also we relay on this
driver on our products.



Best regards!



More information about the linux-arm-kernel mailing list