From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753331AbcLCREa (ORCPT ); Sat, 3 Dec 2016 12:04:30 -0500 Received: from mail-wj0-f193.google.com ([209.85.210.193]:34010 "EHLO mail-wj0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751488AbcLCRE1 (ORCPT ); Sat, 3 Dec 2016 12:04:27 -0500 Date: Sat, 3 Dec 2016 19:03:47 +0200 From: Krzysztof Kozlowski To: Arnd Bergmann Cc: Krzysztof Kozlowski , Olof Johansson , Kevin Hilman , arm@kernel.org, Kukjin Kim , linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, Javier Martinez Canillas Subject: Re: [GIT PULL 1/3] ARM: exynos: Soc/mach for v4.10 Message-ID: <20161203170347.GA3553@kozik-lap> References: <1479967709-6619-1-git-send-email-krzk@kernel.org> <1479967709-6619-2-git-send-email-krzk@kernel.org> <20161201183404.GA2134@kozik-lap> <57568492.9hbg5JCdec@wuerfel> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <57568492.9hbg5JCdec@wuerfel> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Dec 02, 2016 at 10:52:57PM +0100, Arnd Bergmann wrote: > On Thursday, December 1, 2016 8:34:04 PM CET Krzysztof Kozlowski wrote: > > On Thu, Nov 24, 2016 at 08:08:27AM +0200, Krzysztof Kozlowski wrote: > > > Hi, > > > > > > This contains previous dts branch because SCU node in dts is needed > > > prior to removing it from mach code. > > > > > > Below you will find full pull request and one stripped from dependency. > > > > > > > Hi Arnd, Kevin and Olof, > > > > What about this pull from the set? > > > > Sorry, I initially deferred it and then didn't get back to it. > > The dependency on the .dts changes made me a bit nervous about > taking it, mostly because the changelog fails to explain the > exact dependencies. > > This breaks compatibility with existing .dtb files, right? No, strictly speaking not. There was no dt-bindings change here, no DT properties for SCU before. We are converting our drivers to DTB so this is the same as before when switching for pinctrl, clocks or all other drivers to DT. We are not braking DTB ABI because there was no ABI around it before. Otherwise, one would say that lack of SCU DT node was an ABI. That is wrong, because DT should describe the hardware and SCU is in hardware. > What I'd like to see here is an explanation about: > > - what the upside of breaking compatibility is DTBs which do not have SCU are not proper because they skip that part of hardware. However we are breaking them in the way the SMP won't work there. It is not an ABI break, as I mentioned above. > - what exactly stops working with an old dtb > - why we don't keep a fallback for handling old dtb files What is the point for it? This is not an ABI break. Even if it was, Samsung guys don't care for ABI breaks at all (and in fact we wanted to mark the platform experimental...). > It would also be helpful to have a separate pull request for > the commits require the new dtb, and the stuff that is unrelated. I can do that but the pull will be small. Best regards, Krzysztof