linux-input.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Input: omap4-keypad: Remove the OMAP4_KBD_SYSCONFIG and OMAP4_KBD_SYSSTATUS
@ 2012-11-26 10:51 Shubhrajyoti D
  2012-11-26 18:01 ` Dmitry Torokhov
  0 siblings, 1 reply; 4+ messages in thread
From: Shubhrajyoti D @ 2012-11-26 10:51 UTC (permalink / raw)
  To: linux-input; +Cc: linux-omap, linux-arm-kernel, b-cousson, Shubhrajyoti D

The macros OMAP4_KBD_SYSSTATUS and OMAP4_KBD_SYSSTATUS are not used
so remove them.

Signed-off-by: Shubhrajyoti D <shubhrajyoti@ti.com>
---
 drivers/input/keyboard/omap4-keypad.c |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/drivers/input/keyboard/omap4-keypad.c b/drivers/input/keyboard/omap4-keypad.c
index c05f98c..d2b72b7 100644
--- a/drivers/input/keyboard/omap4-keypad.c
+++ b/drivers/input/keyboard/omap4-keypad.c
@@ -36,8 +36,6 @@
 
 /* OMAP4 registers */
 #define OMAP4_KBD_REVISION		0x00
-#define OMAP4_KBD_SYSCONFIG		0x10
-#define OMAP4_KBD_SYSSTATUS		0x14
 #define OMAP4_KBD_IRQSTATUS		0x18
 #define OMAP4_KBD_IRQENABLE		0x1C
 #define OMAP4_KBD_WAKEUPENABLE		0x20
-- 
1.7.5.4


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] Input: omap4-keypad: Remove the OMAP4_KBD_SYSCONFIG and OMAP4_KBD_SYSSTATUS
  2012-11-26 10:51 [PATCH] Input: omap4-keypad: Remove the OMAP4_KBD_SYSCONFIG and OMAP4_KBD_SYSSTATUS Shubhrajyoti D
@ 2012-11-26 18:01 ` Dmitry Torokhov
  2012-11-27  5:16   ` Shubhrajyoti Datta
  2012-11-27  5:17   ` Shubhrajyoti Datta
  0 siblings, 2 replies; 4+ messages in thread
From: Dmitry Torokhov @ 2012-11-26 18:01 UTC (permalink / raw)
  To: Shubhrajyoti D; +Cc: linux-input, linux-omap, linux-arm-kernel, b-cousson

On Mon, Nov 26, 2012 at 04:21:19PM +0530, Shubhrajyoti D wrote:
> The macros OMAP4_KBD_SYSSTATUS and OMAP4_KBD_SYSSTATUS are not used
> so remove them.

They document the existing registers and do not add bloat to the
generated objects so let's leave them be.

Thanks.

-- 
Dmitry

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Input: omap4-keypad: Remove the OMAP4_KBD_SYSCONFIG and OMAP4_KBD_SYSSTATUS
  2012-11-26 18:01 ` Dmitry Torokhov
@ 2012-11-27  5:16   ` Shubhrajyoti Datta
  2012-11-27  5:17   ` Shubhrajyoti Datta
  1 sibling, 0 replies; 4+ messages in thread
From: Shubhrajyoti Datta @ 2012-11-27  5:16 UTC (permalink / raw)
  To: Dmitry Torokhov
  Cc: b-cousson, linux-omap, Shubhrajyoti D, linux-arm-kernel,
	linux-input


[-- Attachment #1.1: Type: text/plain, Size: 652 bytes --]

On Mon, Nov 26, 2012 at 11:31 PM, Dmitry Torokhov <dmitry.torokhov@gmail.com
> wrote:

> On Mon, Nov 26, 2012 at 04:21:19PM +0530, Shubhrajyoti D wrote:
> > The macros OMAP4_KBD_SYSSTATUS and OMAP4_KBD_SYSSTATUS are not used
> > so remove them.
>
> They document the existing registers and do not add bloat to the
> generated objects so let's leave them be.
>

The objective of the patch is not to prevent bloat.

Actually these registers have power/system impact so it was decided that
the drivers will not
configure the sysc registers (infact not define). Currently hwmod
configures the registers.

To be safe I thought of deleting it.


> Thanks.
>

[-- Attachment #1.2: Type: text/html, Size: 1272 bytes --]

[-- Attachment #2: Type: text/plain, Size: 176 bytes --]

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Input: omap4-keypad: Remove the OMAP4_KBD_SYSCONFIG and OMAP4_KBD_SYSSTATUS
  2012-11-26 18:01 ` Dmitry Torokhov
  2012-11-27  5:16   ` Shubhrajyoti Datta
@ 2012-11-27  5:17   ` Shubhrajyoti Datta
  1 sibling, 0 replies; 4+ messages in thread
From: Shubhrajyoti Datta @ 2012-11-27  5:17 UTC (permalink / raw)
  To: Dmitry Torokhov
  Cc: Shubhrajyoti D, linux-input, linux-omap, linux-arm-kernel,
	b-cousson

On Mon, Nov 26, 2012 at 11:31 PM, Dmitry Torokhov
<dmitry.torokhov@gmail.com> wrote:
> On Mon, Nov 26, 2012 at 04:21:19PM +0530, Shubhrajyoti D wrote:
>> The macros OMAP4_KBD_SYSSTATUS and OMAP4_KBD_SYSSTATUS are not used
>> so remove them.
>
> They document the existing registers and do not add bloat to the
> generated objects so let's leave them be.

The objective of the patch is not to prevent bloat.

Actually these registers have power/system impact so it was decided
that the drivers will not
configure the sysc registers (infact not define). Currently hwmod
configures the registers.

To be safe I thought of deleting it.


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

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2012-11-27  5:17 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-11-26 10:51 [PATCH] Input: omap4-keypad: Remove the OMAP4_KBD_SYSCONFIG and OMAP4_KBD_SYSSTATUS Shubhrajyoti D
2012-11-26 18:01 ` Dmitry Torokhov
2012-11-27  5:16   ` Shubhrajyoti Datta
2012-11-27  5:17   ` Shubhrajyoti Datta

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).