Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1265097 > unrolled thread

[PATCH] clk: imx: clk-imx6q: Let OSC to be routed to anaclk2/2b

Started byMichael Trimarchi <michael@amarulasolutions.com>
First post2015-11-08 15:10 +0100
Last post2015-11-17 10:40 +0100
Articles 5 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] clk: imx: clk-imx6q: Let OSC to be routed to anaclk2/2b Michael Trimarchi <michael@amarulasolutions.com> - 2015-11-08 15:10 +0100
    Re: [PATCH] clk: imx: clk-imx6q: Let OSC to be routed to anaclk2/2b Michael Trimarchi <michael@amarulasolutions.com> - 2015-11-15 11:30 +0100
      [PATCH V2] clk: imx: clk-imx6q: Let OSC to be routed to anaclk2/2b Michael Trimarchi <michael@amarulasolutions.com> - 2015-11-15 11:40 +0100
        Re: [PATCH V2] clk: imx: clk-imx6q: Let OSC to be routed to  anaclk2/2b Shawn Guo <shawnguo@kernel.org> - 2015-11-25 06:30 +0100
    Re: [PATCH] clk: imx: clk-imx6q: Let OSC to be routed to anaclk2/2b Lucas Stach <l.stach@pengutronix.de> - 2015-11-17 10:40 +0100

#1265097 — [PATCH] clk: imx: clk-imx6q: Let OSC to be routed to anaclk2/2b

FromMichael Trimarchi <michael@amarulasolutions.com>
Date2015-11-08 15:10 +0100
Subject[PATCH] clk: imx: clk-imx6q: Let OSC to be routed to anaclk2/2b
Message-ID<qsyPn-18x-15@gated-at.bofh.it>
OSC can be used as USB hub source clock. An example we can route to
CLK2_P imx6 pin

Signed-off-by: Michael Trimarchi <michael@amarulasolutions.com>
---
 drivers/clk/imx/clk-imx6q.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/clk/imx/clk-imx6q.c b/drivers/clk/imx/clk-imx6q.c
index b2c1c04..8fe0015 100644
--- a/drivers/clk/imx/clk-imx6q.c
+++ b/drivers/clk/imx/clk-imx6q.c
@@ -70,7 +70,8 @@ static const char *cko_sels[] = { "cko1", "cko2", };
 static const char *lvds_sels[] = {
 	"dummy", "dummy", "dummy", "dummy", "dummy", "dummy",
 	"pll4_audio", "pll5_video", "pll8_mlb", "enet_ref",
-	"pcie_ref_125m", "sata_ref_100m",
+	"pcie_ref_125m", "sata_ref_100m",  "usbphy1", "usbphy2",
+	"dummy", "dummy", "dummy", "dummy", "osc",
 };
 static const char *pll_bypass_src_sels[] = { "osc", "lvds1_in", "lvds2_in", "dummy", };
 static const char *pll1_bypass_sels[] = { "pll1", "pll1_bypass_src", };
-- 
2.6.3

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1269661

FromMichael Trimarchi <michael@amarulasolutions.com>
Date2015-11-15 11:30 +0100
Message-ID<qv2Jj-6m-7@gated-at.bofh.it>
In reply to#1265097
Hi

On Sun, Nov 08, 2015 at 03:07:22PM +0100, Michael Trimarchi wrote:
> OSC can be used as USB hub source clock. An example we can route to
> CLK2_P imx6 pin
> 

Do I need to include an example in the commit message?

	usb_hub: usb-hub {
		compatible = "smsc,usb3503a";
		clocks = <&clks IMX6QDL_CLK_LVDS2_GATE>;
		clock-names = "refclk";
	};
};

