All of lore.kernel.org
 help / color / mirror / Atom feed
* ARM: mvebu: CONFIG_DEBUG_MVEBU_UART_ALTERNATE
@ 2015-01-12  9:33 ` Paul Bolle
  0 siblings, 0 replies; 4+ messages in thread
From: Paul Bolle @ 2015-01-12  9:33 UTC (permalink / raw)
  To: Maxime Ripard
  Cc: Valentin Rothberg, Jason Cooper, Gregory CLEMENT, Andrew Lunn,
	devicetree, linux-arm-kernel, linux-kernel

Maxime,

Your commit d91125ddf962 ("ARM: mvebu: Rename DEBUG_LL to indicate UART
index") is included in today's linux-next (ie, next-20150112). So is the
related commit bd920490047a ("ARM: mvebu: Add UART1 as DEBUG_LL possible
target"). I noticed because a script I use, spotted a minor issue with
the first commit.

See, the first commit renamed DEBUG_MVEBU_UART_ALTERNATE to
DEBUG_MVEBU_UART0_ALTERNATE (so "0" was added). But there's still a last
reference to CONFIG_DEBUG_MVEBU_UART_ALTERNATE in a comment in
arch/arm/boot/dts/armada-xp-axpwifiap.dts. Is a patch to rename that
last reference queued somewhere? If not, should I submit a trivial patch
to do that?


Paul Bolle


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

* ARM: mvebu: CONFIG_DEBUG_MVEBU_UART_ALTERNATE
@ 2015-01-12  9:33 ` Paul Bolle
  0 siblings, 0 replies; 4+ messages in thread
From: Paul Bolle @ 2015-01-12  9:33 UTC (permalink / raw)
  To: linux-arm-kernel

Maxime,

Your commit d91125ddf962 ("ARM: mvebu: Rename DEBUG_LL to indicate UART
index") is included in today's linux-next (ie, next-20150112). So is the
related commit bd920490047a ("ARM: mvebu: Add UART1 as DEBUG_LL possible
target"). I noticed because a script I use, spotted a minor issue with
the first commit.

See, the first commit renamed DEBUG_MVEBU_UART_ALTERNATE to
DEBUG_MVEBU_UART0_ALTERNATE (so "0" was added). But there's still a last
reference to CONFIG_DEBUG_MVEBU_UART_ALTERNATE in a comment in
arch/arm/boot/dts/armada-xp-axpwifiap.dts. Is a patch to rename that
last reference queued somewhere? If not, should I submit a trivial patch
to do that?


Paul Bolle

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

* Re: ARM: mvebu: CONFIG_DEBUG_MVEBU_UART_ALTERNATE
  2015-01-12  9:33 ` Paul Bolle
@ 2015-01-12 13:04   ` Andrew Lunn
  -1 siblings, 0 replies; 4+ messages in thread
From: Andrew Lunn @ 2015-01-12 13:04 UTC (permalink / raw)
  To: Paul Bolle
  Cc: Maxime Ripard, Jason Cooper, devicetree, linux-kernel,
	Gregory CLEMENT, linux-arm-kernel, Valentin Rothberg

On Mon, Jan 12, 2015 at 10:33:56AM +0100, Paul Bolle wrote:
> Maxime,
> 
> Your commit d91125ddf962 ("ARM: mvebu: Rename DEBUG_LL to indicate UART
> index") is included in today's linux-next (ie, next-20150112). So is the
> related commit bd920490047a ("ARM: mvebu: Add UART1 as DEBUG_LL possible
> target"). I noticed because a script I use, spotted a minor issue with
> the first commit.
> 
> See, the first commit renamed DEBUG_MVEBU_UART_ALTERNATE to
> DEBUG_MVEBU_UART0_ALTERNATE (so "0" was added). But there's still a last
> reference to CONFIG_DEBUG_MVEBU_UART_ALTERNATE in a comment in
> arch/arm/boot/dts/armada-xp-axpwifiap.dts. Is a patch to rename that
> last reference queued somewhere? If not, should I submit a trivial patch
> to do that?

Hi Paul

Nothing queued, so please do submit a patch.

Thanks
	Andrew

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

* ARM: mvebu: CONFIG_DEBUG_MVEBU_UART_ALTERNATE
@ 2015-01-12 13:04   ` Andrew Lunn
  0 siblings, 0 replies; 4+ messages in thread
From: Andrew Lunn @ 2015-01-12 13:04 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Jan 12, 2015 at 10:33:56AM +0100, Paul Bolle wrote:
> Maxime,
> 
> Your commit d91125ddf962 ("ARM: mvebu: Rename DEBUG_LL to indicate UART
> index") is included in today's linux-next (ie, next-20150112). So is the
> related commit bd920490047a ("ARM: mvebu: Add UART1 as DEBUG_LL possible
> target"). I noticed because a script I use, spotted a minor issue with
> the first commit.
> 
> See, the first commit renamed DEBUG_MVEBU_UART_ALTERNATE to
> DEBUG_MVEBU_UART0_ALTERNATE (so "0" was added). But there's still a last
> reference to CONFIG_DEBUG_MVEBU_UART_ALTERNATE in a comment in
> arch/arm/boot/dts/armada-xp-axpwifiap.dts. Is a patch to rename that
> last reference queued somewhere? If not, should I submit a trivial patch
> to do that?

Hi Paul

Nothing queued, so please do submit a patch.

Thanks
	Andrew

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

end of thread, other threads:[~2015-01-12 13:06 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-12  9:33 ARM: mvebu: CONFIG_DEBUG_MVEBU_UART_ALTERNATE Paul Bolle
2015-01-12  9:33 ` Paul Bolle
2015-01-12 13:04 ` Andrew Lunn
2015-01-12 13:04   ` Andrew Lunn

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.