Re: [PATCH v2 0/2] clk: meson8b: fix typo "ABP" -> "APB"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Martin,

Le 09/02/2019 01:24, Martin Blumenstingl a écrit :
> 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?

Yes please re-send !

Neil

> 
> as you already noticed this is not critical, so taking this through
> next/drivers is fine for me.
> 
> 
> Regards
> Martin
> 



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux