> -----Original Message----- > From: Paul Walmsley [mailto:paul@xxxxxxxxx] > Sent: Monday, May 20, 2013 8:20 AM > To: Hiremath, Vaibhav > Cc: linux-omap@xxxxxxxxxxxxxxx; tony@xxxxxxxxxxx; Cousson, Benoit; > linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
