Re: [PATCH v3 5/8] MIPS: jz4740: dts: Add bindings for the jz4740-wdt driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi James,

Le 11 mai 2018 11:52, James Hogan <jhogan@xxxxxxxxxx> a écrit :
>
> On Thu, May 10, 2018 at 08:47:48PM +0200, Paul Cercueil wrote: 
> > Also remove the watchdog platform_device from platform.c, since it 
> > wasn't used anywhere anyway. 
>
> Nit: it'd be slightly nicer IMO if the patch body was a superset of the 
> subject line. It's fine to repeat what the subject says since thats 
> meant to summarise the body. 
>
> > -struct platform_device jz4740_wdt_device = { 
>
> There's an extern in arch/mips/include/asm/mach-jz4740/platform.h that 
> should perhaps be removed also? 

You're right. It should be removed.

> Otherwise 
> Acked-by: James Hogan <jhogan@xxxxxxxxxx> 
>
> I'm happy to apply for 4.18 with that change if you want it to go 
> through the MIPS tree. 

Yes please!

> Cheers 
> James 

Thanks,
-Paul




[Index of Archives]     [Linux MIPS Home]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Linux]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux