Re: [PATCH V4 0/4] input: keyboard: tegra: cleanups and DT supports

2013-01-10 Thread Laxman Dewangan

On Thursday 10 January 2013 11:14 PM, Dmitry Torokhov wrote:

Hi Laxman,

On Thu, Jan 10, 2013 at 12:01:23PM +0530, Laxman Dewangan wrote:

Hi Dmitry,

On Monday 07 January 2013 10:22 PM, Stephen Warren wrote:

On 01/06/2013 04:14 AM, Laxman Dewangan wrote:

This patch series:
  - fix build warning,
  - use devm_* for allocation,
  - make column/rows configuration through DT and
  - remove the rarely used  key mapping table.

The series,
Reviewed-by: Stephen Warren 


If you are fine with this series then can it be apply please? I can
handle if there is any comment on this series to close this.


The patches are applied with minor edits to the 3rd and 4th (there was
no point in having a separate keymap setup function anymore).



Thanks for taking care. I looked the final change from your git and the 
changes are  fine.

Thanks,
Laxman
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH V4 0/4] input: keyboard: tegra: cleanups and DT supports

2013-01-10 Thread Dmitry Torokhov
Hi Laxman,

On Thu, Jan 10, 2013 at 12:01:23PM +0530, Laxman Dewangan wrote:
> Hi Dmitry,
> 
> On Monday 07 January 2013 10:22 PM, Stephen Warren wrote:
> >On 01/06/2013 04:14 AM, Laxman Dewangan wrote:
> >>This patch series:
> >>  - fix build warning,
> >>  - use devm_* for allocation,
> >>  - make column/rows configuration through DT and
> >>  - remove the rarely used  key mapping table.
> >The series,
> >Reviewed-by: Stephen Warren 
> >
> 
> If you are fine with this series then can it be apply please? I can
> handle if there is any comment on this series to close this.
> 

The patches are applied with minor edits to the 3rd and 4th (there was
no point in having a separate keymap setup function anymore).

Thanks.

-- 
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH V4 0/4] input: keyboard: tegra: cleanups and DT supports

2013-01-10 Thread Dmitry Torokhov
Hi Laxman,

On Thu, Jan 10, 2013 at 12:01:23PM +0530, Laxman Dewangan wrote:
 Hi Dmitry,
 
 On Monday 07 January 2013 10:22 PM, Stephen Warren wrote:
 On 01/06/2013 04:14 AM, Laxman Dewangan wrote:
 This patch series:
   - fix build warning,
   - use devm_* for allocation,
   - make column/rows configuration through DT and
   - remove the rarely used  key mapping table.
 The series,
 Reviewed-by: Stephen Warren swar...@nvidia.com
 
 
 If you are fine with this series then can it be apply please? I can
 handle if there is any comment on this series to close this.
 

The patches are applied with minor edits to the 3rd and 4th (there was
no point in having a separate keymap setup function anymore).

Thanks.

-- 
Dmitry
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH V4 0/4] input: keyboard: tegra: cleanups and DT supports

2013-01-10 Thread Laxman Dewangan

On Thursday 10 January 2013 11:14 PM, Dmitry Torokhov wrote:

Hi Laxman,

On Thu, Jan 10, 2013 at 12:01:23PM +0530, Laxman Dewangan wrote:

Hi Dmitry,

On Monday 07 January 2013 10:22 PM, Stephen Warren wrote:

On 01/06/2013 04:14 AM, Laxman Dewangan wrote:

This patch series:
  - fix build warning,
  - use devm_* for allocation,
  - make column/rows configuration through DT and
  - remove the rarely used  key mapping table.

The series,
Reviewed-by: Stephen Warren swar...@nvidia.com


If you are fine with this series then can it be apply please? I can
handle if there is any comment on this series to close this.


The patches are applied with minor edits to the 3rd and 4th (there was
no point in having a separate keymap setup function anymore).



Thanks for taking care. I looked the final change from your git and the 
changes are  fine.

Thanks,
Laxman
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH V4 0/4] input: keyboard: tegra: cleanups and DT supports

2013-01-09 Thread Laxman Dewangan

Hi Dmitry,

On Monday 07 January 2013 10:22 PM, Stephen Warren wrote:

On 01/06/2013 04:14 AM, Laxman Dewangan wrote:

This patch series:
  - fix build warning,
  - use devm_* for allocation,
  - make column/rows configuration through DT and
  - remove the rarely used  key mapping table.

The series,
Reviewed-by: Stephen Warren 



If you are fine with this series then can it be apply please? I can 
handle if there is any comment on this series to close this.


Thanks,
Laxman

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH V4 0/4] input: keyboard: tegra: cleanups and DT supports

2013-01-09 Thread Laxman Dewangan

Hi Dmitry,

On Monday 07 January 2013 10:22 PM, Stephen Warren wrote:

On 01/06/2013 04:14 AM, Laxman Dewangan wrote:

This patch series:
  - fix build warning,
  - use devm_* for allocation,
  - make column/rows configuration through DT and
  - remove the rarely used  key mapping table.

The series,
Reviewed-by: Stephen Warren swar...@nvidia.com



If you are fine with this series then can it be apply please? I can 
handle if there is any comment on this series to close this.


Thanks,
Laxman

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH V4 0/4] input: keyboard: tegra: cleanups and DT supports

2013-01-07 Thread Stephen Warren
On 01/06/2013 04:14 AM, Laxman Dewangan wrote:
> This patch series:
>  - fix build warning,
>  - use devm_* for allocation,
>  - make column/rows configuration through DT and 
>  - remove the rarely used  key mapping table.

The series,
Reviewed-by: Stephen Warren 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH V4 0/4] input: keyboard: tegra: cleanups and DT supports

2013-01-07 Thread Stephen Warren
On 01/06/2013 04:14 AM, Laxman Dewangan wrote:
 This patch series:
  - fix build warning,
  - use devm_* for allocation,
  - make column/rows configuration through DT and 
  - remove the rarely used  key mapping table.

The series,
Reviewed-by: Stephen Warren swar...@nvidia.com

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH V4 0/4] input: keyboard: tegra: cleanups and DT supports

2013-01-06 Thread Laxman Dewangan
This patch series:
 - fix build warning,
 - use devm_* for allocation,
 - make column/rows configuration through DT and 
 - remove the rarely used  key mapping table.

Changes from V1:
- renames the rows and pins property array.
- nit cleanups.

Changes from V2:
- remove the error checks changes from original.
- return -EBUSY when reg mapping fail.
- remove unregister_input_device() devm_* version is used.
- Pass correct argument in free_irq.

Changes from V3:
- use devm_request_irq() and remove the kbc_remove() callback.

Laxman Dewangan (4):
  input: keyboard: tegra: fix build warning
  input: keyboard: tegra: use devm_* for resource allocation
  input: keyboard: tegra: add support for rows/cols configuration from
dt
  input: keyboard: tegra: remove default key mapping

 .../bindings/input/nvidia,tegra20-kbc.txt  |   22 ++
 drivers/input/keyboard/tegra-kbc.c |  345 ++--
 2 files changed, 118 insertions(+), 249 deletions(-)

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH V4 0/4] input: keyboard: tegra: cleanups and DT supports

2013-01-06 Thread Laxman Dewangan
This patch series:
 - fix build warning,
 - use devm_* for allocation,
 - make column/rows configuration through DT and 
 - remove the rarely used  key mapping table.

Changes from V1:
- renames the rows and pins property array.
- nit cleanups.

Changes from V2:
- remove the error checks changes from original.
- return -EBUSY when reg mapping fail.
- remove unregister_input_device() devm_* version is used.
- Pass correct argument in free_irq.

Changes from V3:
- use devm_request_irq() and remove the kbc_remove() callback.

Laxman Dewangan (4):
  input: keyboard: tegra: fix build warning
  input: keyboard: tegra: use devm_* for resource allocation
  input: keyboard: tegra: add support for rows/cols configuration from
dt
  input: keyboard: tegra: remove default key mapping

 .../bindings/input/nvidia,tegra20-kbc.txt  |   22 ++
 drivers/input/keyboard/tegra-kbc.c |  345 ++--
 2 files changed, 118 insertions(+), 249 deletions(-)

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/