2014-01-07 17:53:04

by Santosh Shilimkar

[permalink] [raw]
Subject: Re: [PATCH v3 0/2] gpio: davinci: reuse for keystone arch

Sekhar,

On Tuesday 24 December 2013 06:41 AM, Grygorii Strashko wrote:
> This series is intended to update Davinci GPIO driver and reuse
> it for Keystone SoCs, because Keystone uses the similar GPIO IP like Davinci.
> Keystone GPIO IP: supports:
> - up to 32 GPIO lines;
> - only unbanked irqs;
>
> See Documentation:
> Keystone - http://www.ti.com/lit/ug/sprugv1/sprugv1.pdf
>
> This series based on:
> https://git.kernel.org/cgit/linux/kernel/git/nsekhar/linux-davinci.git/log/?h=v3.14/gpio
>
> Changes in v3:
> - fixed code, changed by mistake; fixed sparse warning
> Changes in v2:
> - minor comments applied, no functional changes
>
> v2: https://lkml.org/lkml/2013/12/18/135
> v1: https://lkml.org/lkml/2013/12/12/366
>
> Cc: Linus Walleij <[email protected]>
> Cc: Alexandre Courbot <[email protected]>
> Cc: Sekhar Nori <[email protected]>
> Cc: Santosh Shilimkar <[email protected]>
>
> Grygorii Strashko (2):
> gpio: davinci: don't create irq_domain in case of unbanked irqs
> gpio: davinci: reuse for Keystone SoC
>
> .../devicetree/bindings/gpio/gpio-davinci.txt | 4 +-
> drivers/gpio/gpio-davinci.c | 82 ++++++++++++++------
> 2 files changed, 61 insertions(+), 25 deletions(-)
>
Have you picked up the $subject series in your queue ?

Regards,
Santosh


2014-01-08 04:08:19

by Sekhar Nori

[permalink] [raw]
Subject: Re: [PATCH v3 0/2] gpio: davinci: reuse for keystone arch

On Tuesday 07 January 2014 11:22 PM, Santosh Shilimkar wrote:
> Sekhar,
>
> On Tuesday 24 December 2013 06:41 AM, Grygorii Strashko wrote:
>> This series is intended to update Davinci GPIO driver and reuse
>> it for Keystone SoCs, because Keystone uses the similar GPIO IP like Davinci.
>> Keystone GPIO IP: supports:
>> - up to 32 GPIO lines;
>> - only unbanked irqs;
>>
>> See Documentation:
>> Keystone - http://www.ti.com/lit/ug/sprugv1/sprugv1.pdf
>>
>> This series based on:
>> https://git.kernel.org/cgit/linux/kernel/git/nsekhar/linux-davinci.git/log/?h=v3.14/gpio
>>
>> Changes in v3:
>> - fixed code, changed by mistake; fixed sparse warning
>> Changes in v2:
>> - minor comments applied, no functional changes
>>
>> v2: https://lkml.org/lkml/2013/12/18/135
>> v1: https://lkml.org/lkml/2013/12/12/366
>>
>> Cc: Linus Walleij <[email protected]>
>> Cc: Alexandre Courbot <[email protected]>
>> Cc: Sekhar Nori <[email protected]>
>> Cc: Santosh Shilimkar <[email protected]>
>>
>> Grygorii Strashko (2):
>> gpio: davinci: don't create irq_domain in case of unbanked irqs
>> gpio: davinci: reuse for Keystone SoC
>>
>> .../devicetree/bindings/gpio/gpio-davinci.txt | 4 +-
>> drivers/gpio/gpio-davinci.c | 82 ++++++++++++++------
>> 2 files changed, 61 insertions(+), 25 deletions(-)
>>
> Have you picked up the $subject series in your queue ?

Not yet, at least on the new compatible introduction, I need an ack from
DT folks.

I am happy with the patches though and have tested them as well, In case
I do not get an ack from 2/2 in time, I can at least send 1/2 for
inclusion after my first gpio pull request to ARM-SoC gets pulled.

Thanks,
Sekhar

2014-01-08 14:08:50

by Santosh Shilimkar

[permalink] [raw]
Subject: Re: [PATCH v3 0/2] gpio: davinci: reuse for keystone arch

