Message ID | 1426762587-24923-1-git-send-email-b.michalska@samsung.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
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
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
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
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
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 --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 {
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(+)