Re: [PATCH] clk: x86: Do not gate clocks enabled by the firmware

[Andy Shevchenko <andriy.shevchenko@linux.intel.com>]

On Mon, 2017-07-24 at 15:49 +0200, Pierre-Louis Bossart wrote:
> On 7/11/17 1:54 AM, Darren Hart wrote:
> > On Tue, Jul 11, 2017 at 12:31:28AM +0200, Enric Balletbo Serra
> > wrote:

> > > Tested-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
> > 
> > Excellent, thank you Enric. Pierre, any objections to this going in
> > to
> > 4.13 now and stable back to 4.12?
> 
> I saw an update of this patch being merged while I was away on an 
> extended summer break, but out of curiosity what was the problem with
> no 
> audio on Baytrail? the code was added precisely to add support for
> 19.2 
> MHz on Baytrail and finally solve audio issues. I have no reports of 
> broken functionality with this 282a4e4 commit... Enric, do you have a 
> bugzilla or pointers?

TL;DR: We should respect firmware settings before blindly shut down
"unused" resources. (We used to have similar stuff in GPIO/pinctrl
driver for direct IRQ pins, where we shut up the pins which are used
directly by IOAPIC)

-- 
Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Intel Finland Oy