Hi Guenter, Thanks a lot for this review and all the other ones. On Fri, 14 Mar 2014 11:04:03 -0700, Guenter Roeck wrote: > On Fri, Mar 14, 2014 at 04:07:28PM +0100, Jean Delvare wrote: > > The SC1200 is a SoC based on the Geode GX1 32-bit x86 processor, so > > its drivers are only needed on this architecture. > > The driver is named SC1200, but isn't really SC1200 specific but > supports the PC87307/PC97307. Not really sure if it is good idea > to remove support for it from 64 bit images. Can we say for sure > that there are no 64 bit systems using this chip ? Grmpf. The driver name is definitely confusing but I'm afraid you're right. Even more confusing is the existence of an SCx200_WDT driver, which is supposed to drive the same watchdog but as a PCI device? And that one does depend on SCx200 which in turn depends on X86_32. I don't quite get the point of having two drivers for the same device, but I'm probably missing something. Anyway, let's just forget about this patch then. Sorry for the noise. -- Jean Delvare SUSE L3 Support -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
