From mboxrd@z Thu Jan 1 00:00:00 1970 From: sboyd@codeaurora.org (Stephen Boyd) Date: Wed, 10 Jun 2015 14:33:45 -0700 Subject: [PATCH 1/2 V3] clk: sirf: add CSR atlas7 clk and reset support In-Reply-To: References: <1431675587-8637-1-git-send-email-21cnbao@gmail.com> <20150515183029.GL31753@codeaurora.org> <20150521213324.GA1885@codeaurora.org> Message-ID: <20150610213345.GF29640@codeaurora.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 06/11, Barry Song wrote: > 2015-05-22 5:33 GMT+08:00 Stephen Boyd : > > On 05/16, Barry Song wrote: > >> 2015-05-16 2:30 GMT+08:00 Stephen Boyd : > >> > On 05/15, Barry Song wrote: > > > > It isn't really about making boot time faster, it's about using > > the proper linux device model for clock providers. That way when > > we want to support things like suspend/resume, deferred probe, > > devm_*(), sysfs, etc. we can use the device model instead of > > resorting to things like syscore_ops for suspend/resume or > > forgoing features entirely. > > > > my feeling is if the clock controller is an internal controller which > serve all controllers in the SoC, "deferred probe" might be yes for > almost all HW since all HW need clock. so it seems it makes "deferred > probe" has no meaning. clock controller seems to be 1st HW which needs > to be ready. That sounds like a probe ordering problem, and I agree most likely the clock driver will need to probe first in this case. It doesn't do anything about other problems like suspend/resume or other features that the device model provides solutions for though. Maybe we don't use these features, but it doesn't mean we should forgo the device model just because we don't need it. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project