: fix a couple of checkpatch issues
Signed-off-by: Rob Clark robdcl...@gmail.com
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
Cc: Russell King - ARM Linux li...@arm.linux.org.uk
---
Notes:
I'd like to wake this patch up again. It was rejected back in 2012 on
the grounds
On 14/05/14 16:08, Jason Wessel wrote:
On 05/14/2014 09:55 AM, Daniel Thompson wrote:
The behaviour of the UART poll_put_char infrastructure is inconsistent
with respect to linefeed conversions. This in turn leads to difficulty
using kdb on serial ports that are not also consoles
(e.g
does.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
drivers/tty/serial/kgdb_nmi.c | 16 +---
1 file changed, 13 insertions(+), 3 deletions(-)
diff --git a/drivers/tty/serial/kgdb_nmi.c b/drivers/tty/serial/kgdb_nmi.c
index 20d21d0..cfadf29 100644
--- a/drivers/tty
-by: Daniel Thompson daniel.thomp...@linaro.org
---
drivers/tty/serial/kgdb_nmi.c | 13 +
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git a/drivers/tty/serial/kgdb_nmi.c b/drivers/tty/serial/kgdb_nmi.c
index 5f673b7..d51b2a1 100644
--- a/drivers/tty/serial/kgdb_nmi.c
+++ b/drivers
on
any other part of the kernel.
The series has been runtime tested on ARM and compile tested on
x86 and powerpc.
Changes since v1:
- Trimmed down the first patch to the remaining fragments as described
above.
- Rebased on tty-next as requested by Greg KH.
Daniel Thompson (4):
serial: cpm_uart
kgdb_nmi uses tasklets on the assumption they will not be scheduled
until the next timer tick. This assumption is invalid and can lead to
live lock, continually servicing the kgdb_nmi tasklet. This is fixed
by using the timer API instead.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
than simply rip out the conversion code.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
Cc: Doug Anderson diand...@chromium.org
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Cc: Jiri Slaby jsl...@suse.cz
Cc: Christophe Leroy christophe.le...@c-s.fr
Cc: linux-ser...@vger.kernel.org
gre...@linuxfoundation.org
Cc: Jiri Slaby jsl...@suse.cz
Cc: Barry Song bao...@kernel.org
Cc: Qipan Li qipan...@csr.com
Cc: linux-ser...@vger.kernel.org
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
drivers/tty/serial/sirfsoc_uart.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
gre...@linuxfoundation.org
Cc: Jiri Slaby jsl...@suse.cz
Cc: Barry Song bao...@kernel.org
Cc: Qipan Li qipan...@csr.com
Cc: linux-ser...@vger.kernel.org
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
Notes:
Sorry to resend within a few minutes. Patch is identical to parent
On 01/06/14 14:38, Rickard Strandqvist wrote:
There is a risk that the variable will be used without being initialized.
This was largely found by using a static code analysis program called
cppcheck.
Signed-off-by: Rickard Strandqvist rickard_strandqv...@spectrumdigital.se
---
the existing
users of the established timing bits forcing a rework of the
ESTABLISHED_TIMINGS_BITS macro.
Tested new EDID on an aged (and misbehaving) industrial LCD panel;
existing EDIDs still pass edid-decode's checksum checks.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
Cc: Randy Dunlap
This defconfig already enables DEBUG_LL and by default DEBUG_LL_UART_NONE
will be selected (but due to some back compability magic I'd like to
remove is not actually honoured). DEBUG_LL_UART_PL01X is a much saner
default.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
arch/arm
it.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
arch/arm/Kconfig.debug | 13 -
1 file changed, 12 insertions(+), 1 deletion(-)
diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index eab8ecb..4c8d7db 100644
--- a/arch/arm/Kconfig.debug
+++ b/arch/arm
This defconfig already enables DEBUG_LL and by default DEBUG_LL_UART_NONE
will be selected (but due to some back compability magic I'd like to
remove is not actually honoured). DEBUG_LL_UART_PL01X is a much saner
default.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
arch/arm
the DEBUG_LL backend will be overriden. Problem is fixed by removing the
automatic enabling of this option.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
arch/arm/Kconfig.debug | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/arch/arm/Kconfig.debug b/arch/arm
On EP93XX uncompress.h uses CONFIG_DEBUG_UART_PHYS instead of a hard
coded serial port. This means the build breaks when DEBUG_LL
(and DEBUG_LL_UART_PL01X) is not enabled.
This is fixed by adding a new dependancy.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
arch/arm
:
* Remove pointless single platform support (thanks Arnd Bergmann)
Daniel Thompson (5):
ARM: versatile: Enable DEBUG_LL_UART_PL01X
ARM: ep93xx: Enable DEBUG_LL_UART_PL01X
ARM: Conceal DEBUG_LL_UART_NONE from unsupported platforms
arm: Seperate DEBUG_UART_PHYS from DEBUG_LL on EP93XX
arm: Fix
On 26/05/14 14:39, Arnd Bergmann wrote:
diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index eab8ecb..4c8d7db 100644
--- a/arch/arm/Kconfig.debug
+++ b/arch/arm/Kconfig.debug
@@ -842,7 +842,18 @@ choice
config DEBUG_LL_UART_NONE
bool No low-level
On 27/05/14 14:37, Arnd Bergmann wrote:
I'm very happy to update my patch to follow this one. However since it
will render the corresponding debug-macro.S unreachable by KBuild I like
to nuke the file as well. Good idea?
No. See the description under config DEBUG_LL_UART_NONE:
to a series.
- Tested defconfig builds of all impacted platforms.
Changes since v2:
- Switch from def_bool to bool (thanks Russell King)
Changes since v1:
- Remove pointless single platform support (thanks Arnd Bergmann)
Daniel Thompson (5):
ARM: versatile: Enable DEBUG_LL_UART_PL01X
ARM: ep93xx
means to use DEBUG_LL.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
arch/arm/Kconfig.debug | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index eab8ecb..33f0b38 100644
--- a/arch/arm/Kconfig.debug
+++ b/arch/arm
This defconfig already enables DEBUG_LL and by default DEBUG_LL_UART_NONE
will be selected (but due to some back compability magic I'd like to
remove is not actually honoured). DEBUG_LL_UART_PL01X is a much saner
default.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
arch/arm
the DEBUG_LL backend will be overriden. Problem is fixed by removing the
automatic enabling of this option.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
arch/arm/Kconfig.debug | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/arch/arm/Kconfig.debug b/arch/arm
This defconfig already enables DEBUG_LL and by default DEBUG_LL_UART_NONE
will be selected (but due to some back compability magic I'd like to
remove is not actually honoured). DEBUG_LL_UART_PL01X is a much saner
default.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
arch/arm
On EP93XX uncompress.h uses CONFIG_DEBUG_UART_PHYS instead of a hard
coded serial port. This means the build breaks when DEBUG_LL
(and DEBUG_LL_UART_PL01X) is not enabled.
This is fixed by adding a new dependancy.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
arch/arm
and the busy-wait
until the FIFO is empty in asc_console_write() will complete at the same
time.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
drivers/tty/serial/st-asc.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/tty/serial/st-asc.c b/drivers
does.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
drivers/tty/serial/kgdb_nmi.c | 16 +---
1 file changed, 13 insertions(+), 3 deletions(-)
diff --git a/drivers/tty/serial/kgdb_nmi.c b/drivers/tty/serial/kgdb_nmi.c
index 20d21d0..cfadf29 100644
--- a/drivers/tty
() but it is optional for drivers to use this
function. Fortunately we can be confident the translation is safe
because the (very common) 8250 already does this translation.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
Cc: Kumar Gala ga...@kernel.crashing.org
Cc: Pantelis Antoniou pa...@intracom.gr
kgdb_nmi uses tasklets on the assumption they will not be scheduled
until the next timer tick. This assumption is invalid and can lead to
live lock, continually servicing the kgdb_nmi tasklet. This is fixed
by using the timer API instead.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
-by: Daniel Thompson daniel.thomp...@linaro.org
---
drivers/tty/serial/kgdb_nmi.c | 13 +
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git a/drivers/tty/serial/kgdb_nmi.c b/drivers/tty/serial/kgdb_nmi.c
index 5f673b7..d51b2a1 100644
--- a/drivers/tty/serial/kgdb_nmi.c
+++ b/drivers
and compile tested on
x86 and powerpc.
Daniel Thompson (4):
serial: core: Consistent LF handling for poll_put_char
serial: kgdb_nmi: Use container_of() to locate private data
serial: kgdb_nmi: Switch from tasklets to real timers
serial: kgdb_nmi: Improve console integration with KDB I/O
drivers/tty
On 12/06/14 16:58, Russell King - ARM Linux wrote:
On Thu, Jun 12, 2014 at 04:42:35PM +0100, Daniel Thompson wrote:
A new atomic modeset/pageflip ioctl being developed in DRM requires
get_user() to work for 64bit types (in addition to just put_user()).
v1: original
v2: pass correct size
On 17/06/14 12:09, Russell King - ARM Linux wrote:
On Tue, Jun 17, 2014 at 11:17:23AM +0100, Daniel Thompson wrote:
... at this point there is a narrowing cast followed by an implicit
widening. This results in compiler either ignoring r3 altogether or, if
spilling to the stack, generating code
On 17/06/14 14:36, Russell King - ARM Linux wrote:
On Tue, Jun 17, 2014 at 02:28:44PM +0100, Daniel Thompson wrote:
On 17/06/14 12:09, Russell King - ARM Linux wrote:
On Tue, Jun 17, 2014 at 11:17:23AM +0100, Daniel Thompson wrote:
... at this point there is a narrowing cast followed
On 30/05/14 14:43, mathieu.poir...@linaro.org wrote:
diff --git a/drivers/coresight/coresight-etm-cp14.c
b/drivers/coresight/coresight-etm-cp14.c
new file mode 100644
index 000..0088bbb
--- /dev/null
+++ b/drivers/coresight/coresight-etm-cp14.c
@@ -0,0 +1,511 @@
+/* Copyright (c)
On 03/06/14 17:37, Mathieu Poirier wrote:
+static ssize_t debugfs_status_read(struct file *file, char __user
*user_buf,
+size_t count, loff_t *ppos)
+{
+ ssize_t ret;
+ uint32_t val;
+ unsigned long flags;
+ char *buf = kmalloc(PAGE_SIZE,
Using the generic function saves looking up this custom one in a source
navigator.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
Cc: Thomas Gleixner t...@linutronix.de
Cc: Jason Cooper ja...@lakedaemon.net
---
drivers/irqchip/irq-nvic.c | 10 +-
1 file changed, 1 insertion
Using the generic function saves looking up this custom one in a source
navigator.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
Cc: Thomas Gleixner t...@linutronix.de
Cc: Jason Cooper ja...@lakedaemon.net
---
drivers/irqchip/irq-nvic.c | 13 -
1 file changed, 4
the requires masking and has been tested to confirm
that it correctly handles magic sysrq sequences on ST's B2020 board.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
drivers/tty/serial/st-asc.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/tty
needless complexity when
doing a code review to check for unexpected side effects of any
changes to the serial core.
This is (cross-)compile tested only because I do not have any PA-RISC
hardware.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
drivers/tty/serial/mux.c | 2 +-
1 file
the existing
users of the established timing bits forcing a rework of the
ESTABLISHED_TIMINGS_BITS macro.
Tested new EDID on an aged (and misbehaving) industrial LCD panel;
existing EDIDs still pass edid-decode's checksum checks.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
Documentation
On 25/04/14 17:45, Steven Rostedt wrote:
On Fri, 25 Apr 2014 17:29:22 +0100
Daniel Thompson daniel.thomp...@linaro.org wrote:
If kdb is triggered using SysRq-g then any use of the sr command results
in the SysRq key table lock being recursively acquired, killing the debug
session
On 25/04/14 17:57, Steven Rostedt wrote:
+KDB_ENABLE_ALL_NO_ARGS = KDB_ENABLE_ALL 16,
+KDB_ENABLE_MEM_READ_NO_ARGS = KDB_ENABLE_MEM_READ 16,
+KDB_ENABLE_MEM_WRITE_NO_ARGS = KDB_ENABLE_MEM_WRITE 16,
+KDB_ENABLE_REG_READ_NO_ARGS = KDB_ENABLE_REG_READ 16,
+
On 28/04/14 18:44, Colin Cross wrote:
Is that case documented somewhere in the code comments?
Perhaps not near enough to the _nolock but the primary bit of comment is
here (and in same file as kdb_sr).
--- cut here ---
* kdb_main_loop - After initial setup and assignment of the
*
On 28/04/14 18:44, Colin Cross wrote:
Is that case documented somewhere in the code comments?
Perhaps not near enough to the _nolock but the primary bit of comment is
here (and in same file as kdb_sr).
--- cut here ---
* kdb_main_loop - After initial setup and assignment of the
*
of a brave expert an even bigger subset can be unsafely
examined).
Good arguments to try with this command include: cpuinfo_op,
gpiolib_seq_ops and vmalloc_op.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
kernel/debug/kdb/kdb_main.c | 28
1 file changed
that kdb has interrupt
context. Note also that some of the seq_operations structures hook into
driver supplied code that will only be called if that driver is enabled
so the test above are useful but cannot be exhaustive.
Daniel Thompson (3):
kdb: Add framework to display sequence files
proc
the file to be displayed from kdb.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
fs/proc/interrupts.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/fs/proc/interrupts.c b/fs/proc/interrupts.c
index a352d57..1f8eeaf 100644
--- a/fs/proc/interrupts.c
+++ b/fs/proc
files via kdb.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
include/linux/kdb.h | 3 +++
kernel/debug/kdb/kdb_io.c | 51 +++
2 files changed, 54 insertions(+)
diff --git a/include/linux/kdb.h b/include/linux/kdb.h
index 290db12
files via kdb.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
include/linux/kdb.h | 5 +
kernel/debug/kdb/kdb_io.c | 51 +++
2 files changed, 56 insertions(+)
diff --git a/include/linux/kdb.h b/include/linux/kdb.h
index 290db12
of a brave expert an even bigger subset can be unsafely
examined).
Good arguments to try with this command include: cpuinfo_op,
gpiolib_seq_ops and vmalloc_op.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
kernel/debug/kdb/kdb_io.c | 6 --
kernel/debug/kdb/kdb_main.c | 28
the file to be displayed from kdb.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
fs/proc/interrupts.c | 8
1 file changed, 8 insertions(+)
diff --git a/fs/proc/interrupts.c b/fs/proc/interrupts.c
index a352d57..d8b64f0 100644
--- a/fs/proc/interrupts.c
+++ b/fs/proc
() and used it to fix
build error when CONFIG_KGDB_KDB is not set (oops).
* Better comment explaining why seq_file's lock member can be left
uninitialized (its will not detect errors because we know it is
already not allowed for seq_file ops to use it).
Daniel Thompson (3):
kdb: Add framework
-by: John Stultz john.stu...@linaro.org
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
include/linux/kdb.h| 6 +++---
kernel/debug/kdb/kdb_main.c| 6 +++---
kernel/debug/kdb/kdb_private.h | 2 +-
3 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/include/linux
From: Anton Vorontsov anton.voront...@linaro.org
The actual values of KDB_REPEAT_* enum values and overall logic stayed
the same, but we now treat the values as flags.
This makes it possible to add other flags and combine them, plus makes
the code a lot simpler and shorter. But
: kill
Reboot: reboot
All:cpu, kgdb, (and all of the above), nmi_console
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
include/linux/kdb.h | 68 +-
kernel/debug/kdb/kdb_bp.c | 21 ++
kernel/debug/kdb/kdb_main.c | 100
Currently kiosk mode must be explicitly requested by the bootloader or
userspace. It is convenient to be able to change the default value in a
similar manner to CONFIG_MAGIC_SYSRQ_DEFAULT_MASK.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
kernel/debug/kdb/kdb_main.c | 2 +-
lib
From: Anton Vorontsov anton.voront...@linaro.org
Since we now treat KDB_REPEAT_* as flags, there is no need to
pass KDB_REPEAT_NONE. It's just the default behaviour when no
flags are specified.
Signed-off-by: Anton Vorontsov anton.voront...@linaro.org
Signed-off-by: John Stultz
Stultz john.stu...@linaro.org
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
include/linux/kdb.h | 1 +
kernel/debug/kdb/kdb_main.c | 30 +-
2 files changed, 30 insertions(+), 1 deletion(-)
diff --git a/include/linux/kdb.h b/include/linux/kdb.h
...@linaro.org
Signed-off-by: John Stultz john.stu...@linaro.org
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
include/linux/kdb.h | 10 +++---
kernel/debug/kdb/kdb_bp.c | 14
kernel/debug/kdb/kdb_main.c | 86 ++---
kernel/trace
From: Anton Vorontsov anton.voront...@linaro.org
The struct member is never used in the code, so we can remove it.
We will introduce real flags soon by renaming cmd_repeat to cmd_flags.
Signed-off-by: Anton Vorontsov anton.voront...@linaro.org
Signed-off-by: John Stultz john.stu...@linaro.org
If kdb is triggered using SysRq-g then any use of the sr command results
in the SysRq key table lock being recursively acquired, killing the debug
session. That patch resolves the problem by introducing a _nolock
alternative for __handle_sysrq.
Signed-off-by: Daniel Thompson daniel.thomp
: Rename kdb_register_repeat() to kdb_register_flags()
kdb: Use KDB_REPEAT_* values as flags
kdb: Remove KDB_REPEAT_NONE flag
kdb: Add enable mask for groups of commands
Daniel Thompson (3):
sysrq: Implement __handle_sysrq_nolock to avoid recursive locking in
kdb
kdb: Categorize kdb
The section of the makefile that determines the TEXT_OFFSET is sorted
by address so that, in multi-arch kernel builds, the architecture with the
most stringent requirements for the kernel base address gets to define
TEXT_OFFSET. The comment should reflect that.
Signed-off-by: Daniel Thompson
the compatibility options for SPEAr
when ARCH_MULTIPLATFORM is not set.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
arch/arm/Kconfig.debug | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index 0531da8..f10c784
On 04/04/14 12:39, Arnd Bergmann wrote:
Problem is fixed by only deploying the compatibility options for SPEAr
when ARCH_MULTIPLATFORM is not set.
The assumption is actually wrong: SPEAr does not support single
platform builds any more. The PLAT_SPEAR_SINGLE symbol in Kconfig
is a leftover
DEBUG_UART_PL01X, which is intended to provide some
kind of compatibility with single platform builds, spuriously depends upon
ARCH_SPEAR13XX (and similar SPEAr options).
Problem is fixed by removing problematic dependancies.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
Notes:
v2: Remove
On 04/04/14 15:41, Russell King - ARM Linux wrote:
The happens because the multi-platform mode may include ARCH_SPEAR13XX
and this forcibly engages DEBUG_UART_PL01X to provide some kind of
compatibility with single platform builds (SPEAr supports both single and
multi-platform). This in turn
the DEBUG_LL backend will be overriden. Problem is fixed by removing the
automatic enabling of this option.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
Notes:
v3: Switch from def_bool to bool (thanks Russell King)
v2: Remove pointless single platform support (thanks Arnd Bergmann
On 04/04/14 16:41, Daniel Thompson wrote:
diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index 0531da8..7e19c94 100644
--- a/arch/arm/Kconfig.debug
+++ b/arch/arm/Kconfig.debug
@@ -989,12 +989,7 @@ config DEBUG_LL_INCLUDE
# Compatibility options for PL01x
config
This defconfig already enables DEBUG_LL and by default DEBUG_LL_UART_NONE
will be selected (but due to some back compability magic I'd like to
remove is not actually honoured). DEBUG_LL_UART_PL01X is a much saner
default.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
arch/arm
to a series.
- Tested defconfig builds of all impacted platforms.
Changes since v2:
- Switch from def_bool to bool (thanks Russell King)
Changes since v1:
- Remove pointless single platform support (thanks Arnd Bergmann)
Daniel Thompson (5):
ARM: versatile: Enable DEBUG_LL_UART_PL01X
ARM: ep93xx
This defconfig already enables DEBUG_LL and by default DEBUG_LL_UART_NONE
will be selected (but due to some back compability magic I'd like to
remove is not actually honoured). DEBUG_LL_UART_PL01X is a much saner
default.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
arch/arm
it.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
arch/arm/Kconfig.debug | 13 -
1 file changed, 12 insertions(+), 1 deletion(-)
diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index 0531da8..0702d6d 100644
--- a/arch/arm/Kconfig.debug
+++ b/arch/arm
the DEBUG_LL backend will be overriden. Problem is fixed by removing the
automatic enabling of this option.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
arch/arm/Kconfig.debug | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/arch/arm/Kconfig.debug b/arch/arm
On EP93XX uncompress.h uses CONFIG_DEBUG_UART_PHYS instead of a hard
coded serial port. This means the build breaks when DEBUG_LL
(and DEBUG_LL_UART_PL01X) is not enabled.
This is fixed by adding a new dependancy.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
Notes:
Having
This defconfig already enables DEBUG_LL and by default DEBUG_LL_UART_NONE
will be selected (but due to some back compability magic I'd like to
remove is not actually honoured). DEBUG_LL_UART_PL01X is a much saner
default.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
arch/arm
On EP93XX uncompress.h uses CONFIG_DEBUG_UART_PHYS instead of a hard
coded serial port. This means the build breaks when DEBUG_LL
(and DEBUG_LL_UART_PL01X) is not enabled.
This is fixed by adding a new dependancy.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
Notes:
Having
the DEBUG_LL backend will be overriden. Problem is fixed by removing the
automatic enabling of this option.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
arch/arm/Kconfig.debug | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/arch/arm/Kconfig.debug b/arch/arm
it.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
arch/arm/Kconfig.debug | 13 -
1 file changed, 12 insertions(+), 1 deletion(-)
diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index 4a2fc0b..450e6f7 100644
--- a/arch/arm/Kconfig.debug
+++ b/arch/arm
This defconfig already enables DEBUG_LL and by default DEBUG_LL_UART_NONE
will be selected (but due to some back compability magic I'd like to
remove is not actually honoured). DEBUG_LL_UART_PL01X is a much saner
default.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
arch/arm
Daniel Thompson (5):
ARM: versatile: Enable DEBUG_LL_UART_PL01X
ARM: ep93xx: Enable DEBUG_LL_UART_PL01X
ARM: Conceal DEBUG_LL_UART_NONE from unsupported platforms
arm: Seperate DEBUG_UART_PHYS from DEBUG_LL on EP93XX
arm: Fix DEBUG_LL for multi-platform kernels (without PL01X)
arch/arm
Stultz john.stu...@linaro.org
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
include/linux/kdb.h | 1 +
kernel/debug/kdb/kdb_main.c | 30 +-
2 files changed, 30 insertions(+), 1 deletion(-)
diff --git a/include/linux/kdb.h b/include/linux/kdb.h
Currently kiosk mode must be explicitly requested by the bootloader or
userspace. It is convenient to be able to change the default value in a
similar manner to CONFIG_MAGIC_SYSRQ_DEFAULT_MASK.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
kernel/debug/kdb/kdb_main.c | 2 +-
lib
From: Anton Vorontsov anton.voront...@linaro.org
Since we now treat KDB_REPEAT_* as flags, there is no need to
pass KDB_REPEAT_NONE. It's just the default behaviour when no
flags are specified.
Signed-off-by: Anton Vorontsov anton.voront...@linaro.org
Signed-off-by: John Stultz
: kill
Reboot: reboot
All:cpu, kgdb, (and all of the above), nmi_console
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
include/linux/kdb.h | 52 ++-
kernel/debug/kdb/kdb_bp.c | 21 ++
kernel/debug/kdb/kdb_main.c | 100
From: Anton Vorontsov anton.voront...@linaro.org
The actual values of KDB_REPEAT_* enum values and overall logic stayed
the same, but we now treat the values as flags.
This makes it possible to add other flags and combine them, plus makes
the code a lot simpler and shorter. But
From: Anton Vorontsov anton.voront...@linaro.org
The struct member is never used in the code, so we can remove it.
We will introduce real flags soon by renaming cmd_repeat to cmd_flags.
Signed-off-by: Anton Vorontsov anton.voront...@linaro.org
Signed-off-by: John Stultz john.stu...@linaro.org
From: Anton Vorontsov anton.voront...@linaro.org
We're about to add more options for commands behaviour, so let's give
a more generic name to the low-level kdb command registration function.
There are just various renames, no functional changes.
Signed-off-by: Anton Vorontsov
From: Anton Vorontsov anton.voront...@linaro.org
We're about to add more options for command behaviour, so let's expand
the meaning of kdb_repeat_t.
So far we just do various renames, there should be no functional changes.
Signed-off-by: Anton Vorontsov anton.voront...@linaro.org
Signed-off-by:
to cmd_flags
kdb: Rename kdb_register_repeat() to kdb_register_flags()
kdb: Use KDB_REPEAT_* values as flags
kdb: Remove KDB_REPEAT_NONE flag
kdb: Add enable mask for groups of commands
Daniel Thompson (3):
sysrq: Implement __handle_sysrq_nolock to avoid recursive locking in
kdb
kdb
on the key table when kdb is
triggered by something other than SysRq-g however in that case any other
CPU involved should release the spin lock before kgdb parks the slave
CPUs.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
drivers/tty/sysrq.c | 11 ---
include/linux
Currently kiosk mode must be explicitly requested by the bootloader or
userspace. It is convenient to be able to change the default value in a
similar manner to CONFIG_MAGIC_SYSRQ_DEFAULT_MASK.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
kernel/debug/kdb/kdb_main.c | 2 +-
lib
...@linaro.org
Signed-off-by: John Stultz john.stu...@linaro.org
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
include/linux/kdb.h | 10 +++---
kernel/debug/kdb/kdb_bp.c | 14
kernel/debug/kdb/kdb_main.c | 86 ++---
kernel/trace
classified as safe.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
kernel/debug/kdb/kdb_main.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/kernel/debug/kdb/kdb_main.c b/kernel/debug/kdb/kdb_main.c
index 808bf55..77b6e61 100644
--- a/kernel/debug/kdb/kdb_main.c
+++ b/kernel
-off-by: John Stultz john.stu...@linaro.org
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
include/linux/kdb.h | 1 +
kernel/debug/kdb/kdb_main.c | 31 ++-
2 files changed, 31 insertions(+), 1 deletion(-)
diff --git a/include/linux/kdb.h b
.
Signed-off-by: Anton Vorontsov anton.voront...@linaro.org
Signed-off-by: John Stultz john.stu...@linaro.org
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
include/linux/kdb.h | 2 ++
kernel/debug/kdb/kdb_main.c | 48 -
kernel/trace
-by: John Stultz john.stu...@linaro.org
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
include/linux/kdb.h| 4 ++--
kernel/debug/kdb/kdb_main.c| 6 +++---
kernel/debug/kdb/kdb_private.h | 2 +-
3 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/include/linux
on the key table when kdb is
triggered by something other than SysRq-g however in that case any other
CPU involved should release the spin lock before kgdb parks the slave
CPUs.
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
drivers/tty/sysrq.c | 11 ---
include/linux
-wise, there should
be no changes.
Signed-off-by: Anton Vorontsov anton.voront...@linaro.org
Signed-off-by: John Stultz john.stu...@linaro.org
Signed-off-by: Daniel Thompson daniel.thomp...@linaro.org
---
include/linux/kdb.h | 4 ++--
kernel/debug/kdb/kdb_main.c | 21 +++--
2
1 - 100 of 2746 matches
Mail list logo