[PATCH] ARM: Kirkwood: Iomega IX2-200 Device Tree Support Patch v2 Signed-off-by: Alan M Butler <alanbutty12 at gmail.com>

[Alan M Butler]

On 20 August 2012 13:44, Alan M Butler <alanbutty12 at gmail.com> wrote:
> On 20 August 2012 13:36, Fabio Estevam <festevam at gmail.com> wrote:
>> On Mon, Aug 20, 2012 at 9:30 AM, Alan M Butler <alanbutty12 at gmail.com> wrote:
>>
>>> im not sure but i only get that after applying the patch you sent me
>>> on the linux next from the 17th.
>>> i will try building a new linux-next later today and will send you what happens.
>>
>> Please do not drop the list from Cc.
>>
>>> would it be because i named the buttons power, reset and otb? instead
>>> of button at 1 and so on like the other kirkwood devices?
>>
>> I am not familiar with kirkwood device, sorry.
>
> sorry i forgot to cc the list.
>
> But as far as i can tell with the buttons the only difference between
> my device and the other's like the iconnect is that they named the
> buttons button at 1 and i named mine Power, Reset and OTB. Is it possible
> the capital letters are causing the problem?

i just tried your patch again with the latest linux-next dated 20th of
august with that patch you gave me and i am still getting the errors
on the gpio-keys this is the error:

[    4.838988] orion_gpio_is_valid: invalid GPIO 0
[    4.838999] orion_gpio_is_valid: invalid GPIO 0
[    4.839015] gpio-keys gpio-keys.2: Failed to request GPIO 0, error -22
[    4.845617] gpio-keys: probe of gpio-keys.2 failed with error -22

and this is how the keys are assigned in the dts file:

               Power {
                       label = "Power Button";
                       linux,code = <116>;
                       gpios = <&gpio0 14 1>;
               };
               Reset {
                       label = "Reset Button";
                       linux,code = <0x198>;
                       gpios = <&gpio0 12 1>;
               };
               OTB {
                       label = "OTB Button";
                       linux,code = <133>;
                       gpios = <&gpio1 3 1>;
               };

I hope this helps you figure out the problem.