Re: [PATCH 1/2] clk: renesas: rcar-gen3: add documentation for SD clocks

2018-11-05 Thread Geert Uytterhoeven
Hi Wolfram,

On Thu, Nov 1, 2018 at 8:37 PM Wolfram Sang  wrote:
> On Thu, Nov 01, 2018 at 12:25:17AM +0100, Niklas Söderlund wrote:
> > From: Niklas Söderlund 
> >
> > Document the known use cases of the different clock settings. This is
> > useful as different SoC and ES versions uses different settings to do
> > the same thing as there are more then one combination to achieve the
> > same SDn clock speed.
> >
> > Signed-off-by: Niklas Söderlund 
>
> Reviewed-by: Wolfram Sang 
> Tested-by: Wolfram Sang 

Is a Tested-by appropriate for a comment-only patch?

Gr{oetje,eeting}s,

Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds


Re: [PATCH 1/2] clk: renesas: rcar-gen3: add documentation for SD clocks

2018-11-01 Thread Wolfram Sang
On Thu, Nov 01, 2018 at 12:25:17AM +0100, Niklas Söderlund wrote:
> From: Niklas Söderlund 
> 
> Document the known use cases of the different clock settings. This is
> useful as different SoC and ES versions uses different settings to do
> the same thing as there are more then one combination to achieve the
> same SDn clock speed.
> 
> Signed-off-by: Niklas Söderlund 

Reviewed-by: Wolfram Sang 
Tested-by: Wolfram Sang 



signature.asc
Description: PGP signature


[PATCH 1/2] clk: renesas: rcar-gen3: add documentation for SD clocks

2018-10-31 Thread Niklas Söderlund
From: Niklas Söderlund 

Document the known use cases of the different clock settings. This is
useful as different SoC and ES versions uses different settings to do
the same thing as there are more then one combination to achieve the
same SDn clock speed.

Signed-off-by: Niklas Söderlund 
---
 drivers/clk/renesas/rcar-gen3-cpg.c | 10 +-
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/clk/renesas/rcar-gen3-cpg.c 
b/drivers/clk/renesas/rcar-gen3-cpg.c
index 628b63b85d3f09c5..ff56ac6134111c38 100644
--- a/drivers/clk/renesas/rcar-gen3-cpg.c
+++ b/drivers/clk/renesas/rcar-gen3-cpg.c
@@ -233,13 +233,13 @@ struct sd_clock {
  * sd_srcfc   sd_fc   div
  * stp_hck   stp_ck(div)  (div) = sd_srcfc x sd_fc
  *---
- *  0 0 0 (1)  1 (4)  4
- *  0 0 1 (2)  1 (4)  8
- *  1 0 2 (4)  1 (4) 16
- *  1 0 3 (8)  1 (4) 32
+ *  0 0 0 (1)  1 (4)  4 : SDR104 / HS200 / HS400 (8 
TAP)
+ *  0 0 1 (2)  1 (4)  8 : SDR50
+ *  1 0 2 (4)  1 (4) 16 : HS / SDR25
+ *  1 0 3 (8)  1 (4) 32 : NS / SDR12
  *  1 0 4 (16) 1 (4) 64
  *  0 0 0 (1)  0 (2)  2
- *  0 0 1 (2)  0 (2)  4
+ *  0 0 1 (2)  0 (2)  4 : SDR104 / HS200 / HS400 (4 
TAP)
  *  1 0 2 (4)  0 (2)  8
  *  1 0 3 (8)  0 (2) 16
  *  1 0 4 (16) 0 (2) 32
-- 
2.19.1