Messages in this thread | | | Date | Wed, 4 Nov 2015 08:06:28 -0800 | From | Sören Brinkmann <> | Subject | Re: [PATCH] rtc: Group Kconfig entries by vendor |
| |
Hi Alexandre,
On Wed, 2015-11-04 at 10:25AM +0100, Alexandre Belloni wrote: > On 03/11/2015 at 23:13:46 -0800, Soren Brinkmann wrote : > > The RTC entries are mostly grouped by vendor. Move the few outliers in > > place. > > Also, change the one occurrence of 'nxp' to 'NXP' to make all NXP > > entries consistent. > > > > Yeah, I've been contemplating reordering the Kconfig, the main issue > being that Philips and NXP is the same entity. And also, we have RTCs > manufactured by different vendors that use the same chip > (Abracon/Epson/Philips/Microcrystal) so I'm not sure what is clearer for > the end user :)
Yeah, I wasn't sure if this is wanted or how far to go. It was mainly triggered by the NXP devices that weren't grouped and differed in capitalization. From there I took it a little further. Just let met know if you want this somewhat different or not at all. It's just a suggestion and I find it helpful if things are somewhat ordered (I was thinking go full alphabetical sorting but didn't want to go that far without another opinion). But I'm fine either way.
Thanks, Sören
| |