lkml.org 
[lkml]   [2016]   [Sep]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v5] mtd: spi-nor: Add support for S3AN spi-nor devices
Hi Boris

Thanks for your review.

On Tue, Sep 20, 2016 at 1:21 PM, Boris Brezillon
<boris.brezillon@free-electrons.com> wrote:
>> int sr, fsr;
>> - sr = spi_nor_sr_ready(nor);
>> +
>> + sr = nor->flags & SNOR_F_READY_XSR_RDY ? s3an_sr_ready(nor) :
>> + spi_nor_sr_ready(nor);
>
> Nit: I find
>
> if (nor->flags & SNOR_F_READY_XSR_RDY)
> sr = s3an_sr_ready(nor);
> else
> sr = spi_nor_sr_ready(nor);

Fixed in v6


>> +static loff_t spi_nor_s3an_addr_convert(struct spi_nor *nor, unsigned int addr)
>> +{
>> + unsigned int offset;
>> +
>> + offset = (nor->page_size == 264) ? (addr % 264) : (addr % 528);
>
> Why not just
>
> offset = addr % nor->page_size;
>

Because kbuild test robot does not like it ;)

make.cross ARCH=blackfin

All errors (new ones prefixed by >>):

drivers/built-in.o: In function `spi_nor_write':
>> drivers/mtd/spi-nor/spi-nor.c:1050: undefined reference to `__moddi3'

vim +1050 drivers/mtd/spi-nor/spi-nor.c



Thanks!
--
Ricardo Ribalda

\
 
 \ /
  Last update: 2016-09-20 17:47    [W:0.054 / U:0.296 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site