On Wed, Apr 01, 2015 at 07:18:19PM +0100, Catalin Marinas wrote: > On Wed, Apr 01, 2015 at 06:30:30PM +0100, Russell King - ARM Linux wrote: > > On Wed, Apr 01, 2015 at 06:00:33PM +0100, Catalin Marinas wrote: > > > The driver in general should not be ARM specific, though it runs on an > > > ARMv8 platform. But looking at the patch it has some errata workarounds > > > triggered based on the CPU id (MIDR). That looks dodgy as it doesn't > > > even check the full ID, only the variant part. > > > > Okay, it's not ARM but an ARM64 driver, so that's your territory, not > > mine. :) > > Well, feel free to enter this territory ;). > > Anyway, what I meant is that such driver should not rely on CPU > identification at all, especially since it does not require some > specific CPU features but it tries to guess which SoC revision it is and > which device (not CPU) bugs have been fixed. Better if they follow Mark > R's idea to pass such information in DT for the device. > > So maybe the revert is not too bad an idea. The revert is completely sane. FWIW, NAK to the original patch using the MIDR, or any other patch which uses the MIDR for device (not CPU) errata workarounds. Mark. -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html