On Tuesday 07 January 2014 11:06 PM, Sekhar Nori wrote:
> On Tuesday 07 January 2014 11:22 PM, Santosh Shilimkar wrote:
>> Sekhar,
>>
>> On Tuesday 24 December 2013 06:41 AM, Grygorii Strashko wrote:
>>> This series is intended to update Davinci GPIO driver and reuse
>>> it for Keystone SoCs, because Keystone uses the similar GPIO IP like Davinci.
>>> Keystone GPIO IP: supports:
>>> - up to 32 GPIO lines;
>>> - only unbanked irqs;
>>>
>>> See Documentation:
>>> Keystone - http://www.ti.com/lit/ug/sprugv1/sprugv1.pdf
>>>
>>> This series based on:
>>> https://git.kernel.org/cgit/linux/kernel/git/nsekhar/linux-davinci.git/log/?h=v3.14/gpio
>>>
>>> Changes in v3:
>>> - fixed code, changed by mistake; fixed sparse warning
>>> Changes in v2:
>>> - minor comments applied, no functional changes
>>>
>>> v2: https://lkml.org/lkml/2013/12/18/135
>>> v1: https://lkml.org/lkml/2013/12/12/366
>>>
>>> Cc: Linus Walleij <[email protected]>
>>> Cc: Alexandre Courbot <[email protected]>
>>> Cc: Sekhar Nori <[email protected]>
>>> Cc: Santosh Shilimkar <[email protected]>
>>>
>>> Grygorii Strashko (2):
>>> gpio: davinci: don't create irq_domain in case of unbanked irqs
>>> gpio: davinci: reuse for Keystone SoC
>>>
>>> .../devicetree/bindings/gpio/gpio-davinci.txt | 4 +-
>>> drivers/gpio/gpio-davinci.c | 82 ++++++++++++++------
>>> 2 files changed, 61 insertions(+), 25 deletions(-)
>>>
>> Have you picked up the $subject series in your queue ?
>
> Not yet, at least on the new compatible introduction, I need an ack from
> DT folks.
>
I noticed that but the usual 2 weeks period to get ack is over I guess ;-)
The DT part is really trivial as well but I let you decide.

> I am happy with the patches though and have tested them as well, In case
> I do not get an ack from 2/2 in time, I can at least send 1/2 for
> inclusion after my first gpio pull request to ARM-SoC gets pulled.
>
Would be great to get both of them but if not both at least 1/2.

Regards,
Santosh

2014-01-09 10:34:05

by Sekhar Nori

[permalink] [raw]
Subject: Re: [PATCH v3 0/2] gpio: davinci: reuse for keystone arch

On Wednesday 08 January 2014 07:38 PM, Santosh Shilimkar wrote:
> On Tuesday 07 January 2014 11:06 PM, Sekhar Nori wrote:
>> On Tuesday 07 January 2014 11:22 PM, Santosh Shilimkar wrote:
>>> Sekhar,
>>>
>>> On Tuesday 24 December 2013 06:41 AM, Grygorii Strashko wrote:
>>>> This series is intended to update Davinci GPIO driver and reuse
>>>> it for Keystone SoCs, because Keystone uses the similar GPIO IP like Davinci.
>>>> Keystone GPIO IP: supports:
>>>> - up to 32 GPIO lines;
>>>> - only unbanked irqs;
>>>>
>>>> See Documentation:
>>>> Keystone - http://www.ti.com/lit/ug/sprugv1/sprugv1.pdf
>>>>
>>>> This series based on:
>>>> https://git.kernel.org/cgit/linux/kernel/git/nsekhar/linux-davinci.git/log/?h=v3.14/gpio
>>>>
>>>> Changes in v3:
>>>> - fixed code, changed by mistake; fixed sparse warning
>>>> Changes in v2:
>>>> - minor comments applied, no functional changes
>>>>
>>>> v2: https://lkml.org/lkml/2013/12/18/135
>>>> v1: https://lkml.org/lkml/2013/12/12/366
>>>>
>>>> Cc: Linus Walleij <[email protected]>
>>>> Cc: Alexandre Courbot <[email protected]>
>>>> Cc: Sekhar Nori <[email protected]>
>>>> Cc: Santosh Shilimkar <[email protected]>
>>>>
>>>> Grygorii Strashko (2):
>>>> gpio: davinci: don't create irq_domain in case of unbanked irqs
>>>> gpio: davinci: reuse for Keystone SoC
>>>>
>>>> .../devicetree/bindings/gpio/gpio-davinci.txt | 4 +-
>>>> drivers/gpio/gpio-davinci.c | 82 ++++++++++++++------
>>>> 2 files changed, 61 insertions(+), 25 deletions(-)
>>>>
>>> Have you picked up the $subject series in your queue ?
>>
>> Not yet, at least on the new compatible introduction, I need an ack from
>> DT folks.
>>
> I noticed that but the usual 2 weeks period to get ack is over I guess ;-)
> The DT part is really trivial as well but I let you decide.

