From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932435AbeARSkJ (ORCPT ); Thu, 18 Jan 2018 13:40:09 -0500 Received: from vern.gendns.com ([206.190.152.46]:51937 "EHLO vern.gendns.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755258AbeARSkH (ORCPT ); Thu, 18 Jan 2018 13:40:07 -0500 Subject: Re: [PATCH v5 43/44] ARM: da8xx-dt: switch to device tree clocks To: Sekhar Nori , linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org Cc: Michael Turquette , Stephen Boyd , Rob Herring , Mark Rutland , Kevin Hilman , Adam Ford , linux-kernel@vger.kernel.org References: <1515378307-23718-1-git-send-email-david@lechnology.com> <1515378307-23718-4-git-send-email-david@lechnology.com> <8b30f1e6-74dc-3fd6-71a6-c9ceaf77a9d1@ti.com> From: David Lechner Message-ID: Date: Thu, 18 Jan 2018 12:40:05 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 MIME-Version: 1.0 In-Reply-To: <8b30f1e6-74dc-3fd6-71a6-c9ceaf77a9d1@ti.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - vern.gendns.com X-AntiAbuse: Original Domain - vger.kernel.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - lechnology.com X-Get-Message-Sender-Via: vern.gendns.com: authenticated_id: davidmain+lechnology.com/only user confirmed/virtual account not confirmed X-Authenticated-Sender: vern.gendns.com: davidmain@lechnology.com X-Source: X-Source-Args: X-Source-Dir: Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/18/2018 09:27 AM, Sekhar Nori wrote: > On Monday 08 January 2018 07:55 AM, David Lechner wrote: >> This removes all of the clock init code from da8xx-dt.c. This includes >> all of the OF_DEV_AUXDATA that was just used for looking up clocks. >> >> Note: You need to have clocks defined in your device tree or your system >> won't boot after this patch. > > I am not sure we can do this then, as we cannot break DT compatibility. > In the past, you have told me that you don't want the .dts changes and code changes in the same patch. In this case, if you apply either one separately, it will break clocks. It does not matter which one is first. So either we have to squash [PATCH v5 44/44] ARM: dts: da850: Add clocks into this patch or deal with the breakage.