On Tue, Feb 02, 2016 at 11:21:33AM +0100, Simon Horman wrote:

On Fri, Jan 29, 2016 at 11:17:17AM +0100, Geert Uytterhoeven wrote:

quoted

	Hi Simon, Magnus,

This patch series adds support for the external crystal feeding SCIF_CLK
to (H)SCIF on the various Renesas R-Car Gen1, Gen2, and Gen3 development
boards.  This increases the range and accuracy of supported baud rates.
Support for this has been accepted in the driver in v4.5-rc1.

Changes compared to v3:
  - Change one-line summary prefix to match current arm-soc practices,
  - Tested on r8a7794/alt,
  - Rebased,

Changes compared to v2:
  - Add support for r8a7778/bockw, r8a7779/marzen, r8a7790/lager,
    r8a7791/porter, r8a7793/gose, r8a7794/alt, and r8a7794/silk,
  - Add Reviewed-by.

Testing:
  - This was tested locally on r8a7791/koelsch and r8a7795/salvator-x,
  - This was tested remotely on r8a7778/bockw, r8a7779/marzen, and
    r8a7794/alt,
  - This has been in renesas-drivers for a while, so I assume it has
    received basic testing on other boards,
  - Hints for various levels of testing:
      - Change the "dev_dbg" after "done:" in
	drivers/tty/serial/sh-sci.c to "dev_info" to print clock and bit
	rate,
      - "yes U | tr -d '\n' > /dev/ttySCx" outputs a square wave with a
        frequency of half the bit rate on TXD,
      - Modifying the frequency of SCIF_CLK causes garbage on the serial
        console, unless you compensate by changing the bit rate in
        /chosen/stdout-path,
      - If you disable the SCIF_CLK, the brg_int clock will be used. If
        you change that clock, you'll get garbage again.

Dependencies:
  - renesas-devel-20160129-v4.5-rc1,
  - series "[PATCH v4 0/7] ARM: dts: R-Car: Add SCIF fallback
    compatibility strings",
  - series "[PATCH v4 00/11] ARM: dts: shmobile: Rename the serial port
    clock to fck",
  - series "[PATCH v4 0/7] ARM: dts: r-car: Add BRG support for
    (H)SCIF".

Thanks for applying!

Is this safe to queue up in the light of the dependency on
renesas-devel-20160129-v4.5-rc1?

Discussed elsewhere; I have queued this up.