[U-Boot] [PATCH v5 3/8] clk: sifive: Sync-up DT bindings header with upstream Linux
Anup Patel
anup at brainfault.org
Fri Jun 21 11:36:37 UTC 2019
On Fri, Jun 21, 2019 at 4:54 PM Anup Patel <anup at brainfault.org> wrote:
>
> On Fri, Jun 21, 2019 at 12:46 PM Bin Meng <bmeng.cn at gmail.com> wrote:
> >
> > On Thu, Jun 20, 2019 at 2:49 PM Anup Patel <Anup.Patel at wdc.com> wrote:
> > >
> > > The location and license header of DT bindings header for SiFive
> > > clock driver has changed in upstream Linux hence this patch.
> > >
> > > Signed-off-by: Anup Patel <anup.patel at wdc.com>
> > > ---
> > > drivers/clk/sifive/fu540-prci.c | 2 +-
> > > include/dt-bindings/clk/sifive-fu540-prci.h | 29 -------------------
> > > include/dt-bindings/clock/sifive-fu540-prci.h | 18 ++++++++++++
> >
> > Shouldn't this be a "git mv"?
>
> There are changes in comment block and #ifndef so GIT did
> not consider this as renaming.
>
> >
> > > 3 files changed, 19 insertions(+), 30 deletions(-)
> > > delete mode 100644 include/dt-bindings/clk/sifive-fu540-prci.h
> > > create mode 100644 include/dt-bindings/clock/sifive-fu540-prci.h
> > >
> > > diff --git a/drivers/clk/sifive/fu540-prci.c b/drivers/clk/sifive/fu540-prci.c
> > > index cdbf35e871..ceb318e062 100644
> > > --- a/drivers/clk/sifive/fu540-prci.c
> > > +++ b/drivers/clk/sifive/fu540-prci.c
> > > @@ -38,7 +38,7 @@
> > >
> > > #include <linux/math64.h>
> > > #include <linux/clk/analogbits-wrpll-cln28hpc.h>
> > > -#include <dt-bindings/clk/sifive-fu540-prci.h>
> >
> > What about the existing U-Boot's <dt-bindings/clk/..> directory? With
> > this commit, we introduced 2 clock device tree binding directories.
>
> Sure, I will remove the directory as well. Thanks for catching.
I just realized that dt-bindings/clk directory is not empty so I
cross-checked with Linux sources. Over there we have both
dt-bindings/clk and dt-bindings/clock directories.
For now, I will keep the dt-bindings/clk directory to be in-sync
with Linux sources.
>
> >
> > > +#include <dt-bindings/clock/sifive-fu540-prci.h>
> > >
> > > /*
> > > * EXPECTED_CLK_PARENT_COUNT: how many parent clocks this driver expects:
> > > diff --git a/include/dt-bindings/clk/sifive-fu540-prci.h b/include/dt-bindings/clk/sifive-fu540-prci.h
> > > deleted file mode 100644
> > > index 531523ea62..0000000000
> > > --- a/include/dt-bindings/clk/sifive-fu540-prci.h
> > > +++ /dev/null
> > > @@ -1,29 +0,0 @@
> > > -/* SPDX-License-Identifier: GPL-2.0 */
> > > -/*
> > > - * Copyright (c) 2019 Western Digital Corporation or its affiliates.
> > > - *
> > > - * Copyright (C) 2018 SiFive, Inc.
> > > - * Wesley Terpstra
> > > - * Paul Walmsley
> > > - *
> > > - * This program is free software; you can redistribute it and/or modify
> > > - * it under the terms of the GNU General Public License version 2 as
> > > - * published by the Free Software Foundation.
> > > - *
> > > - * This program is distributed in the hope that it will be useful,
> > > - * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > > - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > > - * GNU General Public License for more details.
> > > - */
> > > -
> > > -#ifndef __LINUX_CLK_SIFIVE_FU540_PRCI_H
> > > -#define __LINUX_CLK_SIFIVE_FU540_PRCI_H
> > > -
> > > -/* Clock indexes for use by Device Tree data */
> > > -
> > > -#define PRCI_CLK_COREPLL 0
> > > -#define PRCI_CLK_DDRPLL 1
> > > -#define PRCI_CLK_GEMGXLPLL 2
> > > -#define PRCI_CLK_TLCLK 3
> > > -
> > > -#endif
> >
>
> Regards,
> Anup
Regards,
Anup
More information about the U-Boot
mailing list