&clks {
	assigned-clocks = <&clks IMX6QDL_CLK_LVDS2_SEL>;
	assigned-clock-parents = <&clks IMX6QDL_CLK_OSC>;
};

 osc                                      5            5    24000000          0 0  
    cko2_sel                              0            0    24000000          0 0  
       cko2_podf                          0            0    24000000          0 0  
          cko2                            0            0    24000000          0 0  
             cko                          0            0    24000000          0 0  
    lvds2_sel                             1            1    24000000          0 0  
       lvds2_gate                         1            1    24000000          0 0  

Best regards
Michael

> Signed-off-by: Michael Trimarchi <michael@amarulasolutions.com>
> ---
>  drivers/clk/imx/clk-imx6q.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/clk/imx/clk-imx6q.c b/drivers/clk/imx/clk-imx6q.c
> index b2c1c04..8fe0015 100644
> --- a/drivers/clk/imx/clk-imx6q.c
> +++ b/drivers/clk/imx/clk-imx6q.c
> @@ -70,7 +70,8 @@ static const char *cko_sels[] = { "cko1", "cko2", };
>  static const char *lvds_sels[] = {
>  	"dummy", "dummy", "dummy", "dummy", "dummy", "dummy",
>  	"pll4_audio", "pll5_video", "pll8_mlb", "enet_ref",
> -	"pcie_ref_125m", "sata_ref_100m",
> +	"pcie_ref_125m", "sata_ref_100m",  "usbphy1", "usbphy2",
> +	"dummy", "dummy", "dummy", "dummy", "osc",
>  };
>  static const char *pll_bypass_src_sels[] = { "osc", "lvds1_in", "lvds2_in", "dummy", };
>  static const char *pll1_bypass_sels[] = { "pll1", "pll1_bypass_src", };
> -- 
> 2.6.3
> 

