Hi Neil, On Mon, Jan 21, 2019 at 11:40 AM Neil Armstrong <narmstrong@xxxxxxxxxxxx> wrote: > > On 19/01/2019 21:51, Martin Blumenstingl wrote: > > It turns out that I'm not good at spelling "APB". Unfortunately an ABP > > (instead of APB) typo snug into two clocks in v5.0. > > > > Currently there are no users of that clock yet, so fix it now > > before it's too late (meaning before it's used by <something>). > > > > I decided to go the "normal" route where we can have separate > > branches for the dt-bindings and the drivers in the clk-meson > > repo. This means: > > - new (typo-free) clock #define > > - change the clock controller to use the new #define > > - (next development cycle) drop the old #define > > > > > > Changes since v1 at [0]: > > - split into a dt-bindings and clock controller driver patch > > > > > > [0] https://patchwork.kernel.org/cover/10742001/ > > > > > > Martin Blumenstingl (2): > > dt-bindings: clock: meson8b: add APB clock definition > > clk: meson: meson8b: fix the naming of the APB clocks > > > > drivers/clk/meson/meson8b.c | 26 ++++++++++++------------ > > drivers/clk/meson/meson8b.h | 2 +- > > include/dt-bindings/clock/meson8b-clkc.h | 1 + > > 3 files changed, 15 insertions(+), 14 deletions(-) > > > > Once rob acks it, I'll apply both to next/drivers since it's not used (yet ?) > and not a critical fix. I have just noticed that I forgot to CC Rob and devicetree@xxxxxxxxxxxxxxx. do you want me to re-send this series? as you already noticed this is not critical, so taking this through next/drivers is fine for me. Regards Martin
