[PATCH RESEND] ARM: dts: exynos3250: add JPEG codec device node

[PATCH RESEND] ARM: dts: exynos3250: add JPEG codec device node

[Jacek Anaszewski]

Signed-off-by: Jacek Anaszewski <j.anaszewski@samsung.com>
Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
Tested-by: Pankaj Dubey <pankaj.dubey@samsung.com>
Cc: Kukjin Kim <kgene@kernel.org>
---
 arch/arm/boot/dts/exynos3250.dtsi |   13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/arch/arm/boot/dts/exynos3250.dtsi b/arch/arm/boot/dts/exynos3250.dtsi
index 2246549..3c043d9 100644
--- a/arch/arm/boot/dts/exynos3250.dtsi
+++ b/arch/arm/boot/dts/exynos3250.dtsi
@@ -228,6 +228,19 @@
 			interrupts = <0 240 0>;
 		};
 
+		jpeg: codec@11830000 {
+			compatible = "samsung,exynos3250-jpeg";
+			reg = <0x11830000 0x1000>;
+			interrupts = <0 171 0>;
+			clocks = <&cmu CLK_JPEG>, <&cmu CLK_SCLK_JPEG>;
+			clock-names = "jpeg", "sclk";
+			samsung,power-domain = <&pd_cam>;
+			assigned-clocks = <&cmu CLK_MOUT_CAM_BLK>, <&cmu CLK_SCLK_JPEG>;
+			assigned-clock-rates = <0>, <150000000>;
+			assigned-clock-parents = <&cmu CLK_DIV_MPLL_PRE>;
+			status = "disabled";
+		};
+
 		fimd: fimd@11c00000 {
 			compatible = "samsung,exynos3250-fimd";
 			reg = <0x11c00000 0x30000>;
-- 
1.7.9.5

[PATCH] ARM: dts: exynos3250-rinato: add support for JPEG codec

[Jacek Anaszewski]

Signed-off-by: Jacek Anaszewski <j.anaszewski@samsung.com>
Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
Cc: Kukjin Kim <kgene@kernel.org>
---
 arch/arm/boot/dts/exynos3250-rinato.dts |    4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/exynos3250-rinato.dts b/arch/arm/boot/dts/exynos3250-rinato.dts
index 80aa8b4..9cfc9ec 100644
--- a/arch/arm/boot/dts/exynos3250-rinato.dts
+++ b/arch/arm/boot/dts/exynos3250-rinato.dts
@@ -480,6 +480,10 @@
 	status = "okay";
 };
 
+&jpeg {
+	status = "okay";
+};
+
 &mshc_0 {
 	#address-cells = <1>;
 	#size-cells = <0>;
-- 
1.7.9.5

Re: [PATCH RESEND] ARM: dts: exynos3250: add JPEG codec device node

[Krzysztof Kozlowski]

W dniu 24.01.2015 o 00:39, Jacek Anaszewski pisze:
> Signed-off-by: Jacek Anaszewski <j.anaszewski@samsung.com>
> Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
> Tested-by: Pankaj Dubey <pankaj.dubey@samsung.com>
> Cc: Kukjin Kim <kgene@kernel.org>
> ---
>  arch/arm/boot/dts/exynos3250.dtsi |   13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/exynos3250.dtsi b/arch/arm/boot/dts/exynos3250.dtsi
> index 2246549..3c043d9 100644
> --- a/arch/arm/boot/dts/exynos3250.dtsi
> +++ b/arch/arm/boot/dts/exynos3250.dtsi
> @@ -228,6 +228,19 @@
>  			interrupts = <0 240 0>;
>  		};
>  
> +		jpeg: codec@11830000 {
> +			compatible = "samsung,exynos3250-jpeg";
> +			reg = <0x11830000 0x1000>;
> +			interrupts = <0 171 0>;
> +			clocks = <&cmu CLK_JPEG>, <&cmu CLK_SCLK_JPEG>;
> +			clock-names = "jpeg", "sclk";
> +			samsung,power-domain = <&pd_cam>;

Please switch to generic binding: power-domains.

Best regards,
Krzysztof

Re: [PATCH RESEND] ARM: dts: exynos3250: add JPEG codec device node

[Jacek Anaszewski]

On 05/05/2015 07:00 AM, Krzysztof Kozlowski wrote:
> W dniu 24.01.2015 o 00:39, Jacek Anaszewski pisze:
>> Signed-off-by: Jacek Anaszewski <j.anaszewski@samsung.com>
>> Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
>> Tested-by: Pankaj Dubey <pankaj.dubey@samsung.com>
>> Cc: Kukjin Kim <kgene@kernel.org>
>> ---
>>   arch/arm/boot/dts/exynos3250.dtsi |   13 +++++++++++++
>>   1 file changed, 13 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/exynos3250.dtsi b/arch/arm/boot/dts/exynos3250.dtsi
>> index 2246549..3c043d9 100644
>> --- a/arch/arm/boot/dts/exynos3250.dtsi
>> +++ b/arch/arm/boot/dts/exynos3250.dtsi
>> @@ -228,6 +228,19 @@
>>   			interrupts = <0 240 0>;
>>   		};
>>
>> +		jpeg: codec@11830000 {
>> +			compatible = "samsung,exynos3250-jpeg";
>> +			reg = <0x11830000 0x1000>;
>> +			interrupts = <0 171 0>;
>> +			clocks = <&cmu CLK_JPEG>, <&cmu CLK_SCLK_JPEG>;
>> +			clock-names = "jpeg", "sclk";
>> +			samsung,power-domain = <&pd_cam>;
>
> Please switch to generic binding: power-domains.

I've sent updated version, thanks.

-- 
Best Regards,
Jacek Anaszewski