Re: [PATCH 1/4] ARM: dts: add support for hdmi to exynos4

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Sylwester,

On Mon, Jan 28, 2013 at 2:24 AM, Sylwester Nawrocki
<sylvester.nawrocki@xxxxxxxxx> wrote:
> On 01/25/2013 11:39 AM, Rahul Sharma wrote:
>>
>> Signed-off-by: Rahul Sharma<rahul.sharma@xxxxxxxxxxx>
>
>
> Please provide proper commit text.
>
>> Conflicts:
>>         arch/arm/mach-exynos/mach-exynos4-dt.c
>
>
> And make sure things like this are removed when submitting patches.
>

I will take care.
>
>> ---
>>   arch/arm/boot/dts/exynos4.dtsi         | 7 +++++++
>>   arch/arm/mach-exynos/mach-exynos4-dt.c | 2 ++
>>   2 files changed, 9 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/exynos4.dtsi
>> b/arch/arm/boot/dts/exynos4.dtsi
>> index e1347fc..6c2c79d 100644
>> --- a/arch/arm/boot/dts/exynos4.dtsi
>> +++ b/arch/arm/boot/dts/exynos4.dtsi
>> @@ -289,4 +289,11 @@
>>                         interrupts =<0 34 0>;
>>                 };
>>         };
>> +
>> +       hdmi {
>> +               compatible = "samsung,exynos4-hdmi14";
>
>
> What "-hdmi14" signifies here ? And to which Exynos4 SoC revisions this
> compatible property is supposed to apply ?
>

-hdmi14 signifies 1.4 compliant Hdmi IP like in Exynos4412, while
Exynos4210 has 1.3
compliant HDMI block. I have done a mistake here. I should have kept
it in 4412.dtsi. OR
I can move it to 4x12.dtsi but not sure whether all 4x12 socs have 1.4 hdmi.

>
>> +               reg =<0x12D00000 0x70000>;
>> +               interrupts =<0 95 0>;
>> +               hpd-gpio =<&gpx3 7 0xf 1 3>;
>> +       };
>
>
> New bindings need to be documented in Documentation/devicetree/bindings/.
>
It is present at Documentation/devicetree/bindings/drm/exynos/hdmi.txt.

Regards,
Rahul Sharma.

> --
>
> Thanks,
> Sylwester
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux