Re: [PATCH v2 2/4] spi: Remove uses of #ifndef __U_BOOT__ from spi-mem.c

On Fri, Aug 7, 2020 at 10:43 PM Sean Anderson seanga2@gmail.com 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 seanga2@gmail.com

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 boris.brezillon@bootlin.com

*/

-#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.