Messages in this thread | | | From | Kevin Hilman <> | Subject | Re: [PATCH v8 6/7] clk: hi6220: Clock driver support for Hisilicon hi6220 SoC | Date | Thu, 28 May 2015 10:32:05 -0700 |
| |
Bintian <bintian.wang@huawei.com> writes:
> Hello Mike, > > On 2015/5/28 13:26, Michael Turquette wrote: >> Quoting Bintian Wang (2015-05-23 21:11:11) >>> Add clock drivers for hi6220 SoC, this driver controls the SoC >>> registers to supply different clocks to different IPs in the SoC. >>> >>> We add one divider clock for hi6220 because the divider in hi6220 >>> also has a mask bit but it doesnot obey the rule defined by flag >>> "CLK_DIVIDER_HIWORD_MASK", we can not get index of the mask bit by >>> left shift fixed bits (e.g. 16 bits), so we add this divider clock >>> to handle it. >>> >>> Signed-off-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org> >>> Signed-off-by: Bintian Wang <bintian.wang@huawei.com> >>> Acked-by: Haojian Zhuang <haojian.zhuang@linaro.org> >>> Reviewed-by: Zhangfei Gao <zhangfei.gao@linaro.org> >>> Tested-by: Will Deacon <will.deacon@arm.com> >>> Tested-by: Tyler Baker <tyler.baker@linaro.org> >> >> Hi Bintian, >> >> Thanks for making the changes requested by Stephen. I've taken his patch >> to add assigned-clock-rate/parent support for AMBA interconnects and >> applied it to 4.1-rc1, and then I've applied your v8 patches #4-6 on top >> of that. You can find it at: >> >> git://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git clk-next-hi6220 > Thank you very much! > > I think you also need to pick patch "[PATCH v5 3/6] clk: hi6220: > Document devicetree bindings for hi6220 clock", which described the > dt binding of clk, and it is also acked by Stephen(v4 is the same to > v5). > >> I have merged this into clk-next so it can get some cycles in >> linux-next. >> >> Stephen, >> >> Can you send your patch out to Russell properly? It needs his ack (or >> for him to take it outright) in order to unblock the hi6220 clock driver >> from being merged. > It doesn't block hi6220 clock driver now, because the UART1 is not > enabled in hi6220 dts now.
Now that the clk changes are queued up, can you (re)post the remaining hikey patches with a changelog stating the dependency on the clk-next branch. I believe what's left is just the DT and Kconfig/defconfig changes, correct?
With some acks from the DT maintainers, these should be ready to be merged through arm-soc.
Thanks,
Kevin
| |