From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marc Zyngier Subject: Re: [PATCHv2 0/3] arm64 xilinx zynqmp firmware interface Date: Wed, 16 Aug 2017 16:39:49 +0100 Message-ID: <43deeed7-32c8-7914-5f44-862ebbe14a3b@arm.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Content-Language: en-GB Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Michal Simek , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Arnd Bergmann Cc: edgar.iglesias-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org, Rob Herring , Punnaiah Choudary Kalluri , Jean Delvare , Dinh Nguyen , Rob Herring , Tero Kristo , Catalin Marinas , Olof Johansson , =?UTF-8?Q?S=c3=b6ren_Brinkmann?= , Kevin Hilman , Nishanth Menon , Thierry Reding , Kevin Brodsky , Will Deacon , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Alexander Graf , Sudeep Holla , Moritz Fischer , Michal Simek List-Id: devicetree@vger.kernel.org On 16/08/17 13:24, Michal Simek wrote: > Hi, > > xilinx is using this interface for very long time and we can't merge our > driver changes to Linux because of missing communication layer with > firmware. This interface was developed before scpi and scmi was > available. In connection to power management scpi and scmi are missing > pieces which we already use. There is a separate discussion how to > extend scmi to support all our use cases. So maybe we should wait and see where this discussion is going before we merge yet another firmware interface? > This simply patch is not adding any power management features but only > adding minimum functionality which are needed for drivers. I don't get it. 600 lines of firmware interface that isn't used by anything? Or is it? Needed by which driver? Thanks, M. -- Jazz is not dead. It just smells funny... -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html