> -----Original Message----- > From: Paul Walmsley [mailto:[email protected]] > Sent: Monday, May 20, 2013 8:20 AM > To: Hiremath, Vaibhav > Cc: [email protected]; [email protected]; Cousson, Benoit; > [email protected] > Subject: Re: [PATCH 1/2] ARM: OMAP AM33XX: clock data: Enable clkout2 > as part of init > > On Tue, 7 May 2013, Vaibhav Hiremath wrote: > > > clkout2 comes out on the pad and is being used by various > > external on-board peripherals like, Audio codecs and stuff. > > So enable the clkout2 by default during init sequence itself. > > I don't like this: the clock should be enabled by the drivers for those > external peripherals, not enabled by default.
Neither do I. And certainly respective driver should make sure that he enables all required clocks. The gap here is, we only support DT only boot and currently clock bindings Is not coming from DT. Once we get there this needs to change. Also, note that, I do not expect impact on PM (atleast on AM335x). > So I think you should > reconsider the part of the patch that enables it upon init. But if you > really want to do this, I'm not inclined to stand in the way; you can > add > my ack. > Thanks for your ack, I will send out next version shortly. Thanks, Vaibhav -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
