On Mon, Nov 14, 2016 at 04:47:26PM +0000, Andre Przywara wrote:
> Hi,
> 
> On 14/11/16 16:30, Jagan Teki wrote:
> > On Thu, Nov 3, 2016 at 6:28 AM, Andre Przywara <andre.przyw...@arm.com> 
> > wrote:
> >> Somehow an int returning function without a return statement sneaked
> >> in. Fix it.
> >>
> >> Signed-off-by: Andre Przywara <andre.przyw...@arm.com>
> >> ---
> >>  drivers/mtd/spi/sunxi_spi_spl.c | 3 ++-
> >>  1 file changed, 2 insertions(+), 1 deletion(-)
> >>
> >> diff --git a/drivers/mtd/spi/sunxi_spi_spl.c 
> >> b/drivers/mtd/spi/sunxi_spi_spl.c
> >> index 67c7edd..7502314 100644
> >> --- a/drivers/mtd/spi/sunxi_spi_spl.c
> >> +++ b/drivers/mtd/spi/sunxi_spi_spl.c
> >> @@ -158,9 +158,10 @@ static void spi0_disable_clock(void)
> >>                              (1 << AHB_RESET_SPI0_SHIFT));
> >>  }
> >>
> >> -static int spi0_init(void)
> >> +static void spi0_init(void)
> >>  {
> >>         unsigned int pin_function = SUNXI_GPC_SPI0;
> >> +
> > 
> > Space not needed or unrelated, please  remove this.
> 
> This is Linux coding style, which U-Boot adheres to.
> "WARNING: Missing a blank line after declarations"
> 
> I thought I should fix this since this is was in the context of this
> very simple patch and it improves readability.
> If this is too much, then please remove the line before committing.

Making things checkpatch clean is good, in the future please also
mention that in commit messages.  Thanks!

-- 
Tom

Attachment: signature.asc
Description: Digital signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to