[PATCH v1] ata: ahci_xgene: Add AHCI Support for second generation of APM X-Gene SoC

[Catalin Marinas]

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.

-- 
Catalin