I just realize that Rob's e-mail address bounces. I have added his
updated address now and hopefully he will see this to provide his ack.

Rob,

We need your ack on 2/2 of this series. If you do not have the patch, I
can forward it to you.

>
>> I am happy with the patches though and have tested them as well, In case
>> I do not get an ack from 2/2 in time, I can at least send 1/2 for
>> inclusion after my first gpio pull request to ARM-SoC gets pulled.
>>
> Would be great to get both of them but if not both at least 1/2.

I had already sent it with my first pull request. Hmph. Everything
except 2/2 of tis series should be in linux-next now.

Thanks,
Sekhar

2014-01-09 14:11:18

by Santosh Shilimkar

[permalink] [raw]
Subject: Re: [PATCH v3 0/2] gpio: davinci: reuse for keystone arch

On Thursday 09 January 2014 05:31 AM, Sekhar Nori wrote:
> On Wednesday 08 January 2014 07:38 PM, Santosh Shilimkar wrote:
>> On Tuesday 07 January 2014 11:06 PM, Sekhar Nori wrote:
>>> On Tuesday 07 January 2014 11:22 PM, Santosh Shilimkar wrote:
>>>> Sekhar,
>>>>
>>>> On Tuesday 24 December 2013 06:41 AM, Grygorii Strashko wrote:
>>>>> This series is intended to update Davinci GPIO driver and reuse
>>>>> it for Keystone SoCs, because Keystone uses the similar GPIO IP like Davinci.
>>>>> Keystone GPIO IP: supports:
>>>>> - up to 32 GPIO lines;
>>>>> - only unbanked irqs;
>>>>>
>>>>> See Documentation:
>>>>> Keystone - http://www.ti.com/lit/ug/sprugv1/sprugv1.pdf
>>>>>
>>>>> This series based on:
>>>>> https://git.kernel.org/cgit/linux/kernel/git/nsekhar/linux-davinci.git/log/?h=v3.14/gpio
>>>>>
>>>>> Changes in v3:
>>>>> - fixed code, changed by mistake; fixed sparse warning
>>>>> Changes in v2:
>>>>> - minor comments applied, no functional changes
>>>>>
>>>>> v2: https://lkml.org/lkml/2013/12/18/135
>>>>> v1: https://lkml.org/lkml/2013/12/12/366
>>>>>
>>>>> Cc: Linus Walleij <[email protected]>
>>>>> Cc: Alexandre Courbot <[email protected]>
>>>>> Cc: Sekhar Nori <[email protected]>
>>>>> Cc: Santosh Shilimkar <[email protected]>
>>>>>
>>>>> Grygorii Strashko (2):
>>>>> gpio: davinci: don't create irq_domain in case of unbanked irqs
>>>>> gpio: davinci: reuse for Keystone SoC
>>>>>
>>>>> .../devicetree/bindings/gpio/gpio-davinci.txt | 4 +-
>>>>> drivers/gpio/gpio-davinci.c | 82 ++++++++++++++------
>>>>> 2 files changed, 61 insertions(+), 25 deletions(-)
>>>>>
>>>> Have you picked up the $subject series in your queue ?
>>>
>>> Not yet, at least on the new compatible introduction, I need an ack from
>>> DT folks.
>>>
>> I noticed that but the usual 2 weeks period to get ack is over I guess ;-)
>> The DT part is really trivial as well but I let you decide.
>
> I just realize that Rob's e-mail address bounces. I have added his
> updated address now and hopefully he will see this to provide his ack.
>
> Rob,
>
> We need your ack on 2/2 of this series. If you do not have the patch, I
> can forward it to you.
>
>>
>>> I am happy with the patches though and have tested them as well, In case
>>> I do not get an ack from 2/2 in time, I can at least send 1/2 for
>>> inclusion after my first gpio pull request to ARM-SoC gets pulled.
>>>
>> Would be great to get both of them but if not both at least 1/2.
>
> I had already sent it with my first pull request. Hmph. Everything
> except 2/2 of tis series should be in linux-next now.
>
Thanks !!