[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <042e9067-7ef1-c942-0341-634963ff3fb0@osg.samsung.com>
Date: Fri, 20 Jan 2017 07:46:29 -0300
From: Javier Martinez Canillas <javier@....samsung.com>
To: Marek Szyprowski <m.szyprowski@...sung.com>,
linux-kernel@...r.kernel.org
Cc: Inki Dae <inki.dae@...sung.com>,
Andi Shyti <andi.shyti@...sung.com>,
Shuah Khan <shuahkh@....samsung.com>,
Mauro Carvalho Chehab <mchehab@...nel.org>,
Kukjin Kim <kgene@...nel.org>,
linux-samsung-soc@...r.kernel.org,
Sylwester Nawrocki <s.nawrocki@...sung.com>,
linux-media@...r.kernel.org, Krzysztof Kozlowski <krzk@...nel.org>,
linux-arm-kernel@...ts.infradead.org,
Ulf Hansson <ulf.hansson@...aro.org>
Subject: Re: [PATCH 2/2] [media] exynos-gsc: Fix imprecise external abort due
disabled power domain
Hello,
On 01/20/2017 07:06 AM, Javier Martinez Canillas wrote:
> Hello Marek,
>
> On 01/20/2017 05:37 AM, Marek Szyprowski wrote:
>
> [snip]
>>
>> Please send this patch instead of adding more clocks to the power domains.
>> This way we will avoid adding more dependencies to userspace (DT ABI).
>> Likely those clocks are also needed to access any device in that power
>> domains.
>>
>> Later, once the runtime PM for clocks get merged, a patch for exynos542x
>> clocks driver can be made to replace IS_CRITICAL with proper runtime
>> handling.
>>
>
> Ok, I thought that we didn't want to mark more clocks as CLK_IGNORE_UNUSED
> or CLK_IS_CRITICAL but I agree this can be done as a temporary workaround
> until a proper runtime PM based solution gets merged.
>
I posted following patch [0] for clk-exynos5420, so $SUBJECT can be dropped.
[0]: https://patchwork.kernel.org/patch/9527957/
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
Powered by blists - more mailing lists