From mboxrd@z Thu Jan 1 00:00:00 1970 From: w.sang@pengutronix.de (Wolfram Sang) Date: Wed, 16 Nov 2011 14:51:26 +0100 Subject: [PATCH] arm: mx28: check for gated clocks when setting saif divider In-Reply-To: <65EE16ACC360FA4D99C96DC085B3F7722D4E65@039-SN1MPN1-002.039d.mgd.msft.net> References: <1315650583-4793-1-git-send-email-w.sang@pengutronix.de> <20111116132222.GD2596@pengutronix.de> <65EE16ACC360FA4D99C96DC085B3F7722D4E65@039-SN1MPN1-002.039d.mgd.msft.net> Message-ID: <20111116135125.GE2596@pengutronix.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, Nov 16, 2011 at 01:35:04PM +0000, Dong Aisheng-B29396 wrote: > > -----Original Message----- > > From: Wolfram Sang [mailto:w.sang at pengutronix.de] > > Sent: Wednesday, November 16, 2011 9:22 PM > > To: linux-arm-kernel at lists.infradead.org > > Cc: Sascha Hauer; Guo Shawn-R65073; Dong Aisheng-B29396 > > Subject: Re: [PATCH] arm: mx28: check for gated clocks when setting saif > > divider > > > > On Sat, Sep 10, 2011 at 12:29:43PM +0200, Wolfram Sang wrote: > > > Like with all other clocks, the divider for the SAIF devices should > > > not be altered when the clock is gated. Bail out when this is the case > > > like the other clocks do. > > > > > > Signed-off-by: Wolfram Sang > > > Cc: Sascha Hauer > > > Cc: Shawn Guo > > > Cc: Dong Aisheng-B29396 > > > --- > > > > > > Aisheng: I think this is the correct solution for clock-mx28.c. If > > > setting the rate of the saif clocks hit the error path, it should be > > fixed in the driver? > > > > Ping. Trying to catch up, has this been resolved meanwhile? > > > Sorry, I missed this patch. > > If I understand right, the convention way is to clk_set_rate() then > clk_enable().I f that, is it reasonable for driver to do something like: > Clk_enable -> clk_set_rate->clk_disable to set a proper rate, > then when needs the clock on, do clk_enable again? Confused, do you really mean enable -> set_rate -> disable? Because you can't set clocks when they are enabled? Well, to be honest, this is all is not very nice due to mxs restrictions. I chose this approach because this is the common pattern in all other clocks. There might be a better way of handling this, but then we'd need to adapt all other clocks as well. What we definately should not have is one kind of handling the 'already enabled' case for a few clocks and another kind for others. Regards, Wolfram -- Pengutronix e.K. | Wolfram Sang | Industrial Linux Solutions | http://www.pengutronix.de/ | -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 198 bytes Desc: Digital signature URL: