Re: [PATCHv3] OMAP: Enable Magic SysRq on serial console ttyOx

2011-01-31 Thread Thomas Weber
On Mon, Jan 24, 2011 at 10:47 PM, Kevin Hilman khil...@ti.com wrote:
 Govindraj govindraj...@gmail.com writes:

 On Fri, Jan 21, 2011 at 3:31 PM, Thomas Weber we...@corscience.de wrote:
 Magic SysRq key is not working for OMAP on new serial
 console ttyOx because SUPPORT_SYSRQ is not defined
 for omap-serial.

 This patch defines SUPPORT_SYSRQ in omap-serial and
 enables handling of Magic SysRq character.

 Further there is an issue of losing first break character.
 Removing the reset of the lsr_break_flag fixes this issue.

 Signed-off-by: Thomas Weber we...@corscience.de


 Acked-by: Govindraj.R govindraj.r...@ti.com
 Tested-by: Manjunath G Kondaiah manj...@ti.com


 Acked-by: Kevin Hilman khil...@ti.com

 Greg, this one should go for .38-rc.   Feel free to merge, or we can
 take it through the OMAP tree with your ack.

 Thanks,

 Kevin

 --

Hello,
should I rebase this patch, because it does not apply after movement
of driver/serial to driver/tty/serial?

Thomas
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCHv3] OMAP: Enable Magic SysRq on serial console ttyOx

2011-01-31 Thread Kevin Hilman
Thomas Weber thomas.weber.li...@googlemail.com writes:

 On Mon, Jan 24, 2011 at 10:47 PM, Kevin Hilman khil...@ti.com wrote:
 Govindraj govindraj...@gmail.com writes:

 On Fri, Jan 21, 2011 at 3:31 PM, Thomas Weber we...@corscience.de wrote:
 Magic SysRq key is not working for OMAP on new serial
 console ttyOx because SUPPORT_SYSRQ is not defined
 for omap-serial.

 This patch defines SUPPORT_SYSRQ in omap-serial and
 enables handling of Magic SysRq character.

 Further there is an issue of losing first break character.
 Removing the reset of the lsr_break_flag fixes this issue.

 Signed-off-by: Thomas Weber we...@corscience.de


 Acked-by: Govindraj.R govindraj.r...@ti.com
 Tested-by: Manjunath G Kondaiah manj...@ti.com


 Acked-by: Kevin Hilman khil...@ti.com

 Greg, this one should go for .38-rc.   Feel free to merge, or we can
 take it through the OMAP tree with your ack.

 Thanks,

 Kevin

 --

 Hello,
 should I rebase this patch, because it does not apply after movement
 of driver/serial to driver/tty/serial?


Yes, please rebase.

Thanks,

Kevin

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


Re: [PATCHv3] OMAP: Enable Magic SysRq on serial console ttyOx

2011-01-24 Thread Kevin Hilman
Govindraj govindraj...@gmail.com writes:

 On Fri, Jan 21, 2011 at 3:31 PM, Thomas Weber we...@corscience.de wrote:
 Magic SysRq key is not working for OMAP on new serial
 console ttyOx because SUPPORT_SYSRQ is not defined
 for omap-serial.

 This patch defines SUPPORT_SYSRQ in omap-serial and
 enables handling of Magic SysRq character.

 Further there is an issue of losing first break character.
 Removing the reset of the lsr_break_flag fixes this issue.

 Signed-off-by: Thomas Weber we...@corscience.de


 Acked-by: Govindraj.R govindraj.r...@ti.com
 Tested-by: Manjunath G Kondaiah manj...@ti.com


Acked-by: Kevin Hilman khil...@ti.com

Greg, this one should go for .38-rc.   Feel free to merge, or we can
take it through the OMAP tree with your ack.

Thanks,

Kevin

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


[PATCHv3] OMAP: Enable Magic SysRq on serial console ttyOx

2011-01-21 Thread Thomas Weber
Magic SysRq key is not working for OMAP on new serial
console ttyOx because SUPPORT_SYSRQ is not defined
for omap-serial.

This patch defines SUPPORT_SYSRQ in omap-serial and
enables handling of Magic SysRq character.

Further there is an issue of losing first break character.
Removing the reset of the lsr_break_flag fixes this issue.

Signed-off-by: Thomas Weber we...@corscience.de
---
 drivers/serial/omap-serial.c |5 -
 1 files changed, 4 insertions(+), 1 deletions(-)

diff --git a/drivers/serial/omap-serial.c b/drivers/serial/omap-serial.c
index 7f2f010..699b344 100644
--- a/drivers/serial/omap-serial.c
+++ b/drivers/serial/omap-serial.c
@@ -20,6 +20,10 @@
  * this driver as required for the omap-platform.
  */
 
+#if defined(CONFIG_SERIAL_OMAP_CONSOLE)  defined(CONFIG_MAGIC_SYSRQ)
+#define SUPPORT_SYSRQ
+#endif
+
 #include linux/module.h
 #include linux/init.h
 #include linux/console.h
@@ -190,7 +194,6 @@ static inline void receive_chars(struct uart_omap_port *up, 
int *status)
if (up-port.line == up-port.cons-index) {
/* Recover the break flag from console xmit */
lsr |= up-lsr_break_flag;
-   up-lsr_break_flag = 0;
}
 #endif
if (lsr  UART_LSR_BI)
-- 
1.7.4.rc2

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


Re: [PATCHv3] OMAP: Enable Magic SysRq on serial console ttyOx

2011-01-21 Thread Govindraj
On Fri, Jan 21, 2011 at 3:31 PM, Thomas Weber we...@corscience.de wrote:
 Magic SysRq key is not working for OMAP on new serial
 console ttyOx because SUPPORT_SYSRQ is not defined
 for omap-serial.

 This patch defines SUPPORT_SYSRQ in omap-serial and
 enables handling of Magic SysRq character.

 Further there is an issue of losing first break character.
 Removing the reset of the lsr_break_flag fixes this issue.

 Signed-off-by: Thomas Weber we...@corscience.de


Acked-by: Govindraj.R govindraj.r...@ti.com
Tested-by: Manjunath G Kondaiah manj...@ti.com


 ---
  drivers/serial/omap-serial.c |    5 -
  1 files changed, 4 insertions(+), 1 deletions(-)

 diff --git a/drivers/serial/omap-serial.c b/drivers/serial/omap-serial.c
 index 7f2f010..699b344 100644
 --- a/drivers/serial/omap-serial.c
 +++ b/drivers/serial/omap-serial.c
 @@ -20,6 +20,10 @@
  * this driver as required for the omap-platform.
  */

 +#if defined(CONFIG_SERIAL_OMAP_CONSOLE)  defined(CONFIG_MAGIC_SYSRQ)
 +#define SUPPORT_SYSRQ
 +#endif
 +
  #include linux/module.h
  #include linux/init.h
  #include linux/console.h
 @@ -190,7 +194,6 @@ static inline void receive_chars(struct uart_omap_port 
 *up, int *status)
                        if (up-port.line == up-port.cons-index) {
                                /* Recover the break flag from console xmit */
                                lsr |= up-lsr_break_flag;
 -                               up-lsr_break_flag = 0;
                        }
  #endif
                        if (lsr  UART_LSR_BI)
 --
 1.7.4.rc2

 --
 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/

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