On 03/29/2017 10:42 PM, Michael Turquette wrote: > Hi Jerome, > > Adding Neil Armstong to Cc. > > Quoting Jerome Brunet (2017-03-28 07:45:57) >> The patchset is the 2nd round of update to the meson gxbb clock controller >> to add initial audio support. The patchset is based on clk-next. >> >> There is not much out of the ordinary here (adding new clocks and exposing >> them) except maybe for 2 patches: >> Patch #2: Adds a safety check while registering clocks to protect against >> holes in clk_hw_onecell_data, if it ever happens. Same thing is >> done for the meson8b clock controller. >> Patch #3: Adds a new clock divider driver to implement the necessary >> policy for the i2s master clock (see patch changelog) >> >> This patchset has been test on the gxbb p200 and gxl p230. > > First off, this series looks fine to me. Please add, > > Acked-by: Michael Turquette <mturquette@xxxxxxxxxxxx> > > Secondly, it seems the AmLogic clock drivers have mostly calmed down and > things are in the "add new clocks when we need them" phase, which is > nice. Since you and Neil are doing a lot of this work, might I suggest > that you both start collecting patches for the AmLogic/meson clock > drivers begin submitting pull requests to Stephen and myself? > > As usual the rules are the same as always: all patches in the PR must > first be posted for review on the list. PRs should correspond to signed > tags. Stephen and I might ignore PRs sent after -rc4, and will > definitely ignore PRs sent after -rc6 since we want some stabilization > time before the merge window. Base your branch on -rc1, not on clk-next. > > Also feel free to submit a patch to MAINTAINERS with either one or both > of you maintaining the meson clk stuff, assuming that you're OK to > review all of those patches and collect them into a PR. > > Thoughts? Thanks Mike, I'll be glad to co-maintain this with jerome ! Neil > > If it sounds good to you then I suggest grabbing the clk-meson branch > from the clk tree and using that as a baseline for your first PR. In the > future you'll just Linus' -rc1, but I have already created a branch for > this cycle. You can apply these 8 patches with my Ack and send a PR by > -rc6 (possibly with other stuff collected from Martin, etc). Acked ! > > Thanks, > Mike > >> >> Jerome Brunet (8): >> dt-bindings: clock: gxbb: expose spdif clock gates >> clk: meson: gxbb: protect against holes in the onecell_data array >> clk: meson: add audio clock divider support >> clk: meson: gxbb: add cts_amclk >> clk: meson: gxbb: add cts_mclk_i958 >> clk: meson: gxbb: add cts_i958 clock >> dt-bindings: clock: gxbb: expose i2s master clock >> dt-bindings: clock: gxbb: expose spdif master clock >> >> drivers/clk/meson/Makefile | 2 +- >> drivers/clk/meson/clk-audio-divider.c | 149 ++++++++++++++++++++++++++++++++++ >> drivers/clk/meson/clkc.h | 10 +++ >> drivers/clk/meson/gxbb.c | 144 ++++++++++++++++++++++++++++++++ >> drivers/clk/meson/gxbb.h | 13 ++- >> include/dt-bindings/clock/gxbb-clkc.h | 5 ++ >> 6 files changed, 319 insertions(+), 4 deletions(-) >> create mode 100644 drivers/clk/meson/clk-audio-divider.c >> >> -- >> 2.9.3 >> -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html