From mboxrd@z Thu Jan 1 00:00:00 1970 From: Catalin Marinas Subject: Re: [PATCH v1] ata: ahci_xgene: Add AHCI Support for second generation of APM X-Gene SoC Date: Wed, 1 Apr 2015 19:18:19 +0100 Message-ID: <20150401181819.GD28616@e104818-lin.cambridge.arm.com> References: <1427899682-10912-1-git-send-email-stripathi@apm.com> <1427899682-10912-2-git-send-email-stripathi@apm.com> <20150401161336.GS9974@htj.duckdns.org> <20150401163116.GT9974@htj.duckdns.org> <20150401163956.GE24899@n2100.arm.linux.org.uk> <20150401170032.GC28616@e104818-lin.cambridge.arm.com> <20150401173029.GF24899@n2100.arm.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20150401173029.GF24899@n2100.arm.linux.org.uk> Sender: linux-scsi-owner@vger.kernel.org To: Russell King - ARM Linux Cc: devicetree@vger.kernel.org, mlangsdo@redhat.com, Suman Tripathi , arnd@arndb.de, linux-scsi@vger.kernel.org, jcm@redhat.com, olof@lixom.net, patches@apm.com, linux-ide@vger.kernel.org, Tejun Heo , linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org 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