From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Boyd Subject: Re: [PATCHv4 1/5] clk: mvebu: support for 98DX3236 SoC Date: Mon, 23 Jan 2017 15:53:47 -0800 Message-ID: References: <20170113091222.7132-1-chris.packham@alliedtelesis.co.nz> <20170113091222.7132-2-chris.packham@alliedtelesis.co.nz> <20170121004819.GA8801@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Chris Packham Cc: "linux-arm-kernel@lists.infradead.org" , Michael Turquette , Rob Herring , Mark Rutland , "linux-clk@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" List-Id: devicetree@vger.kernel.org On 01/22/2017 11:53 PM, Chris Packham wrote: > Not 100% sure what you mean. Something like this? > > +static void __init of_mv98dx3236_cpu_clk_setup(struct device_node *node) > +{ > + of_clk_add_provider(node, of_clk_src_simple_get, NULL); > +} > > Seems to work as expected (i.e. does nothing, kernel boots/runs). > Yep that's what I mean. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project