Subject: Re: [PATCH RESEND] phy: exynos-mipi-video: simplify check for coupled
 phy status
To: Javier Martinez Canillas <javier@osg.samsung.com>,
        linux-kernel@vger.kernel.org, linux-samsung-soc@vger.kernel.org
Cc: Sylwester Nawrocki <s.nawrocki@samsung.com>,
        Kishon Vijay Abraham I <kishon@ti.com>,
        Krzysztof Kozlowski <krzk@kernel.org>
From: Marek Szyprowski <m.szyprowski@samsung.com>
Message-id: <35688181-78f1-b1c4-5824-e878052d88eb@samsung.com>
Date: Tue, 08 Nov 2016 07:27:12 +0100
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101
        Thunderbird/45.4.0
MIME-version: 1.0
In-reply-to: <2d8a48d1-2ecf-ad27-1e5a-a4233612c602@osg.samsung.com>
Content-type: text/plain; charset=utf-8; format=flowed
Content-transfer-encoding: 7bit
CMS-TYPE: 201P
References: <CGME20161107104844eucas1p2b5571119dff76e33d6ec038c359224f7@eucas1p2.samsung.com>
 <1478515712-650-1-git-send-email-m.szyprowski@samsung.com>
 <2d8a48d1-2ecf-ad27-1e5a-a4233612c602@osg.samsung.com>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 864
Lines: 24

Hi Javier,


On 2016-11-07 19:50, Javier Martinez Canillas wrote:
> On 11/07/2016 07:48 AM, Marek Szyprowski wrote:
>> There is no need to access regmap of coupled phy to check its state - such
>> information is already in the phy device itself, so use it directly. This
>> let us to avoid possible access to registers of the device in the disabled
>> power domain if the coupled phy is already disabled.
>>
> Do you think this patch needs also go to stable?

Nope, there is no such need, because the problem with potential power domain
is specific for Exynos 5433, which is not yet merged to mainline.

>> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
>> Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
>> ---
> Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com>

Best regards
-- 
Marek Szyprowski, PhD
Samsung R&D Institute Poland