Stephan GUILLOUX wrote:
> >>+static int read_flash(struct flashchip *flash, uint8_t *buf, uint32_t 
> >>offset, uint32_t length)
> >>+{
> >>+   int total_size = flash->total_size * 1024;
> >
> >total_size should be unsigned.
> 
> Agreed, but found some inconsistencies for total_size :

I disagree, this does not matter at all right now, and will not for
some time to come. Worry about it later.


//Peter

--
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to