Re: [PATCH v2 0/2] Remove MT6779 UART3 clock support

[Matthias Brugger]

On 22/07/2020 13:16, Hanks Chen wrote:
> On Wed, 2020-07-22 at 10:43 +0200, Matthias Brugger wrote:
> > On 22/07/2020 05:09, Hanks Chen wrote:
> > > remove the redundant clk interface of uart.
> > > CLK_INFRA_UART3 is a dummy clk interface,
> > > it has no effect on the operation of the read/write instruction.
> > >
> > > Change since v2:
> > > Commit "dt-bindings: clock: remove UART3 clock support"
> >
> > Sorry just another comment. I think we can make this one patch deleting everything.
> Sorry, I don't understand the comment. Do I need to merge them into one
> big patch?
> Btw, if I use one patch, I would get check patch fail.
> ---
> [2020-07-22 19:12:49,046 ERROR] Run check patch results:
> WARNING:DT_SPLIT_BINDING_PATCH: DT binding docs and includes should be a
> separate patch. See:
> Documentation/devicetree/bindings/submitting-patches.rst
> ---

Hm, I think that's a debatable error message from checkpatch, but let's leave it 
as it is.

Regards,
Matthias


> Hanks
>
> > > -- remove Fixes tag
> > > Commit "clk: mediatek: remove UART3 clock support"
> > > -- remove Fixes tag
> > >
> > > Hanks Chen (2):
> > >     dt-bindings: clock: remove UART3 clock support
> > >     clk: mediatek: remove UART3 clock support
> > >
> > >    drivers/clk/mediatek/clk-mt6779.c      | 2 --
> > >    include/dt-bindings/clock/mt6779-clk.h | 1 -
> > >    2 files changed, 3 deletions(-)