RE: [PATCH] ARM: shmobile: r7s72100: add restart handler

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

 



Hi Geert,

On Monday, February 13, 2017, Geert Uytterhoeven wrote: 
> > static const struct of_device_id rmobile_reset_of_match[] = {
> >         { .compatible = "renesas,sysc-rmobile", },
> >         { .compatible = "renesas,wdt-rmobile", },
> 
> renesas,r7s72100-wdt

I remember why I suggested "renesas,wdt-rmobile".

If I thought this WDT was going to be reused again in the RZ/A series,
wouldn't it make sense to just give it a generic name instead off a
specific part number?

And hence "renesas,wdt-rmobile" would be a better name than 
"renesas,r7s72100-wdt", "renesas,rza2-wdt", "renesas,rza3-wdt", etc... ?


Or, just stick with the part number and don't worry about it?


Thanks,
Chris





[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux