On Fri, Dec 5, 2014 at 1:34 AM, Hans de Goede <[email protected]> wrote:

> Hi,
>
> On 12/04/2014 12:06 AM, vishnupatekar wrote:
>
>> ---
>>   arch/arm/boot/dts/sun7i-a20-olinuxino-lime2.dts |   10 ++++++++
>>   arch/arm/boot/dts/sun7i-a20.dtsi                |   30
>> ++++++++++++++++++++++-
>>   2 files changed, 39 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2.dts
>> b/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2.dts
>> index ed364d5..374897b 100644
>> --- a/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2.dts
>> +++ b/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2.dts
>> @@ -112,6 +112,16 @@
>>                         pinctrl-0 = <&uart0_pins_a>;
>>                         status = "okay";
>>                 };
>> +               ps20: ps2@0x01c2a000 {
>> +                       pinctrl-names = "default";
>> +                       pinctrl-0 = <&ps2_0_pins>;
>> +                       status = "okay";
>> +               };
>> +               ps21: ps2@0x01c2a400 {
>> +                       pinctrl-names = "default";
>> +                       pinctrl-0 = <&ps2_1_pins>;
>> +                       status = "okay";
>> +               };
>>
>
> I don't think this should be enabled by default, since there
> is no ps2 connector on the board, and the pins can be used
> in a different manner too.
>
I do think it is useful to have it in there as an example for people who
> want to use
> it, but commented out, please also add a comment stating that using ps20
> conflicts
> with using the hdmi connector.
>
> Okie, For lime2 board dts file, I'll comment out these ps20 and ps21 nodes
stating that In case uncommented this will confilt with hdmi connector.

This also will be included in next patch version.



> Regards,
>
> Hans
>
>
>
>>                 i2c0: i2c@01c2ac00 {
>>                         pinctrl-names = "default";
>> diff --git a/arch/arm/boot/dts/sun7i-a20.dtsi
>> b/arch/arm/boot/dts/sun7i-a20.dtsi
>> index 8605f2b..c77635d 100644
>> --- a/arch/arm/boot/dts/sun7i-a20.dtsi
>> +++ b/arch/arm/boot/dts/sun7i-a20.dtsi
>> @@ -357,7 +357,7 @@
>>
>>                 usb_clk: clk@01c200cc {
>>                         #clock-cells = <1>;
>> -                       #reset-cells = <1>;
>> +                       #reset-cells = <1>;
>>                         compatible = "allwinner,sun4i-a10-usb-clk";
>>                         reg = <0x01c200cc 0x4>;
>>                         clocks = <&pll6 1>;
>> @@ -852,6 +852,19 @@
>>                                     allwinner,drive = <0>;
>>                                     allwinner,pull = <0>;
>>                         };
>> +                       ps2_0_pins: ps2_0@0 {
>> +                                   allwinner,pins = "PI20","PI21";
>> +                                   allwinner,function = "ps2";
>> +                                   allwinner,drive = <0>;
>> +                                   allwinner,pull = <0>;
>> +                       };
>> +                       ps2_1_pins: ps2_1@0 {
>> +                                   allwinner,pins = "PH12","PH13";
>> +                                   allwinner,function = "ps2";
>> +                                   allwinner,drive = <0>;
>> +                                   allwinner,pull = <0>;
>> +                       };
>> +
>>                 };
>>
>>                 timer@01c20c00 {
>> @@ -1079,5 +1092,20 @@
>>                         #interrupt-cells = <3>;
>>                         interrupts = <1 9 0xf04>;
>>                 };
>> +               ps20: ps2@0x01c2a000 {
>> +                       compatible = "allwinner,sun7i-a20-ps2";
>> +                       reg = <0x01c2a000 0x400>;
>> +                       interrupts = <0 62 4>;
>> +                       clocks = <&apb1_gates 6>;
>> +                       status = "disabled";
>> +               };
>> +               ps21: ps2@0x01c2a400 {
>> +                       compatible = "allwinner,sun7i-a20-ps2";
>> +                       reg = <0x01c2a400 0x400>;
>> +                       interrupts = <0 63 4>;
>> +                       clocks = <&apb1_gates 7>;
>> +                       status = "disabled";
>> +               };
>> +
>>         };
>>   };
>>
>>

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to