On Fri, Aug 7, 2020 at 10:43 PM Sean Anderson <[email protected]> wrote:
>
> Preprocessing out large sections of the file is confusing and makes it
> difficult to follow the control flow. Presumably these were initially added
> to make porting easier, but this code has not been synced with Linux since
> it was introduced two years ago.
>
> Signed-off-by: Sean Anderson <[email protected]>
> ---
>
> Changes in v2:
> - Don't remove in-use headers
> - Split removal of EXPORT_SYMBOL_GPL into another patch
>
>  drivers/spi/spi-mem.c | 273 ------------------------------------------
>  1 file changed, 273 deletions(-)
>
> diff --git a/drivers/spi/spi-mem.c b/drivers/spi/spi-mem.c
> index c095ae9505..d7209837d0 100644
> --- a/drivers/spi/spi-mem.c
> +++ b/drivers/spi/spi-mem.c
> @@ -6,13 +6,6 @@
>   * Author: Boris Brezillon <[email protected]>
>   */
>
> -#ifndef __UBOOT__

I think these ifdef to track Linux change. Of course, I'm not fond of
these changes, if Vignesh is fine I'll push.

Jagan.

Reply via email to