Re: [PATCH 6/6] dt-bindings: mt8173-mtu3: add reference clock
On Mon, 2017-01-23 at 08:02 -0600, Rob Herring wrote: > On Sat, Jan 21, 2017 at 7:49 PM, Chunfeng Yun> wrote: > > Hi, > > > > On Sat, 2017-01-21 at 14:11 -0600, Rob Herring wrote: > >> On Wed, Jan 18, 2017 at 02:08:27PM +0800, Chunfeng Yun wrote: > >> > add a reference clock for compatibility > >> > >> Why? This block suddenly has 2 clocks instead of 1? > > In fact, there is a reference clock which comes from 26M oscillator > > directly. I ignore it because it is a fixed-clock in DTS, and always > > turned on for mt8173. But later, I find that I made a mistake before > > when I bring up it on a new platform whose reference clock comes from > > PLL, and need control it. So here add it, no matter it is a fixed-clock > > or not. > > Add this explanation to the changelog. > Happy Chinese New Year! Ok, I'll add it. Thanks and sorry for later reply. > Rob
Re: [PATCH 6/6] dt-bindings: mt8173-mtu3: add reference clock
On Mon, 2017-01-23 at 08:02 -0600, Rob Herring wrote: > On Sat, Jan 21, 2017 at 7:49 PM, Chunfeng Yun > wrote: > > Hi, > > > > On Sat, 2017-01-21 at 14:11 -0600, Rob Herring wrote: > >> On Wed, Jan 18, 2017 at 02:08:27PM +0800, Chunfeng Yun wrote: > >> > add a reference clock for compatibility > >> > >> Why? This block suddenly has 2 clocks instead of 1? > > In fact, there is a reference clock which comes from 26M oscillator > > directly. I ignore it because it is a fixed-clock in DTS, and always > > turned on for mt8173. But later, I find that I made a mistake before > > when I bring up it on a new platform whose reference clock comes from > > PLL, and need control it. So here add it, no matter it is a fixed-clock > > or not. > > Add this explanation to the changelog. > Happy Chinese New Year! Ok, I'll add it. Thanks and sorry for later reply. > Rob
Re: [PATCH 6/6] dt-bindings: mt8173-mtu3: add reference clock
On Sat, Jan 21, 2017 at 7:49 PM, Chunfeng Yunwrote: > Hi, > > On Sat, 2017-01-21 at 14:11 -0600, Rob Herring wrote: >> On Wed, Jan 18, 2017 at 02:08:27PM +0800, Chunfeng Yun wrote: >> > add a reference clock for compatibility >> >> Why? This block suddenly has 2 clocks instead of 1? > In fact, there is a reference clock which comes from 26M oscillator > directly. I ignore it because it is a fixed-clock in DTS, and always > turned on for mt8173. But later, I find that I made a mistake before > when I bring up it on a new platform whose reference clock comes from > PLL, and need control it. So here add it, no matter it is a fixed-clock > or not. Add this explanation to the changelog. Rob
Re: [PATCH 6/6] dt-bindings: mt8173-mtu3: add reference clock
On Sat, Jan 21, 2017 at 7:49 PM, Chunfeng Yun wrote: > Hi, > > On Sat, 2017-01-21 at 14:11 -0600, Rob Herring wrote: >> On Wed, Jan 18, 2017 at 02:08:27PM +0800, Chunfeng Yun wrote: >> > add a reference clock for compatibility >> >> Why? This block suddenly has 2 clocks instead of 1? > In fact, there is a reference clock which comes from 26M oscillator > directly. I ignore it because it is a fixed-clock in DTS, and always > turned on for mt8173. But later, I find that I made a mistake before > when I bring up it on a new platform whose reference clock comes from > PLL, and need control it. So here add it, no matter it is a fixed-clock > or not. Add this explanation to the changelog. Rob
Re: [PATCH 6/6] dt-bindings: mt8173-mtu3: add reference clock
Hi, On Sat, 2017-01-21 at 14:11 -0600, Rob Herring wrote: > On Wed, Jan 18, 2017 at 02:08:27PM +0800, Chunfeng Yun wrote: > > add a reference clock for compatibility > > Why? This block suddenly has 2 clocks instead of 1? In fact, there is a reference clock which comes from 26M oscillator directly. I ignore it because it is a fixed-clock in DTS, and always turned on for mt8173. But later, I find that I made a mistake before when I bring up it on a new platform whose reference clock comes from PLL, and need control it. So here add it, no matter it is a fixed-clock or not. > > > > > Signed-off-by: Chunfeng Yun> > --- > > .../devicetree/bindings/usb/mt8173-mtu3.txt| 10 +- > > 1 file changed, 5 insertions(+), 5 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt > > b/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt > > index e049d19..8c976cd 100644 > > --- a/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt > > +++ b/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt > > @@ -10,7 +10,7 @@ Required properties: > > - vusb33-supply : regulator of USB avdd3.3v > > - clocks : a list of phandle + clock-specifier pairs, one for each > > entry in clock-names > > - - clock-names : must contain "sys_ck" for clock of controller; > > + - clock-names : must contain "sys_ck" and "ref_ck" for clock of > > controller; > > "wakeup_deb_p0" and "wakeup_deb_p1" are optional, they are > > depends on "mediatek,enable-wakeup" > > - phys : a list of phandle + phy specifier pairs > > @@ -56,10 +56,10 @@ ssusb: usb@11271000 { > > phys = <_port0 PHY_TYPE_USB3>, > ><_port1 PHY_TYPE_USB2>; > > power-domains = < MT8173_POWER_DOMAIN_USB>; > > - clocks = < CLK_TOP_USB30_SEL>, > > + clocks = < CLK_TOP_USB30_SEL>, <>, > > < CLK_PERI_USB0>, > > < CLK_PERI_USB1>; > > - clock-names = "sys_ck", > > + clock-names = "sys_ck", "ref_ck", > > "wakeup_deb_p0", > > "wakeup_deb_p1"; > > vusb33-supply = <_vusb_reg>; > > @@ -79,8 +79,8 @@ ssusb: usb@11271000 { > > reg-names = "mac"; > > interrupts = ; > > power-domains = < MT8173_POWER_DOMAIN_USB>; > > - clocks = < CLK_TOP_USB30_SEL>; > > - clock-names = "sys_ck"; > > + clocks = < CLK_TOP_USB30_SEL>, <>; > > + clock-names = "sys_ck", "ref_ck"; > > vusb33-supply = <_vusb_reg>; > > status = "disabled"; > > }; > > -- > > 1.7.9.5 > >
Re: [PATCH 6/6] dt-bindings: mt8173-mtu3: add reference clock
Hi, On Sat, 2017-01-21 at 14:11 -0600, Rob Herring wrote: > On Wed, Jan 18, 2017 at 02:08:27PM +0800, Chunfeng Yun wrote: > > add a reference clock for compatibility > > Why? This block suddenly has 2 clocks instead of 1? In fact, there is a reference clock which comes from 26M oscillator directly. I ignore it because it is a fixed-clock in DTS, and always turned on for mt8173. But later, I find that I made a mistake before when I bring up it on a new platform whose reference clock comes from PLL, and need control it. So here add it, no matter it is a fixed-clock or not. > > > > > Signed-off-by: Chunfeng Yun > > --- > > .../devicetree/bindings/usb/mt8173-mtu3.txt| 10 +- > > 1 file changed, 5 insertions(+), 5 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt > > b/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt > > index e049d19..8c976cd 100644 > > --- a/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt > > +++ b/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt > > @@ -10,7 +10,7 @@ Required properties: > > - vusb33-supply : regulator of USB avdd3.3v > > - clocks : a list of phandle + clock-specifier pairs, one for each > > entry in clock-names > > - - clock-names : must contain "sys_ck" for clock of controller; > > + - clock-names : must contain "sys_ck" and "ref_ck" for clock of > > controller; > > "wakeup_deb_p0" and "wakeup_deb_p1" are optional, they are > > depends on "mediatek,enable-wakeup" > > - phys : a list of phandle + phy specifier pairs > > @@ -56,10 +56,10 @@ ssusb: usb@11271000 { > > phys = <_port0 PHY_TYPE_USB3>, > ><_port1 PHY_TYPE_USB2>; > > power-domains = < MT8173_POWER_DOMAIN_USB>; > > - clocks = < CLK_TOP_USB30_SEL>, > > + clocks = < CLK_TOP_USB30_SEL>, <>, > > < CLK_PERI_USB0>, > > < CLK_PERI_USB1>; > > - clock-names = "sys_ck", > > + clock-names = "sys_ck", "ref_ck", > > "wakeup_deb_p0", > > "wakeup_deb_p1"; > > vusb33-supply = <_vusb_reg>; > > @@ -79,8 +79,8 @@ ssusb: usb@11271000 { > > reg-names = "mac"; > > interrupts = ; > > power-domains = < MT8173_POWER_DOMAIN_USB>; > > - clocks = < CLK_TOP_USB30_SEL>; > > - clock-names = "sys_ck"; > > + clocks = < CLK_TOP_USB30_SEL>, <>; > > + clock-names = "sys_ck", "ref_ck"; > > vusb33-supply = <_vusb_reg>; > > status = "disabled"; > > }; > > -- > > 1.7.9.5 > >
Re: [PATCH 6/6] dt-bindings: mt8173-mtu3: add reference clock
On Wed, Jan 18, 2017 at 02:08:27PM +0800, Chunfeng Yun wrote: > add a reference clock for compatibility Why? This block suddenly has 2 clocks instead of 1? > > Signed-off-by: Chunfeng Yun> --- > .../devicetree/bindings/usb/mt8173-mtu3.txt| 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt > b/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt > index e049d19..8c976cd 100644 > --- a/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt > +++ b/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt > @@ -10,7 +10,7 @@ Required properties: > - vusb33-supply : regulator of USB avdd3.3v > - clocks : a list of phandle + clock-specifier pairs, one for each > entry in clock-names > - - clock-names : must contain "sys_ck" for clock of controller; > + - clock-names : must contain "sys_ck" and "ref_ck" for clock of controller; > "wakeup_deb_p0" and "wakeup_deb_p1" are optional, they are > depends on "mediatek,enable-wakeup" > - phys : a list of phandle + phy specifier pairs > @@ -56,10 +56,10 @@ ssusb: usb@11271000 { > phys = <_port0 PHY_TYPE_USB3>, > <_port1 PHY_TYPE_USB2>; > power-domains = < MT8173_POWER_DOMAIN_USB>; > - clocks = < CLK_TOP_USB30_SEL>, > + clocks = < CLK_TOP_USB30_SEL>, <>, >< CLK_PERI_USB0>, >< CLK_PERI_USB1>; > - clock-names = "sys_ck", > + clock-names = "sys_ck", "ref_ck", > "wakeup_deb_p0", > "wakeup_deb_p1"; > vusb33-supply = <_vusb_reg>; > @@ -79,8 +79,8 @@ ssusb: usb@11271000 { > reg-names = "mac"; > interrupts = ; > power-domains = < MT8173_POWER_DOMAIN_USB>; > - clocks = < CLK_TOP_USB30_SEL>; > - clock-names = "sys_ck"; > + clocks = < CLK_TOP_USB30_SEL>, <>; > + clock-names = "sys_ck", "ref_ck"; > vusb33-supply = <_vusb_reg>; > status = "disabled"; > }; > -- > 1.7.9.5 >
Re: [PATCH 6/6] dt-bindings: mt8173-mtu3: add reference clock
On Wed, Jan 18, 2017 at 02:08:27PM +0800, Chunfeng Yun wrote: > add a reference clock for compatibility Why? This block suddenly has 2 clocks instead of 1? > > Signed-off-by: Chunfeng Yun > --- > .../devicetree/bindings/usb/mt8173-mtu3.txt| 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt > b/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt > index e049d19..8c976cd 100644 > --- a/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt > +++ b/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt > @@ -10,7 +10,7 @@ Required properties: > - vusb33-supply : regulator of USB avdd3.3v > - clocks : a list of phandle + clock-specifier pairs, one for each > entry in clock-names > - - clock-names : must contain "sys_ck" for clock of controller; > + - clock-names : must contain "sys_ck" and "ref_ck" for clock of controller; > "wakeup_deb_p0" and "wakeup_deb_p1" are optional, they are > depends on "mediatek,enable-wakeup" > - phys : a list of phandle + phy specifier pairs > @@ -56,10 +56,10 @@ ssusb: usb@11271000 { > phys = <_port0 PHY_TYPE_USB3>, > <_port1 PHY_TYPE_USB2>; > power-domains = < MT8173_POWER_DOMAIN_USB>; > - clocks = < CLK_TOP_USB30_SEL>, > + clocks = < CLK_TOP_USB30_SEL>, <>, >< CLK_PERI_USB0>, >< CLK_PERI_USB1>; > - clock-names = "sys_ck", > + clock-names = "sys_ck", "ref_ck", > "wakeup_deb_p0", > "wakeup_deb_p1"; > vusb33-supply = <_vusb_reg>; > @@ -79,8 +79,8 @@ ssusb: usb@11271000 { > reg-names = "mac"; > interrupts = ; > power-domains = < MT8173_POWER_DOMAIN_USB>; > - clocks = < CLK_TOP_USB30_SEL>; > - clock-names = "sys_ck"; > + clocks = < CLK_TOP_USB30_SEL>, <>; > + clock-names = "sys_ck", "ref_ck"; > vusb33-supply = <_vusb_reg>; > status = "disabled"; > }; > -- > 1.7.9.5 >
[PATCH 6/6] dt-bindings: mt8173-mtu3: add reference clock
add a reference clock for compatibility Signed-off-by: Chunfeng Yun--- .../devicetree/bindings/usb/mt8173-mtu3.txt| 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt b/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt index e049d19..8c976cd 100644 --- a/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt +++ b/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt @@ -10,7 +10,7 @@ Required properties: - vusb33-supply : regulator of USB avdd3.3v - clocks : a list of phandle + clock-specifier pairs, one for each entry in clock-names - - clock-names : must contain "sys_ck" for clock of controller; + - clock-names : must contain "sys_ck" and "ref_ck" for clock of controller; "wakeup_deb_p0" and "wakeup_deb_p1" are optional, they are depends on "mediatek,enable-wakeup" - phys : a list of phandle + phy specifier pairs @@ -56,10 +56,10 @@ ssusb: usb@11271000 { phys = <_port0 PHY_TYPE_USB3>, <_port1 PHY_TYPE_USB2>; power-domains = < MT8173_POWER_DOMAIN_USB>; - clocks = < CLK_TOP_USB30_SEL>, + clocks = < CLK_TOP_USB30_SEL>, <>, < CLK_PERI_USB0>, < CLK_PERI_USB1>; - clock-names = "sys_ck", + clock-names = "sys_ck", "ref_ck", "wakeup_deb_p0", "wakeup_deb_p1"; vusb33-supply = <_vusb_reg>; @@ -79,8 +79,8 @@ ssusb: usb@11271000 { reg-names = "mac"; interrupts = ; power-domains = < MT8173_POWER_DOMAIN_USB>; - clocks = < CLK_TOP_USB30_SEL>; - clock-names = "sys_ck"; + clocks = < CLK_TOP_USB30_SEL>, <>; + clock-names = "sys_ck", "ref_ck"; vusb33-supply = <_vusb_reg>; status = "disabled"; }; -- 1.7.9.5
[PATCH 6/6] dt-bindings: mt8173-mtu3: add reference clock
add a reference clock for compatibility Signed-off-by: Chunfeng Yun --- .../devicetree/bindings/usb/mt8173-mtu3.txt| 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt b/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt index e049d19..8c976cd 100644 --- a/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt +++ b/Documentation/devicetree/bindings/usb/mt8173-mtu3.txt @@ -10,7 +10,7 @@ Required properties: - vusb33-supply : regulator of USB avdd3.3v - clocks : a list of phandle + clock-specifier pairs, one for each entry in clock-names - - clock-names : must contain "sys_ck" for clock of controller; + - clock-names : must contain "sys_ck" and "ref_ck" for clock of controller; "wakeup_deb_p0" and "wakeup_deb_p1" are optional, they are depends on "mediatek,enable-wakeup" - phys : a list of phandle + phy specifier pairs @@ -56,10 +56,10 @@ ssusb: usb@11271000 { phys = <_port0 PHY_TYPE_USB3>, <_port1 PHY_TYPE_USB2>; power-domains = < MT8173_POWER_DOMAIN_USB>; - clocks = < CLK_TOP_USB30_SEL>, + clocks = < CLK_TOP_USB30_SEL>, <>, < CLK_PERI_USB0>, < CLK_PERI_USB1>; - clock-names = "sys_ck", + clock-names = "sys_ck", "ref_ck", "wakeup_deb_p0", "wakeup_deb_p1"; vusb33-supply = <_vusb_reg>; @@ -79,8 +79,8 @@ ssusb: usb@11271000 { reg-names = "mac"; interrupts = ; power-domains = < MT8173_POWER_DOMAIN_USB>; - clocks = < CLK_TOP_USB30_SEL>; - clock-names = "sys_ck"; + clocks = < CLK_TOP_USB30_SEL>, <>; + clock-names = "sys_ck", "ref_ck"; vusb33-supply = <_vusb_reg>; status = "disabled"; }; -- 1.7.9.5