From mboxrd@z Thu Jan 1 00:00:00 1970 From: ariel@vanguardiasur.com.ar (Ariel D'Alessandro) Date: Sat, 04 Jul 2015 23:46:50 -0300 Subject: [PATCH v2 1/2] watchdog: NXP LPC18xx Watchdog Timer Driver In-Reply-To: References: <1435783930-20184-1-git-send-email-ariel@vanguardiasur.com.ar> <1435783930-20184-2-git-send-email-ariel@vanguardiasur.com.ar> <5594826F.6090203@roeck-us.net> Message-ID: <55989A9A.4090507@vanguardiasur.com.ar> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org El 02/07/15 a las 06:03, Joachim Eastwood escibi?: > On 2 July 2015 at 02:14, Guenter Roeck wrote: >> On 07/01/2015 04:04 PM, Joachim Eastwood wrote: >>> >>> Hi Ariel, >>> >> >>>> +/* >>>> + * NXP LPC18xx Watchdog Timer (WDT) >>> >>> >>> I think you should use "Windowed Watchdog Timer (WWDT)" here since >>> that is name from the user manual, even though the driver doesn't have >>> window functionality. >>> At least use wwdt for the compatible string, ie: "nxp,lpc1850-wwdt". >>> >>> I don't think Guenter asked you to remove the "Windowed" part last >>> time, but only asked what it meant. >> >> >> I did, separately, but I don't want to start a war of words either. >> No one will know what that term means, and I am personally not too hot >> having to explain it to others. > > Fine. > > My only request then is to make the compatible string > "nxp,lpc1850-wwdt", to leave the WWDT name somewhere. As this is the > name you will find through out the user manual for this hw block. Well, then I'll just change the compatible string to "nxp,lpc1850-wwdt".