From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932181AbcFNRjU (ORCPT ); Tue, 14 Jun 2016 13:39:20 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:45844 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752935AbcFNRjS (ORCPT ); Tue, 14 Jun 2016 13:39:18 -0400 Date: Tue, 14 Jun 2016 10:39:10 -0700 From: Stephen Boyd To: Ricardo Ribalda Delgado Cc: Michael Turquette , s.hauer@pengutronix.de, jeremy.kerr@canonical.com, linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 0/3] Convert clk-fixed into module platform driver Message-ID: <20160614173910.GB28218@codeaurora.org> References: <1465381201-11537-1-git-send-email-ricardo.ribalda@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1465381201-11537-1-git-send-email-ricardo.ribalda@gmail.com> 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 06/08, Ricardo Ribalda Delgado wrote: > When clock providers are added to the device tree after of_clk_init is called > they are not added to the clock provider list. This makes that drivers such > as i2c-xiic.c fail to init, as they may depend on the unadded clock provider. Who's the provider here? It isn't clear to me why we're populating fixed factor and fixed rate clks from DT for i2c devices? Presumably there's an i2c device driver that should be populating clks from C code instead? -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project