[PATCH linux dev-4.10] spi-nor: aspeed-smc: Fix flash window size message

Andrew Jeffery andrew at aj.id.au
Thu May 4 21:52:53 AEST 2017


On Thu, 2017-05-04 at 12:04 +0930, Joel Stanley wrote:
> When booting we see something like this:
> 
>  aspeed-smc 1e620000.flash-controller: Using IRQ 17
>  aspeed-smc 1e620000.flash-controller: DMA support disabled.
>  aspeed-smc 1e620000.flash-controller: mx25l25635e (32768 Kbytes)
>  aspeed-smc 1e620000.flash-controller: CE0 window [ 0x20000000- 0x22000000 ] 32MB
>  aspeed-smc 1e620000.flash-controller: CE1 window [ 0x22000000- 0x2a000000 ] 128MB
> 
> The lack of space after the first hex number haunts me every time I
> boot a BMC.
> 
> With this fix we see the pleasingly symmetrical:
> 
>  aspeed-smc 1e620000.flash-controller: CE0 window [ 0x20000000 - 0x22000000 ] 32MB
>  aspeed-smc 1e620000.flash-controller: CE1 window [ 0x22000000 - 0x2a000000 ] 128MB
> 
> > Signed-off-by: Joel Stanley <joel at jms.id.au>
> ---
>  drivers/mtd/spi-nor/aspeed-smc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/mtd/spi-nor/aspeed-smc.c b/drivers/mtd/spi-nor/aspeed-smc.c
> index 875b029198fc..23622bf62b6b 100644
> --- a/drivers/mtd/spi-nor/aspeed-smc.c
> +++ b/drivers/mtd/spi-nor/aspeed-smc.c
> @@ -709,7 +709,7 @@ static u32 chip_set_segment(struct aspeed_smc_chip *chip, u32 cs, u32 start,
> >  		size = end - start;
> >  	}
>  
> > -	dev_info(chip->nor.dev, "CE%d window [ 0x%.8x- 0x%.8x ] %dMB",
> > +	dev_info(chip->nor.dev, "CE%d window [ 0x%.8x - 0x%.8x ] %dMB",
>  		 cs, start, end, size >> 20);

NAK! I want *no* spaces!

Reviewed-by: Andrew Jeffery <andrew at aj.id.au>

>  
> >  	return size;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: This is a digitally signed message part
URL: <http://lists.ozlabs.org/pipermail/openbmc/attachments/20170504/b511ed7b/attachment.sig>


More information about the openbmc mailing list