Hi Mark,

> -----Original Message-----
> From: Mark Brown <[email protected]>
> Sent: Wednesday, April 10, 2019 4:01 PM
> To: Naga Sureshkumar Relli <[email protected]>
> Cc: [email protected]; [email protected]; [email protected]; 
> linux-
> [email protected]; Michal Simek <[email protected]>; 
> [email protected]
> Subject: Re: [LINUX PATCH v2] spi: spi-mem: Fix build error without 
> CONFIG_SPI_MEM
> 
> On Wed, Apr 10, 2019 at 11:14:45AM +0530, Naga Sureshkumar Relli wrote:
> > When building with CONFIG_SPI_MEM is not set gc warns this:
> >
> > drivers/spi/spi-zynq-qspi.o: In function `zynq_qspi_supports_op':
> > spi-zynq-qspi.c:(.text+0x1da): undefined reference to 
> > `spi_mem_default_supports_op'
> >
> > Fixes: 67dca5e580f1 ("spi: spi-mem: Add support for Zynq QSPI
> > controller")
> >
> > Signed-off-by: YueHaibing <[email protected]>
> > Signed-off-by: Naga Sureshkumar Relli
> > <[email protected]>
> 
> Why is there a signed off by for YueHaibing here?  Did they write the patch 
> (in which case the
> authorship is incorrect)?
Yes, YueHaibing has wrote this patch.
I will update the Author name.

> 
> You've also got an extra blank line between the Fixes and the rest of the 
> tags which is unusual
> and might confuse some tools.
Ok I will remove that extra line.

Thanks,
Naga Sureshkumar Relli

Reply via email to