-- 
| Michael Nazzareno Trimarchi                     Amarula Solutions BV |
| COO  -  Founder                                      Cruquiuskade 47 |
| +31(0)851119172                                 Amsterdam 1018 AM NL |
|                  [`as] http://www.amarulasolutions.com               |
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1269662 — [PATCH V2] clk: imx: clk-imx6q: Let OSC to be routed to anaclk2/2b

FromMichael Trimarchi <michael@amarulasolutions.com>
Date2015-11-15 11:40 +0100
Subject[PATCH V2] clk: imx: clk-imx6q: Let OSC to be routed to anaclk2/2b
Message-ID<qv2SZ-9B-9@gated-at.bofh.it>
In reply to#1269661
OSC can be used as USB hub source clock. An example we can route to
CLK2_P imx6 pin.

This show a usage example:

[...]
	usb_hub: usb-hub {
		compatible = "smsc,usb3503a";
		clocks = <&clks IMX6QDL_CLK_LVDS2_GATE>;
		clock-names = "refclk";
	};
};

[...]
&clks {
        assigned-clocks = <&clks IMX6QDL_CLK_LVDS2_SEL>;
        assigned-clock-parents = <&clks IMX6QDL_CLK_OSC>;
};

/sys/kernel/debug/clk/clk_summary

osc                                 5            5    24000000          0 0
[...]
    lvds2_sel                       1            1    24000000          0 0
       lvds2_gate                   1            1    24000000          0 0
[...]

Signed-off-by: Michael Trimarchi <michael@amarulasolutions.com>
---
Changes in v2:
	- rework the commit message to give a better description

 drivers/clk/imx/clk-imx6q.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/clk/imx/clk-imx6q.c b/drivers/clk/imx/clk-imx6q.c
index b2c1c04..8fe0015 100644
--- a/drivers/clk/imx/clk-imx6q.c
+++ b/drivers/clk/imx/clk-imx6q.c
@@ -70,7 +70,8 @@ static const char *cko_sels[] = { "cko1", "cko2", };
 static const char *lvds_sels[] = {
 	"dummy", "dummy", "dummy", "dummy", "dummy", "dummy",
 	"pll4_audio", "pll5_video", "pll8_mlb", "enet_ref",
-	"pcie_ref_125m", "sata_ref_100m",
+	"pcie_ref_125m", "sata_ref_100m",  "usbphy1", "usbphy2",
+	"dummy", "dummy", "dummy", "dummy", "osc",
 };
 static const char *pll_bypass_src_sels[] = { "osc", "lvds1_in", "lvds2_in", "dummy", };
 static const char *pll1_bypass_sels[] = { "pll1", "pll1_bypass_src", };
-- 
2.6.3
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1277020 — Re: [PATCH V2] clk: imx: clk-imx6q: Let OSC to be routed to anaclk2/2b

FromShawn Guo <shawnguo@kernel.org>
Date2015-11-25 06:30 +0100
SubjectRe: [PATCH V2] clk: imx: clk-imx6q: Let OSC to be routed to anaclk2/2b
Message-ID<qyAOt-1M9-1@gated-at.bofh.it>
In reply to#1269662
On Sun, Nov 15, 2015 at 11:38:04AM +0100, Michael Trimarchi wrote:
> OSC can be used as USB hub source clock. An example we can route to
> CLK2_P imx6 pin.
> 
> This show a usage example:
> 
> [...]
> 	usb_hub: usb-hub {
> 		compatible = "smsc,usb3503a";
> 		clocks = <&clks IMX6QDL_CLK_LVDS2_GATE>;
> 		clock-names = "refclk";
> 	};
> };
> 
> [...]
> &clks {
>         assigned-clocks = <&clks IMX6QDL_CLK_LVDS2_SEL>;
>         assigned-clock-parents = <&clks IMX6QDL_CLK_OSC>;
> };
> 
> /sys/kernel/debug/clk/clk_summary
> 
> osc                                 5            5    24000000          0 0
> [...]
>     lvds2_sel                       1            1    24000000          0 0
>        lvds2_gate                   1            1    24000000          0 0
> [...]
> 
> Signed-off-by: Michael Trimarchi <michael@amarulasolutions.com>

Applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1271008

FromLucas Stach <l.stach@pengutronix.de>
Date2015-11-17 10:40 +0100
Message-ID<qvKU2-3cW-31@gated-at.bofh.it>
In reply to#1265097
Am Sonntag, den 08.11.2015, 15:07 +0100 schrieb Michael Trimarchi:
> OSC can be used as USB hub source clock. An example we can route to
> CLK2_P imx6 pin
> 
> Signed-off-by: Michael Trimarchi <michael@amarulasolutions.com>

I don't think the more verbose commit message of V2 adds any more value,
so I'm explicitly looking at V1. Change looks in line with the TRM, so:

Reviewed-by: Lucas Stach <l.stach@pengutronix.de>

> ---
>  drivers/clk/imx/clk-imx6q.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/clk/imx/clk-imx6q.c b/drivers/clk/imx/clk-imx6q.c
> index b2c1c04..8fe0015 100644
> --- a/drivers/clk/imx/clk-imx6q.c
> +++ b/drivers/clk/imx/clk-imx6q.c
> @@ -70,7 +70,8 @@ static const char *cko_sels[] = { "cko1", "cko2", };
>  static const char *lvds_sels[] = {
>  	"dummy", "dummy", "dummy", "dummy", "dummy", "dummy",
>  	"pll4_audio", "pll5_video", "pll8_mlb", "enet_ref",
> -	"pcie_ref_125m", "sata_ref_100m",
> +	"pcie_ref_125m", "sata_ref_100m",  "usbphy1", "usbphy2",
> +	"dummy", "dummy", "dummy", "dummy", "osc",
>  };
>  static const char *pll_bypass_src_sels[] = { "osc", "lvds1_in", "lvds2_in", "dummy", };
>  static const char *pll1_bypass_sels[] = { "pll1", "pll1_bypass_src", };

-- 
Pengutronix e.K.             | Lucas Stach                 |
Industrial Linux Solutions   | http://www.pengutronix.de/  |

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web