From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754697AbdGUUon (ORCPT ); Fri, 21 Jul 2017 16:44:43 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:47910 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752780AbdGUUol (ORCPT ); Fri, 21 Jul 2017 16:44:41 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 0F7D660AA6 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=sboyd@codeaurora.org Date: Fri, 21 Jul 2017 13:44:39 -0700 From: Stephen Boyd To: Rob Herring Cc: Neil Armstrong , jbrunet@baylibre.com, linux-clk@vger.kernel.org, linux-amlogic@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH 3/3] dt-bindings: clock: amlogic,gxbb-aoclkc: Update bindings Message-ID: <20170721204439.GJ19878@codeaurora.org> References: <1499336663-23875-1-git-send-email-narmstrong@baylibre.com> <1499336663-23875-4-git-send-email-narmstrong@baylibre.com> <20170710035031.rdbi64ookkbq6zeo@rob-hp-laptop> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170710035031.rdbi64ookkbq6zeo@rob-hp-laptop> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 07/09, Rob Herring wrote: > On Thu, Jul 06, 2017 at 12:24:23PM +0200, Neil Armstrong wrote: > > On the first revision of the bindings, only the gates + resets were known > > in the AO Clock HW, but more registers used to configures AO clock are known > > to be spread among the AO register space. > > This patch adds these registers to the Ao Clock bindings with direct access > > and shared extcon access. > > > > Signed-off-by: Neil Armstrong > > --- > > .../devicetree/bindings/clock/amlogic,gxbb-aoclkc.txt | 11 +++++++++-- > > 1 file changed, 9 insertions(+), 2 deletions(-) > > This looks like the binding might be too specific with a reg list of > single registers, and you should define a system controller node > instead. Depends on what else is in the "A0" block. > Agreed. Why can't we expand the size in DT and then access the registers directly in the driver. Hopefully it keeps working to apply the dts patch without the driver patch too (and vice-versa), because the kernel can only make a mapping as small as a page which would cover these newly added reg properties anyway. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project