diff mbox

ARM: dts: Update video-phy node with syscon phandle for Exynos3250

Message ID 1426762587-24923-1-git-send-email-b.michalska@samsung.com (mailing list archive)
State New, archived
Headers show

Commit Message

Beata Michalska March 19, 2015, 10:56 a.m. UTC
As a follow-up to recent changes to Exynos mipi video phy
driver, introducing support for PMU regmap in commit
e4b3d38088df6f3acd40 ("phy: exynos-video-mipi: Fix regression
by adding support for PMU regmap") add a syscon phandle to
video-phy node to bring back to life both MIPI DSI displays
and MIPI CSIS-2 camera sensors on Exynos3250.

Signed-off-by: Beata Michalska <b.michalska@samsung.com>
---
 arch/arm/boot/dts/exynos3250.dtsi |    1 +
 1 file changed, 1 insertion(+)

Comments

Krzysztof Kozlowski May 21, 2015, 1:57 a.m. UTC | #1
2015-03-19 19:56 GMT+09:00 Beata Michalska <b.michalska@samsung.com>:
> As a follow-up to recent changes to Exynos mipi video phy
> driver, introducing support for PMU regmap in commit
> e4b3d38088df6f3acd40 ("phy: exynos-video-mipi: Fix regression
> by adding support for PMU regmap") add a syscon phandle to
> video-phy node to bring back to life both MIPI DSI displays
> and MIPI CSIS-2 camera sensors on Exynos3250.
>
> Signed-off-by: Beata Michalska <b.michalska@samsung.com>

Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Tested-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>

Tested on Gear 2 (Exynos3250) board.

I'll queue it and send later to Kukjin.

Best regards,
Krzysztof
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Krzysztof Kozlowski May 21, 2015, 2:04 a.m. UTC | #2
2015-05-21 10:57 GMT+09:00 Krzysztof Kozlowski <k.kozlowski@samsung.com>:
> 2015-03-19 19:56 GMT+09:00 Beata Michalska <b.michalska@samsung.com>:
>> As a follow-up to recent changes to Exynos mipi video phy
>> driver, introducing support for PMU regmap in commit
>> e4b3d38088df6f3acd40 ("phy: exynos-video-mipi: Fix regression
>> by adding support for PMU regmap") add a syscon phandle to
>> video-phy node to bring back to life both MIPI DSI displays
>> and MIPI CSIS-2 camera sensors on Exynos3250.
>>
>> Signed-off-by: Beata Michalska <b.michalska@samsung.com>
>
> Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
> Tested-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>
> Tested on Gear 2 (Exynos3250) board.

Oh, I was too fast. It seems that "reg" property is not needed
anymore. Can you remove it?

Best regards,
Krzysztof
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Javier Martinez Canillas May 21, 2015, 7:25 a.m. UTC | #3
On Thu, May 21, 2015 at 4:04 AM, Krzysztof Kozlowski
<k.kozlowski@samsung.com> wrote:
> 2015-05-21 10:57 GMT+09:00 Krzysztof Kozlowski <k.kozlowski@samsung.com>:
>> 2015-03-19 19:56 GMT+09:00 Beata Michalska <b.michalska@samsung.com>:
>>> As a follow-up to recent changes to Exynos mipi video phy
>>> driver, introducing support for PMU regmap in commit
>>> e4b3d38088df6f3acd40 ("phy: exynos-video-mipi: Fix regression
>>> by adding support for PMU regmap") add a syscon phandle to
>>> video-phy node to bring back to life both MIPI DSI displays
>>> and MIPI CSIS-2 camera sensors on Exynos3250.
>>>
>>> Signed-off-by: Beata Michalska <b.michalska@samsung.com>
>>
>> Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>> Tested-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>>
>> Tested on Gear 2 (Exynos3250) board.
>
> Oh, I was too fast. It seems that "reg" property is not needed
> anymore. Can you remove it?
>

After removing the reg property as Krzysztof suggested.

Reviewed-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>

Best regards,
Javier
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
kgene@kernel.org May 23, 2015, 3:53 a.m. UTC | #4
On 05/21/15 16:25, Javier Martinez Canillas wrote:
> On Thu, May 21, 2015 at 4:04 AM, Krzysztof Kozlowski
> <k.kozlowski@samsung.com> wrote:
>> 2015-05-21 10:57 GMT+09:00 Krzysztof Kozlowski <k.kozlowski@samsung.com>:
>>> 2015-03-19 19:56 GMT+09:00 Beata Michalska <b.michalska@samsung.com>:
>>>> As a follow-up to recent changes to Exynos mipi video phy
>>>> driver, introducing support for PMU regmap in commit
>>>> e4b3d38088df6f3acd40 ("phy: exynos-video-mipi: Fix regression
>>>> by adding support for PMU regmap") add a syscon phandle to
>>>> video-phy node to bring back to life both MIPI DSI displays
>>>> and MIPI CSIS-2 camera sensors on Exynos3250.
>>>>
>>>> Signed-off-by: Beata Michalska <b.michalska@samsung.com>
>>>
>>> Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>>> Tested-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>>>
>>> Tested on Gear 2 (Exynos3250) board.
>>
>> Oh, I was too fast. It seems that "reg" property is not needed
>> anymore. Can you remove it?
>>
> 
> After removing the reg property as Krzysztof suggested.
> 
> Reviewed-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>
> 
will apply with changing as per you guys' review comment ;-)

Thanks,
Kukjin
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Beata Michalska May 26, 2015, 8:50 a.m. UTC | #5
Hi,

On 05/23/2015 05:53 AM, Kukjin Kim wrote:
> On 05/21/15 16:25, Javier Martinez Canillas wrote:
>> On Thu, May 21, 2015 at 4:04 AM, Krzysztof Kozlowski
>> <k.kozlowski@samsung.com> wrote:
>>> 2015-05-21 10:57 GMT+09:00 Krzysztof Kozlowski <k.kozlowski@samsung.com>:
>>>> 2015-03-19 19:56 GMT+09:00 Beata Michalska <b.michalska@samsung.com>:
>>>>> As a follow-up to recent changes to Exynos mipi video phy
>>>>> driver, introducing support for PMU regmap in commit
>>>>> e4b3d38088df6f3acd40 ("phy: exynos-video-mipi: Fix regression
>>>>> by adding support for PMU regmap") add a syscon phandle to
>>>>> video-phy node to bring back to life both MIPI DSI displays
>>>>> and MIPI CSIS-2 camera sensors on Exynos3250.
>>>>>
>>>>> Signed-off-by: Beata Michalska <b.michalska@samsung.com>
>>>>
>>>> Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>>>> Tested-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>>>>
>>>> Tested on Gear 2 (Exynos3250) board.
>>>
>>> Oh, I was too fast. It seems that "reg" property is not needed
>>> anymore. Can you remove it?
>>>
>>
>> After removing the reg property as Krzysztof suggested.
>>
>> Reviewed-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>
>>
> will apply with changing as per you guys' review comment ;-)
> 
> Thanks,
> Kukjin
> 

Apologies for not jumping in earlier, but  I've been away for a while,
with basically no e-mail access. Would You like me to resend the patch 
with the 'reg' property removed or will You handle it while applying
the change ?

Best Regards
Beata
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/boot/dts/exynos3250.dtsi b/arch/arm/boot/dts/exynos3250.dtsi
index ac6b0ae..6c89670 100644
--- a/arch/arm/boot/dts/exynos3250.dtsi
+++ b/arch/arm/boot/dts/exynos3250.dtsi
@@ -137,6 +137,7 @@ 
 			compatible = "samsung,s5pv210-mipi-video-phy";
 			reg = <0x10020710 8>;
 			#phy-cells = <1>;
+			syscon = <&pmu_system_controller>;
 		};
 
 		pd_cam: cam-power-domain@10023C00 {