From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1031043AbcCQRgo (ORCPT ); Thu, 17 Mar 2016 13:36:44 -0400 Received: from mout.kundenserver.de ([212.227.126.134]:57082 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1030862AbcCQRgl (ORCPT ); Thu, 17 Mar 2016 13:36:41 -0400 From: Arnd Bergmann To: Tejun Heo Cc: Brijesh Singh , linux-kernel@vger.kernel.org, hdegoede@redhat.com, linux-ide@vger.kernel.org, Graeme Gregory Subject: Re: [PATCH v2] ata: add AMD Seattle platform driver Date: Thu, 17 Mar 2016 18:36:19 +0100 Message-ID: <31951835.kMFH9CgruZ@wuerfel> User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; ) In-Reply-To: <20160316210713.GI21104@mtj.duckdns.org> References: <1452789071-4090-1-git-send-email-brijesh.singh@amd.com> <5709817.yQT8N3CaLj@wuerfel> <20160316210713.GI21104@mtj.duckdns.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:W08L+OBFJm6FCij5snj+itVFvDJPMhD+n7pvkzxUjsqYgktJFRu 8KcA7O2EH4UA8xwmTKyz4G6JeWtOTEZM2VZLzLdjqeiQzD53AxyEuPmeiZDhO7Bg3VEiQ3j tR4SLPjj4CC1bHiZtPZqPice5g7RgToN3kvHdWqc9/btwyUY4T3ox8ooggu5iIi7HRX42rq Zz66pfck2gR/wzXrX+cqg== X-UI-Out-Filterresults: notjunk:1;V01:K0:/SEm5G/0Duo=:K7JITRatbmTTQBnhtXrlTu T6Aa2luk7MFJ81jnfrjMm41UejboPA/WMR1JyCEtt5C/2/zCeJmGDJSsWqAOitfYBmDaRFLRM nkBrQdONT69ezXfoQ/W9LocO3vwQVN7JJwNobzbn/cVuiS4tLsLaRnVnFfNGIR0szx5UgOhdI sql4z6aEypAa0bPbjuaARgI3asjvUjnOjGazpUKZrKqgqpTS6nwHJVbXLJGPkRz35K5SXjgpf R0MKSKMJh67Ca/I/BdoB35SHPcT/0SwXb7Avlm1ZnoatucUDgEyFczzZySmChwMhkb7phNv43 qtacZIIU+HML4VvOJg/IWMngPyL+dQe5Nc3jv4mwKp7gECAGQii5bol2JR+iHGiBDsqHcFQJ/ bfTZJZP3Kxk/XEr4mSj8Zd0LLbKnSdhTFsQJaUl2U5poxExRAFEcYTQLdxxP8qb5CST3Pihpk SU63JyPP4vyWIDnzJ783eXwksRPLDaR3rk47UuR/TuoozhCxTC+o61GrUxJ10IhWjFo1B1uhg pgTo7UCtraR8Xjvp3/zqf75e93omyJJKK85gEVbVaXu0BDBZfpPaTmyGOuWeMJhTsHjkY3rD2 Bx6fXLUO8B9BFSRCfvRtxv+mLTics9nPCaNQVJ3AnfltQ+gXWEfMpzYc9qbXaCamHpeQEJebA vghdB8G/vCATZRUOzhmVqP7bKIVj994/ZM0SGfdX9ed4JUdeLRX/zzXR67euEO325IoZXoBq9 R0E6j35KnRHfoKZq Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wednesday 16 March 2016 14:07:13 Tejun Heo wrote: > Hello, Arnd. > > On Mon, Feb 01, 2016 at 09:14:17PM +0100, Arnd Bergmann wrote: > >> I am not debating on your AML call recommendation, it sounds like > >> a good idea however BIOS is already released hence its bit late to > >> add AML methods for this. I am seeking guidance on what can be > >> done in the given situation. I thought platform driver is one > >> option to get this feature enabled in kernel. > > > This is where we really need the ACPI maintainers to explain the > > general policy for dealing with firmware updates. > > > > I would assume that adding the feature in a later firmware version > > is a compatible change, and the feature is non-essential (the > > device will work fine with the generic SATA driver, except > > the LEDs don't blink), so it's not a big deal, it's just what > > you get for having the firmware shipped before the driver is > > reviewed (don't do that). > > So, if it were x86, I'd commit the custom driver without thinking too > much as ata drivers have always been working around bios issues (there > often wasn't any other recourse). If the hardware is already out > there and it's not too easy to roll out bios updates, from libata > side, I'm okay with having a custom driver to work around that. What > do you think? It's your call in the end. My main objection is to the fact that I have suggested a clean implementation for the normal DT based path that also fixes existing platforms that used to work in the past and were broken by the (long-ago) move from drivers/ide to drivers/ata, Brijesh has not implemented that but has instead continued pushing the hack for the ACPI mode that is still experimental on ARM64. Arnd