From: Simon Horman <horms@verge.net.au>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 0/3] ARM: shmobile: Lager reference serial port support
Date: Fri, 01 Nov 2013 00:16:22 +0000 [thread overview]
Message-ID: <20131101001621.GC6818@verge.net.au> (raw)
In-Reply-To: <3965826.mx1T873Bg9@avalon>
[ CC Magnus ]
On Thu, Oct 31, 2013 at 01:21:23PM +0100, Laurent Pinchart wrote:
> Hi Simon,
>
> On Thursday 31 October 2013 14:31:44 Simon Horman wrote:
> > On Tue, Oct 29, 2013 at 04:23:10PM +0100, Laurent Pinchart wrote:
> > > Hello,
> > >
> > > This patch set implements serial port support for the Lager reference
> > > board, and adds early debugging support for the r8a7790 SoC.
> > >
> > > The code is available in my git tree at
> > >
> > > git://linuxtv.org/pinchartl/fbdev.git clocks/ccf/serial-lager
> > >
> > > The branch is based on a merge of renesas-devel-20131016, v3.12-rc7 and
> > > multiarch drivers, clk-prepare, r8a7790 CCF and sh-sci OF patches I've
> > > posted earlier today.
> > >
> > > Laurent Pinchart (3):
> > > ARM: shmobile: r8a7790: Add early debugging support
> >
> > The patch above could be applied now, right?
>
> That's correct. It will be useless though, as DEBUG_LL can only be used on
> multiarch.
I will wait, especially in the light of the point you raise below.
> Given that the SCIF/SCIFA/SCIFB port operate similarly, I wonder whether it
> wouldn't make more sense to add a generic DEBUG_SHMOBILE_SCIF implementation
> instead. We could then have sub-options to select the exact SoC type and port
> number, and compute the base address from that.
That makes a lot of sense to me.
Perhaps Magnus has an opinion?
> > If so, Magnus, could you review it?
> >
> > > ARM: shmobile: r8a7790: Add serial ports to the device tree
> > > ARM: shmobile: lager: Enable SCIF0 and SCIF1 serial ports in DT
> >
> > I would like to ask you to repost these patches once their
> > pre-requisites have been merged.
>
> Sure, no problem.
>
> --
> Regards,
>
> Laurent Pinchart
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
WARNING: multiple messages have this Message-ID (diff)
From: horms@verge.net.au (Simon Horman)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/3] ARM: shmobile: Lager reference serial port support
Date: Fri, 1 Nov 2013 09:16:22 +0900 [thread overview]
Message-ID: <20131101001621.GC6818@verge.net.au> (raw)
In-Reply-To: <3965826.mx1T873Bg9@avalon>
[ CC Magnus ]
On Thu, Oct 31, 2013 at 01:21:23PM +0100, Laurent Pinchart wrote:
> Hi Simon,
>
> On Thursday 31 October 2013 14:31:44 Simon Horman wrote:
> > On Tue, Oct 29, 2013 at 04:23:10PM +0100, Laurent Pinchart wrote:
> > > Hello,
> > >
> > > This patch set implements serial port support for the Lager reference
> > > board, and adds early debugging support for the r8a7790 SoC.
> > >
> > > The code is available in my git tree at
> > >
> > > git://linuxtv.org/pinchartl/fbdev.git clocks/ccf/serial-lager
> > >
> > > The branch is based on a merge of renesas-devel-20131016, v3.12-rc7 and
> > > multiarch drivers, clk-prepare, r8a7790 CCF and sh-sci OF patches I've
> > > posted earlier today.
> > >
> > > Laurent Pinchart (3):
> > > ARM: shmobile: r8a7790: Add early debugging support
> >
> > The patch above could be applied now, right?
>
> That's correct. It will be useless though, as DEBUG_LL can only be used on
> multiarch.
I will wait, especially in the light of the point you raise below.
> Given that the SCIF/SCIFA/SCIFB port operate similarly, I wonder whether it
> wouldn't make more sense to add a generic DEBUG_SHMOBILE_SCIF implementation
> instead. We could then have sub-options to select the exact SoC type and port
> number, and compute the base address from that.
That makes a lot of sense to me.
Perhaps Magnus has an opinion?
> > If so, Magnus, could you review it?
> >
> > > ARM: shmobile: r8a7790: Add serial ports to the device tree
> > > ARM: shmobile: lager: Enable SCIF0 and SCIF1 serial ports in DT
> >
> > I would like to ask you to repost these patches once their
> > pre-requisites have been merged.
>
> Sure, no problem.
>
> --
> Regards,
>
> Laurent Pinchart
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
next prev parent reply other threads:[~2013-11-01 0:16 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-29 15:23 [PATCH 0/3] ARM: shmobile: Lager reference serial port support Laurent Pinchart
2013-10-29 15:23 ` Laurent Pinchart
2013-10-29 15:23 ` [PATCH 1/3] ARM: shmobile: r8a7790: Add early debugging support Laurent Pinchart
2013-10-29 15:23 ` Laurent Pinchart
2014-01-08 16:31 ` [PATCH 1/3] ARM: shmobile: r8a7790: Add VIN clock support Valentine Barshak
2014-01-09 8:14 ` Simon Horman
2014-01-09 14:26 ` Valentine
2014-06-10 2:33 ` [PATCH 1/3] ARM: shmobile: r8a7790: add MSTP10 support on DTSI Kuninori Morimoto
2014-06-10 7:32 ` Geert Uytterhoeven
2014-06-10 7:35 ` Geert Uytterhoeven
2014-06-10 7:48 ` Kuninori Morimoto
2014-06-10 8:10 ` Geert Uytterhoeven
2014-06-10 8:23 ` Kuninori Morimoto
2015-01-27 9:13 ` [PATCH 1/3] ARM: shmobile: r8a7790: Add IPMMU DT nodes Laurent Pinchart
2013-10-29 15:23 ` [PATCH 2/3] ARM: shmobile: r8a7790: Add serial ports to the device tree Laurent Pinchart
2013-10-29 15:23 ` Laurent Pinchart
2013-10-29 15:23 ` [PATCH 3/3] ARM: shmobile: lager: Enable SCIF0 and SCIF1 serial ports in DT Laurent Pinchart
2013-10-29 15:23 ` Laurent Pinchart
2013-10-29 18:09 ` Sergei Shtylyov
2013-10-29 18:09 ` Sergei Shtylyov
2013-10-29 18:30 ` Laurent Pinchart
2013-10-29 18:30 ` Laurent Pinchart
2013-10-30 0:24 ` Simon Horman
2013-10-30 0:24 ` Simon Horman
2013-10-31 5:31 ` [PATCH 0/3] ARM: shmobile: Lager reference serial port support Simon Horman
2013-10-31 5:31 ` Simon Horman
2013-10-31 12:21 ` Laurent Pinchart
2013-10-31 12:21 ` Laurent Pinchart
2013-11-01 0:16 ` Simon Horman [this message]
2013-11-01 0:16 ` Simon Horman
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20131101001621.GC6818@verge.net.au \
--to=horms@verge.net.au \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.