From: Simon Horman <horms@verge.net.au>
To: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Cc: linux-sh@vger.kernel.org, linux-serial@vger.kernel.org,
Bastian Hecht <hechtb+renesas@gmail.com>,
Paul Mundt <lethal@linux-sh.org>,
devicetree@vger.kernel.org
Subject: Re: [PATCH v2 00/29] Add OF support to the sh-sci serial port driver
Date: Tue, 19 Nov 2013 01:55:39 +0000 [thread overview]
Message-ID: <20131119015539.GA9186@verge.net.au> (raw)
In-Reply-To: <1384054421-13357-1-git-send-email-laurent.pinchart+renesas@ideasonboard.com>
On Sun, Nov 10, 2013 at 04:33:12AM +0100, Laurent Pinchart wrote:
> Hello,
>
> This is the second version of the patch set adds device tree bindings for the
> sh sci serial port devices and adds OF parsing to the sh-sci driver.
>
> The bindings are based on Bastian Hecht's proposal (see
> http://www.spinics.net/lists/arm-kernel/msg228129.html). The approach taken
> here is more minimalistic: instead of describing all hardware characteristics
> that vary between the SCI device revisions in DT (such as registers layout),
> that information is stored in the driver and selected based on the compatible
> property value. Only SCI revisions used on ARM devices are supported through
> DT, as DT support for SuperH is nowhere down the line.
>
> Patches 01/29 to 06/29 clean up the sh-sci driver.
It seems to me that these could be queued up by Greg.
Let me know if you would like me to pass them on to him.
I'm quite happy for you to do that yourself. Or to wait.
Its entirely up to you.
In any case for those patches:
Acked-by: Simon Horman <horms+renesas@verge.net.au>
> Patches 07/29 to 27/29
> replace memory and interrupt resources passed through platform data with
> platform resources. Beside replacing a custom mechanism with a standard one,
> it will also make the DT parsing code simpler as resource allocation will be
> shared between DT and non-DT code paths.
> Finally, patches 28/29 to 29/29 add
> OF parsing to the sh-sci driver and create DT bindings documentation.
29 is awaiting devicetree review and 28 makes little sense
before that happens. Right?
> The patches have been test on a Lager board (r8a7790-based). Support for other
> SoCs will be added as needed. Note that all current Renesas ARM SoCs seem to
> be compatible with the generic (H)SCI(F)(AB) devices, but the plan is for
> their DT bindings to list the SoC-specific version in case incompatibilities
> are found later.
>
> Patch 08/29 ("serial: sh-sci: Support resources passed through platform
> resources") is the only dependency of the arch/arm and arch/sh patches present
> in this series. As it's too late to get it in v3.13, Simon will need a stable
> branch of the linux serial tree with that patch included.
That is patch 7 not 8, right?
Do all of the shmobile patches depends on that patch?
I would be happy to take some earlier if practical.
>
> Changes compared to v1:
>
> - Patches 04/29 to 27/29 have been added
> - Multi-interrupt support has been removed from the DT bindings as they don't
> support any SoC requiring multi-interrupt yet
>
> Cc: devicetree@vger.kernel.org
>
> Bastian Hecht (1):
> serial: sh-sci: Add OF support
>
> Laurent Pinchart (28):
> serial: sh-sci: Sort headers alphabetically
> serial: sh-sci: Remove baud rate calculation algorithm 5
> serial: sh-sci: Simplify baud rate calculation algorithms
> serial: sh-sci: Remove duplicate interrupt check in verify port op
> serial: sh-sci: Set the UPF_FIXED_PORT flag
> serial: sh-sci: Don't check IRQ in verify port operation
> serial: sh-sci: Support resources passed through platform resources
> ARM: shmobile: sh7372: Use macros to declare SCIF devices
> ARM: shmobile: sh73a0: Use macros to declare SCIF devices
> ARM: shmobile: r8a7740: Use macros to declare SCIF devices
> ARM: shmobile: r8a7779: Use macros to declare SCIF devices
> ARM: shmobile: r7s72100: Don't define SCIF platform data in an array
> ARM: shmobile: r8a73a4: Don't define SCIF platform data in an array
> ARM: shmobile: r8a7778: Don't define SCIF platform data in an array
> ARM: shmobile: r8a7790: Don't define SCIF platform data in an array
> ARM: shmobile: r8a7791: Don't define SCIF platform data in an array
> ARM: shmobile: sh7372: Declare SCIF register base and IRQ as resources
> ARM: shmobile: sh73a0: Declare SCIF register base and IRQ as resources
> ARM: shmobile: r7s72100: Declare SCIF register base and IRQ as
> resources
> ARM: shmobile: r8a73a4: Declare SCIF register base and IRQ as
> resources
> ARM: shmobile: r8a7740: Declare SCIF register base and IRQ as
> resources
> ARM: shmobile: r8a7778: Declare SCIF register base and IRQ as
> resources
> ARM: shmobile: r8a7779: Declare SCIF register base and IRQ as
> resources
> ARM: shmobile: r8a7790: Declare SCIF register base and IRQ as
> resources
> ARM: shmobile: r8a7791: Declare SCIF register base and IRQ as
> resources
> sh: Declare SCIF register base and IRQ as resources
> serial: sh-sci: Remove platform data mapbase and irqs fields
> serial: sh-sci: Add device tree bindings documentation
>
> .../bindings/serial/renesas,sci-serial.txt | 42 ++++
> arch/arm/mach-shmobile/setup-r7s72100.c | 63 +++---
> arch/arm/mach-shmobile/setup-r8a73a4.c | 65 +++---
> arch/arm/mach-shmobile/setup-r8a7740.c | 196 +++--------------
> arch/arm/mach-shmobile/setup-r8a7778.c | 45 ++--
> arch/arm/mach-shmobile/setup-r8a7779.c | 129 +++--------
> arch/arm/mach-shmobile/setup-r8a7790.c | 117 +++++-----
> arch/arm/mach-shmobile/setup-r8a7791.c | 130 +++++------
> arch/arm/mach-shmobile/setup-sh7372.c | 161 +++-----------
> arch/arm/mach-shmobile/setup-sh73a0.c | 192 +++--------------
> arch/sh/kernel/cpu/sh2/setup-sh7619.c | 27 ++-
> arch/sh/kernel/cpu/sh2a/setup-mxg.c | 9 +-
> arch/sh/kernel/cpu/sh2a/setup-sh7201.c | 72 +++++--
> arch/sh/kernel/cpu/sh2a/setup-sh7203.c | 36 +++-
> arch/sh/kernel/cpu/sh2a/setup-sh7206.c | 36 +++-
> arch/sh/kernel/cpu/sh2a/setup-sh7264.c | 96 +++++++--
> arch/sh/kernel/cpu/sh2a/setup-sh7269.c | 96 +++++++--
> arch/sh/kernel/cpu/sh3/setup-sh7705.c | 18 +-
> arch/sh/kernel/cpu/sh3/setup-sh770x.c | 27 ++-
> arch/sh/kernel/cpu/sh3/setup-sh7710.c | 18 +-
> arch/sh/kernel/cpu/sh3/setup-sh7720.c | 18 +-
> arch/sh/kernel/cpu/sh4/setup-sh4-202.c | 15 +-
> arch/sh/kernel/cpu/sh4/setup-sh7750.c | 18 +-
> arch/sh/kernel/cpu/sh4/setup-sh7760.c | 58 +++--
> arch/sh/kernel/cpu/sh4a/setup-sh7343.c | 36 +++-
> arch/sh/kernel/cpu/sh4a/setup-sh7366.c | 9 +-
> arch/sh/kernel/cpu/sh4a/setup-sh7722.c | 27 ++-
> arch/sh/kernel/cpu/sh4a/setup-sh7723.c | 54 +++--
> arch/sh/kernel/cpu/sh4a/setup-sh7724.c | 54 +++--
> arch/sh/kernel/cpu/sh4a/setup-sh7734.c | 66 ++++--
> arch/sh/kernel/cpu/sh4a/setup-sh7757.c | 27 ++-
> arch/sh/kernel/cpu/sh4a/setup-sh7763.c | 27 ++-
> arch/sh/kernel/cpu/sh4a/setup-sh7770.c | 90 ++++++--
> arch/sh/kernel/cpu/sh4a/setup-sh7780.c | 18 +-
> arch/sh/kernel/cpu/sh4a/setup-sh7785.c | 54 +++--
> arch/sh/kernel/cpu/sh4a/setup-sh7786.c | 82 +++++--
> arch/sh/kernel/cpu/sh4a/setup-shx3.c | 45 ++--
> arch/sh/kernel/cpu/sh5/setup-sh5.c | 11 +-
> drivers/tty/serial/sh-sci.c | 238 ++++++++++++++++-----
> include/linux/serial_sci.h | 35 +--
> 40 files changed, 1443 insertions(+), 1114 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/serial/renesas,sci-serial.txt
>
> --
> 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
>
next prev parent reply other threads:[~2013-11-19 1:55 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-10 3:33 [PATCH v2 00/29] Add OF support to the sh-sci serial port driver Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 01/29] serial: sh-sci: Sort headers alphabetically Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 02/29] serial: sh-sci: Remove baud rate calculation algorithm 5 Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 03/29] serial: sh-sci: Simplify baud rate calculation algorithms Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 04/29] serial: sh-sci: Remove duplicate interrupt check in verify port op Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 05/29] serial: sh-sci: Set the UPF_FIXED_PORT flag Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 06/29] serial: sh-sci: Don't check IRQ in verify port operation Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 07/29] serial: sh-sci: Support resources passed through platform resources Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 08/29] ARM: shmobile: sh7372: Use macros to declare SCIF devices Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 09/29] ARM: shmobile: sh73a0: " Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 10/29] ARM: shmobile: r8a7740: " Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 11/29] ARM: shmobile: r8a7779: " Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 12/29] ARM: shmobile: r7s72100: Don't define SCIF platform data in an array Laurent Pinchart
2013-11-10 18:35 ` Sergei Shtylyov
2013-11-11 2:41 ` Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 13/29] ARM: shmobile: r8a73a4: " Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 14/29] ARM: shmobile: r8a7778: " Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 15/29] ARM: shmobile: r8a7790: " Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 16/29] ARM: shmobile: r8a7791: " Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 17/29] ARM: shmobile: sh7372: Declare SCIF register base and IRQ as resources Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 18/29] ARM: shmobile: sh73a0: " Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 19/29] ARM: shmobile: r7s72100: " Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 20/29] ARM: shmobile: r8a73a4: " Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 21/29] ARM: shmobile: r8a7740: " Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 22/29] ARM: shmobile: r8a7778: " Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 23/29] ARM: shmobile: r8a7779: " Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 24/29] ARM: shmobile: r8a7790: " Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 25/29] ARM: shmobile: r8a7791: " Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 26/29] sh: " Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 27/29] serial: sh-sci: Remove platform data mapbase and irqs fields Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 28/29] serial: sh-sci: Add device tree bindings documentation Laurent Pinchart
2013-11-10 18:07 ` Sergei Shtylyov
2013-11-11 2:39 ` Laurent Pinchart
2013-11-11 9:40 ` Mark Rutland
2013-11-11 13:26 ` Laurent Pinchart
2013-11-11 15:48 ` Mark Rutland
2013-11-12 13:56 ` Laurent Pinchart
2013-11-10 3:33 ` [PATCH v2 29/29] serial: sh-sci: Add OF support Laurent Pinchart
2013-11-19 1:55 ` Simon Horman [this message]
2013-11-19 13:12 ` [PATCH v2 00/29] Add OF support to the sh-sci serial port driver Laurent Pinchart
2013-11-20 0:18 ` 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=20131119015539.GA9186@verge.net.au \
--to=horms@verge.net.au \
--cc=devicetree@vger.kernel.org \
--cc=hechtb+renesas@gmail.com \
--cc=laurent.pinchart+renesas@ideasonboard.com \
--cc=lethal@linux-sh.org \
--cc=linux-serial@vger.kernel.org \
--cc=linux-sh@vger.kernel.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 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).