Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-10 Thread Eddie Huang
On Fri, 2015-07-10 at 16:11 +0800, Daniel Kurtz wrote: > On Fri, Jul 10, 2015 at 3:27 PM, Eddie Huang wrote: > > Hi all, > > > > On Wed, 2015-07-08 at 13:44 +0800, Sascha Hauer wrote: > >> On Wed, Jul 08, 2015 at 10:37:21AM +0800, Eddie Huang wrote: > >> > On Tue, 2015-07-07 at 23:10 +0800,

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-10 Thread Daniel Kurtz
On Fri, Jul 10, 2015 at 3:27 PM, Eddie Huang wrote: > Hi all, > > On Wed, 2015-07-08 at 13:44 +0800, Sascha Hauer wrote: >> On Wed, Jul 08, 2015 at 10:37:21AM +0800, Eddie Huang wrote: >> > On Tue, 2015-07-07 at 23:10 +0800, Daniel Kurtz wrote: >> > > On Tue, Jul 7, 2015 at 10:36 PM, Sascha Hauer

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-10 Thread Eddie Huang
Hi all, On Wed, 2015-07-08 at 13:44 +0800, Sascha Hauer wrote: > On Wed, Jul 08, 2015 at 10:37:21AM +0800, Eddie Huang wrote: > > On Tue, 2015-07-07 at 23:10 +0800, Daniel Kurtz wrote: > > > On Tue, Jul 7, 2015 at 10:36 PM, Sascha Hauer > > > wrote: > > > > On Tue, Jul 07, 2015 at 10:15:29PM

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-10 Thread Daniel Kurtz
On Fri, Jul 10, 2015 at 3:27 PM, Eddie Huang eddie.hu...@mediatek.com wrote: Hi all, On Wed, 2015-07-08 at 13:44 +0800, Sascha Hauer wrote: On Wed, Jul 08, 2015 at 10:37:21AM +0800, Eddie Huang wrote: On Tue, 2015-07-07 at 23:10 +0800, Daniel Kurtz wrote: On Tue, Jul 7, 2015 at 10:36 PM,

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-10 Thread Eddie Huang
Hi all, On Wed, 2015-07-08 at 13:44 +0800, Sascha Hauer wrote: On Wed, Jul 08, 2015 at 10:37:21AM +0800, Eddie Huang wrote: On Tue, 2015-07-07 at 23:10 +0800, Daniel Kurtz wrote: On Tue, Jul 7, 2015 at 10:36 PM, Sascha Hauer s.ha...@pengutronix.de wrote: On Tue, Jul 07, 2015 at

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-10 Thread Eddie Huang
On Fri, 2015-07-10 at 16:11 +0800, Daniel Kurtz wrote: On Fri, Jul 10, 2015 at 3:27 PM, Eddie Huang eddie.hu...@mediatek.com wrote: Hi all, On Wed, 2015-07-08 at 13:44 +0800, Sascha Hauer wrote: On Wed, Jul 08, 2015 at 10:37:21AM +0800, Eddie Huang wrote: On Tue, 2015-07-07 at 23:10

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-07 Thread Sascha Hauer
On Wed, Jul 08, 2015 at 10:37:21AM +0800, Eddie Huang wrote: > On Tue, 2015-07-07 at 23:10 +0800, Daniel Kurtz wrote: > > On Tue, Jul 7, 2015 at 10:36 PM, Sascha Hauer > > wrote: > > > On Tue, Jul 07, 2015 at 10:15:29PM +0800, Daniel Kurtz wrote: > > >> On Tue, Jul 7, 2015 at 9:07 PM, Sascha

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-07 Thread Eddie Huang
On Tue, 2015-07-07 at 23:10 +0800, Daniel Kurtz wrote: > On Tue, Jul 7, 2015 at 10:36 PM, Sascha Hauer wrote: > > On Tue, Jul 07, 2015 at 10:15:29PM +0800, Daniel Kurtz wrote: > >> On Tue, Jul 7, 2015 at 9:07 PM, Sascha Hauer > >> wrote: > >> > On Thu, Jun 18, 2015 at 01:29:11PM +0800, Eddie

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-07 Thread Daniel Kurtz
On Tue, Jul 7, 2015 at 10:36 PM, Sascha Hauer wrote: > On Tue, Jul 07, 2015 at 10:15:29PM +0800, Daniel Kurtz wrote: >> On Tue, Jul 7, 2015 at 9:07 PM, Sascha Hauer wrote: >> > On Thu, Jun 18, 2015 at 01:29:11PM +0800, Eddie Huang wrote: >> >> Add clk_null, which represents clocks that can not /

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-07 Thread Sascha Hauer
On Tue, Jul 07, 2015 at 10:15:29PM +0800, Daniel Kurtz wrote: > On Tue, Jul 7, 2015 at 9:07 PM, Sascha Hauer wrote: > > On Thu, Jun 18, 2015 at 01:29:11PM +0800, Eddie Huang wrote: > >> Add clk_null, which represents clocks that can not / need not > >> controlled by software. > >> There are many

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-07 Thread Daniel Kurtz
On Tue, Jul 7, 2015 at 9:07 PM, Sascha Hauer wrote: > On Thu, Jun 18, 2015 at 01:29:11PM +0800, Eddie Huang wrote: >> Add clk_null, which represents clocks that can not / need not >> controlled by software. >> There are many clocks' parent set to clk_null. >> >> Signed-off-by: James Liao >>

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-07 Thread Sascha Hauer
On Thu, Jun 18, 2015 at 01:29:11PM +0800, Eddie Huang wrote: > Add clk_null, which represents clocks that can not / need not > controlled by software. > There are many clocks' parent set to clk_null. > > Signed-off-by: James Liao > Signed-off-by: Eddie Huang > --- > Base on 4.1-rc1 > >

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-07 Thread Daniel Kurtz
On Tue, Jul 7, 2015 at 10:36 PM, Sascha Hauer s.ha...@pengutronix.de wrote: On Tue, Jul 07, 2015 at 10:15:29PM +0800, Daniel Kurtz wrote: On Tue, Jul 7, 2015 at 9:07 PM, Sascha Hauer s.ha...@pengutronix.de wrote: On Thu, Jun 18, 2015 at 01:29:11PM +0800, Eddie Huang wrote: Add clk_null,

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-07 Thread Eddie Huang
On Tue, 2015-07-07 at 23:10 +0800, Daniel Kurtz wrote: On Tue, Jul 7, 2015 at 10:36 PM, Sascha Hauer s.ha...@pengutronix.de wrote: On Tue, Jul 07, 2015 at 10:15:29PM +0800, Daniel Kurtz wrote: On Tue, Jul 7, 2015 at 9:07 PM, Sascha Hauer s.ha...@pengutronix.de wrote: On Thu, Jun 18,

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-07 Thread Daniel Kurtz
On Tue, Jul 7, 2015 at 9:07 PM, Sascha Hauer s.ha...@pengutronix.de wrote: On Thu, Jun 18, 2015 at 01:29:11PM +0800, Eddie Huang wrote: Add clk_null, which represents clocks that can not / need not controlled by software. There are many clocks' parent set to clk_null. Signed-off-by: James

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-07 Thread Sascha Hauer
On Thu, Jun 18, 2015 at 01:29:11PM +0800, Eddie Huang wrote: Add clk_null, which represents clocks that can not / need not controlled by software. There are many clocks' parent set to clk_null. Signed-off-by: James Liao jamesjj.l...@mediatek.com Signed-off-by: Eddie Huang

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-07 Thread Sascha Hauer
On Wed, Jul 08, 2015 at 10:37:21AM +0800, Eddie Huang wrote: On Tue, 2015-07-07 at 23:10 +0800, Daniel Kurtz wrote: On Tue, Jul 7, 2015 at 10:36 PM, Sascha Hauer s.ha...@pengutronix.de wrote: On Tue, Jul 07, 2015 at 10:15:29PM +0800, Daniel Kurtz wrote: On Tue, Jul 7, 2015 at 9:07 PM,

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-07 Thread Sascha Hauer
On Tue, Jul 07, 2015 at 10:15:29PM +0800, Daniel Kurtz wrote: On Tue, Jul 7, 2015 at 9:07 PM, Sascha Hauer s.ha...@pengutronix.de wrote: On Thu, Jun 18, 2015 at 01:29:11PM +0800, Eddie Huang wrote: Add clk_null, which represents clocks that can not / need not controlled by software. There

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-03 Thread Heiko Stübner
Am Freitag, 3. Juli 2015, 15:45:12 schrieb James Liao: > On Thu, 2015-07-02 at 12:23 +0800, Daniel Kurtz wrote: > > On Thu, Jul 2, 2015 at 11:06 AM, James Liao wrote: > > > These clocks such as clkph_mck_o are configured by other modules before > > > kernel init, and their rates may different

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-03 Thread James Liao
On Thu, 2015-07-02 at 12:23 +0800, Daniel Kurtz wrote: > On Thu, Jul 2, 2015 at 11:06 AM, James Liao wrote: > > These clocks such as clkph_mck_o are configured by other modules before > > kernel init, and their rates may different among platforms. > > What other modules? > Do you mean the rates

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-03 Thread Heiko Stübner
Am Freitag, 3. Juli 2015, 15:45:12 schrieb James Liao: On Thu, 2015-07-02 at 12:23 +0800, Daniel Kurtz wrote: On Thu, Jul 2, 2015 at 11:06 AM, James Liao jamesjj.l...@mediatek.com wrote: These clocks such as clkph_mck_o are configured by other modules before kernel init, and their rates

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-03 Thread James Liao
On Thu, 2015-07-02 at 12:23 +0800, Daniel Kurtz wrote: On Thu, Jul 2, 2015 at 11:06 AM, James Liao jamesjj.l...@mediatek.com wrote: These clocks such as clkph_mck_o are configured by other modules before kernel init, and their rates may different among platforms. What other modules? Do

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-01 Thread Daniel Kurtz
On Thu, Jul 2, 2015 at 11:06 AM, James Liao wrote: > Hi Daniel, > > On Wed, 2015-07-01 at 19:54 +0800, Daniel Kurtz wrote: >> On Wed, Jul 1, 2015 at 2:49 PM, Sascha Hauer wrote: >> > The problem is not that you use fixed clocks for non software >> > controllable clocks of unknwown rates, but

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-01 Thread James Liao
Hi Daniel, On Wed, 2015-07-01 at 19:54 +0800, Daniel Kurtz wrote: > On Wed, Jul 1, 2015 at 2:49 PM, Sascha Hauer wrote: > > The problem is not that you use fixed clocks for non software > > controllable clocks of unknwown rates, but that you try to use a single > > clock for all of them and name

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-01 Thread James Liao
Hi Sascha, On Wed, 2015-07-01 at 08:49 +0200, Sascha Hauer wrote: > On Tue, Jun 30, 2015 at 05:07:09PM +0800, James Liao wrote: > > There are 4 clocks which are derived from clk_null directly in current > > topckgen implementation: > > > > clkph_mck_o, dpi_ck, usb_syspll_125m, hdmitx_dig_cts

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-01 Thread Daniel Kurtz
On Wed, Jul 1, 2015 at 2:49 PM, Sascha Hauer wrote: > > On Tue, Jun 30, 2015 at 05:07:09PM +0800, James Liao wrote: > > Hi Heiko, > > > > There are 4 clocks which are derived from clk_null directly in current > > topckgen implementation: > > > > clkph_mck_o, dpi_ck, usb_syspll_125m,

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-01 Thread Sascha Hauer
On Tue, Jun 30, 2015 at 05:07:09PM +0800, James Liao wrote: > Hi Heiko, > > There are 4 clocks which are derived from clk_null directly in current > topckgen implementation: > > clkph_mck_o, dpi_ck, usb_syspll_125m, hdmitx_dig_cts > > Our designer mentioned 2 things about external clocks:

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-01 Thread Daniel Kurtz
On Wed, Jul 1, 2015 at 2:49 PM, Sascha Hauer s.ha...@pengutronix.de wrote: On Tue, Jun 30, 2015 at 05:07:09PM +0800, James Liao wrote: Hi Heiko, There are 4 clocks which are derived from clk_null directly in current topckgen implementation: clkph_mck_o, dpi_ck, usb_syspll_125m,

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-01 Thread James Liao
Hi Daniel, On Wed, 2015-07-01 at 19:54 +0800, Daniel Kurtz wrote: On Wed, Jul 1, 2015 at 2:49 PM, Sascha Hauer s.ha...@pengutronix.de wrote: The problem is not that you use fixed clocks for non software controllable clocks of unknwown rates, but that you try to use a single clock for all

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-01 Thread Daniel Kurtz
On Thu, Jul 2, 2015 at 11:06 AM, James Liao jamesjj.l...@mediatek.com wrote: Hi Daniel, On Wed, 2015-07-01 at 19:54 +0800, Daniel Kurtz wrote: On Wed, Jul 1, 2015 at 2:49 PM, Sascha Hauer s.ha...@pengutronix.de wrote: The problem is not that you use fixed clocks for non software

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-01 Thread James Liao
Hi Sascha, On Wed, 2015-07-01 at 08:49 +0200, Sascha Hauer wrote: On Tue, Jun 30, 2015 at 05:07:09PM +0800, James Liao wrote: There are 4 clocks which are derived from clk_null directly in current topckgen implementation: clkph_mck_o, dpi_ck, usb_syspll_125m, hdmitx_dig_cts Our

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-07-01 Thread Sascha Hauer
On Tue, Jun 30, 2015 at 05:07:09PM +0800, James Liao wrote: Hi Heiko, There are 4 clocks which are derived from clk_null directly in current topckgen implementation: clkph_mck_o, dpi_ck, usb_syspll_125m, hdmitx_dig_cts Our designer mentioned 2 things about external clocks: 1.

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-06-30 Thread James Liao
On Wed, 2015-06-24 at 12:24 +0200, Heiko Stübner wrote: > Am Mittwoch, 24. Juni 2015, 15:54:15 schrieb James Liao: > > On Mon, 2015-06-22 at 14:53 +0200, Heiko Stübner wrote: > > > Am Montag, 22. Juni 2015, 11:38:37 schrieb James Liao: > > > > On Fri, 2015-06-19 at 13:36 +0200, Heiko Stuebner

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-06-30 Thread James Liao
On Wed, 2015-06-24 at 12:24 +0200, Heiko Stübner wrote: Am Mittwoch, 24. Juni 2015, 15:54:15 schrieb James Liao: On Mon, 2015-06-22 at 14:53 +0200, Heiko Stübner wrote: Am Montag, 22. Juni 2015, 11:38:37 schrieb James Liao: On Fri, 2015-06-19 at 13:36 +0200, Heiko Stuebner wrote:

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-06-24 Thread Heiko Stübner
Hi James, Am Mittwoch, 24. Juni 2015, 15:54:15 schrieb James Liao: > On Mon, 2015-06-22 at 14:53 +0200, Heiko Stübner wrote: > > Hi James, > > > > Am Montag, 22. Juni 2015, 11:38:37 schrieb James Liao: > > > On Fri, 2015-06-19 at 13:36 +0200, Heiko Stuebner wrote: > > > > > > Some clocks such

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-06-24 Thread James Liao
Hi Heiko, On Mon, 2015-06-22 at 14:53 +0200, Heiko Stübner wrote: > Hi James, > > Am Montag, 22. Juni 2015, 11:38:37 schrieb James Liao: > > On Fri, 2015-06-19 at 13:36 +0200, Heiko Stuebner wrote: > > > > Some clocks such as clkph_mck_o, we don't really care where they come > > from and what

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-06-24 Thread James Liao
Hi Heiko, On Mon, 2015-06-22 at 14:53 +0200, Heiko Stübner wrote: Hi James, Am Montag, 22. Juni 2015, 11:38:37 schrieb James Liao: On Fri, 2015-06-19 at 13:36 +0200, Heiko Stuebner wrote: Some clocks such as clkph_mck_o, we don't really care where they come from and what frequencies

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-06-24 Thread Heiko Stübner
Hi James, Am Mittwoch, 24. Juni 2015, 15:54:15 schrieb James Liao: On Mon, 2015-06-22 at 14:53 +0200, Heiko Stübner wrote: Hi James, Am Montag, 22. Juni 2015, 11:38:37 schrieb James Liao: On Fri, 2015-06-19 at 13:36 +0200, Heiko Stuebner wrote: Some clocks such as clkph_mck_o,

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-06-22 Thread Heiko Stübner
Hi James, Am Montag, 22. Juni 2015, 11:38:37 schrieb James Liao: > On Fri, 2015-06-19 at 13:36 +0200, Heiko Stuebner wrote: > > Am Donnerstag, 18. Juni 2015, 18:15:03 schrieb Heiko Stuebner: > > > Am Donnerstag, 18. Juni 2015, 13:29:11 schrieb Eddie Huang: > > > > Add clk_null, which represents

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-06-22 Thread Heiko Stübner
Hi James, Am Montag, 22. Juni 2015, 11:38:37 schrieb James Liao: On Fri, 2015-06-19 at 13:36 +0200, Heiko Stuebner wrote: Am Donnerstag, 18. Juni 2015, 18:15:03 schrieb Heiko Stuebner: Am Donnerstag, 18. Juni 2015, 13:29:11 schrieb Eddie Huang: Add clk_null, which represents clocks that

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-06-21 Thread James Liao
Hi Heiko, On Fri, 2015-06-19 at 13:36 +0200, Heiko Stuebner wrote: > Am Donnerstag, 18. Juni 2015, 18:15:03 schrieb Heiko Stuebner: > > Am Donnerstag, 18. Juni 2015, 13:29:11 schrieb Eddie Huang: > > > Add clk_null, which represents clocks that can not / need not > > > controlled by software. > >

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-06-21 Thread James Liao
Hi Heiko, On Fri, 2015-06-19 at 13:36 +0200, Heiko Stuebner wrote: Am Donnerstag, 18. Juni 2015, 18:15:03 schrieb Heiko Stuebner: Am Donnerstag, 18. Juni 2015, 13:29:11 schrieb Eddie Huang: Add clk_null, which represents clocks that can not / need not controlled by software. There are

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-06-19 Thread Heiko Stuebner
Am Donnerstag, 18. Juni 2015, 18:15:03 schrieb Heiko Stuebner: > Am Donnerstag, 18. Juni 2015, 13:29:11 schrieb Eddie Huang: > > Add clk_null, which represents clocks that can not / need not > > controlled by software. > > There are many clocks' parent set to clk_null. > > Devicetree is supposed

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-06-19 Thread Heiko Stuebner
Am Donnerstag, 18. Juni 2015, 18:15:03 schrieb Heiko Stuebner: Am Donnerstag, 18. Juni 2015, 13:29:11 schrieb Eddie Huang: Add clk_null, which represents clocks that can not / need not controlled by software. There are many clocks' parent set to clk_null. Devicetree is supposed to

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-06-18 Thread Heiko Stuebner
Am Donnerstag, 18. Juni 2015, 13:29:11 schrieb Eddie Huang: > Add clk_null, which represents clocks that can not / need not > controlled by software. > There are many clocks' parent set to clk_null. Devicetree is supposed to describe hardware, and ideally not what software does with it. If the

Re: [PATCH] arm64: dts: mt8173: add clock_null

2015-06-18 Thread Heiko Stuebner
Am Donnerstag, 18. Juni 2015, 13:29:11 schrieb Eddie Huang: Add clk_null, which represents clocks that can not / need not controlled by software. There are many clocks' parent set to clk_null. Devicetree is supposed to describe hardware, and ideally not what software does with it. If the

[PATCH] arm64: dts: mt8173: add clock_null

2015-06-17 Thread Eddie Huang
Add clk_null, which represents clocks that can not / need not controlled by software. There are many clocks' parent set to clk_null. Signed-off-by: James Liao Signed-off-by: Eddie Huang --- Base on 4.1-rc1 Change-Id: I4db9b40d07e28f54f7bae9b676316cbd6a962124 ---

[PATCH] arm64: dts: mt8173: add clock_null

2015-06-17 Thread Eddie Huang
Add clk_null, which represents clocks that can not / need not controlled by software. There are many clocks' parent set to clk_null. Signed-off-by: James Liao jamesjj.l...@mediatek.com Signed-off-by: Eddie Huang eddie.hu...@mediatek.com --- Base on 4.1-rc1 Change-Id: