Thread (24 messages) 24 messages, 3 authors, 2018-12-14

RE: [PATCH V12 0/5] clk: imx: add imx8qxp clock support

From: Stephen Boyd <sboyd@kernel.org>
Date: 2018-12-14 05:44:21
Also in: linux-clk

Quoting Aisheng Dong (2018-12-13 20:57:46)
quoted
-----Original Message-----
From: Shawn Guo [mailto:shawnguo@kernel.org]
On Fri, Dec 14, 2018 at 03:37:47AM +0000, Aisheng Dong wrote:
quoted
quoted
-----Original Message-----
From: Shawn Guo [mailto:shawnguo@kernel.org] On Fri, Dec 14, 2018 at
02:17:04AM +0000, Aisheng Dong wrote:
quoted
[...]
quoted
quoted
Note: It depends on SCU driver which has already been merged
by
Shawn.
quoted
quoted
quoted
So this patch series could go through Shawn's tree as well.
To be clear, I'm not going to take this via my tree, and it
definitely needs to go through clk tree.  If there is a
dependency on my tree, you will likely need to wait for the
dependency to land on
mainline and then retry.
quoted
quoted
Sorry, I should have corrected that comment.
It's not depend on SCU driver which is already in the Stephen's tree.
It actually depends on the resource ID definitions[1] and the
centralized PM related service definitions headifle in in power
domain series which is already in your tree.

[2] can be fixed by defining them in clock driver.
But [1] seems can't be fixed as resource ID is shared by power
domain and possible many other SCU client drivers.

Any suggestion?
After [1] and [2] land on v4.21-rc1, you rebase the series on that.
The dependency will be gone, right?  This is what I suggested above.
Okay, that may need a few more weeks.
BTW, after v4.21-rc1 is out, can the mx8qxp CLK and DTS hit the final v4.21
release?
quoted
As I understand, usually we don't receive new feature patch after the
first RC, right?
Yes, they will have to land on 4.22 then.
That's unfortunately and may need wait 3 more months.
I really wish we can hit v4.21-rc1, then other modules driver can start
their work based on it.

Stephen,
What's your option?

Can we try the same way as Shawn did for arch and dts part?
I see some options:

	1. Shawn can provide the required header files in some signed
	tag that I can pull into clk tree and then apply these patches
	on top and merge it all up into clk-next.

	2. I can pick any patches required for the header files the clk
	driver needs into clk tree and duplicate the commits in Shawn's
	tree. I don't see a big downside here, git manages just fine
	when it sees duplicate content on both sides of a merge so it's
	just more noise than anything else.

	3. I apply the clk driver bits but nobody tries to enable the
	config option around it just yet. Instead, we wait for
	everything to meet up in linux-next via different trees and then
	enable compilation later. This is bad for compile testing
	drivers and makes it harder to bisect problems later. Probably
	not a big deal here for bringing in new hardware support but it
	might work out.

	4. We all wait three months (happy new year!) but otherwise are
	sad that we can't figure this out.


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help