On 10/12/26 2:10 PM, Jerome Brunet wrote:
This series adds support for the clock controller present within
the Toshiba TC9564 SoC.
This SoC attaches to a host system via PCIe. It includes a PCIe
switch with three downstream ports. To one of these ports an
embedded PCIe endpoint is attached, and it implements two PCI
functions. BAR 4 for both PCI functions provides access to the
registers that enable or disable a set of clocks.
Both TC9564 PCI functions can provide PCI endpoint buses via
devicetree. Each PCI endpoint bus provides access to one of the
function's BARs. Subordinate devices (like the clock controller)
can then be defined as nodes accessible on the endpoint bus.
Only one of the two PCI functions is permitted to define the clock
device node.
-Alex
NOTE:
This series is built upon linux-next, with this patch applied:
https://lore.kernel.org/lkml/[email protected]/
next AND a patch on top ?? you are not making this easy and this is mostly
to accommodate the DT change that could have been sent separately.
Yes. I considered dropping the DT change for this reason, but
on v2 I didn't say I would do this, and thought it would be best
to not change this.
Here I'll be clear: I will send v4, and in v4, I will *not*
include the DTS file patch at the end.
Basing on linux-next means there are other deps and you just did not explain.
This makes it very difficult for me to apply your series if I wanted to.
OK.
Next time around, please send a series on the clk list with only the 2 first
change. That way we do not have to cherry pick, we can use your cover letter
for the merge commit, which makes it easier to provide an immutable branch
to another subsystem, if need be.
I understand. Thanks for your feedback.
-Alex
*Clearly* list you dependencies if any. Please consider using b4 to list
the pre-requisite.
Thanks