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 --- 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(-) > >