[U-Boot] [PATCH 04/10] arm: socfpga: arria10: Added clock manager and pin mux compat macro

[Chee, Tien Fong]

On Rab, 2016-12-07 at 14:54 +0100, Marek Vasut wrote:
> On 12/07/2016 11:48 AM, Chee, Tien Fong wrote:
> > 
> > On Sel, 2016-12-06 at 13:49 +0100, Marek Vasut wrote:
> > > 
> > > On 12/06/2016 08:52 AM, Chee Tien Fong wrote:
> > > > 
> > > > 
> > > > From: Tien Fong Chee <tien.fong.chee at intel.com>
> > > > 
> > > > These compat macros would be used by clock manager and pin mux
> > > > drivers
> > > > to look the required HW info from DTS for hardware
> > > > initialization.
> > > > 
> > > > Signed-off-by: Tien Fong Chee <tien.fong.chee at intel.com>
> > > > Cc: Marek Vasut <marex at denx.de>
> > > > Cc: Dinh Nguyen <dinguyen at kernel.org>
> > > > Cc: Chin Liang See <chin.liang.see at intel.com>
> > > > Cc: Tien Fong <skywindctf at gmail.com>
> > > > ---
> > > >  include/fdtdec.h |    8 ++++++++
> > > >  lib/fdtdec.c     |    2 ++
> > > >  2 files changed, 10 insertions(+), 0 deletions(-)
> > > > 
> > > > diff --git a/include/fdtdec.h b/include/fdtdec.h
> > > > index 27887c8..68cb199 100644
> > > > --- a/include/fdtdec.h
> > > > +++ b/include/fdtdec.h
> > > > @@ -155,6 +155,14 @@ enum fdt_compat_id {
> > > >  	COMPAT_INTEL_BAYTRAIL_FSP_MDP,	/* Intel FSP
> > > > memory-
> > > > down params */
> > > >  	COMPAT_INTEL_IVYBRIDGE_FSP,	/* Intel Ivy Bridge
> > > > FSP
> > > > */
> > > >  	COMPAT_SUNXI_NAND,		/* SUNXI NAND
> > > > controller
> > > > */
> > > > +	COMPAT_ALTERA_SOCFPGA_CLK,	/* SoCFPGA Clock
> > > > initialization */
> > > > +	COMPAT_ALTERA_SOCFPGA_PINCTRL_SINGLE,	/*
> > > > pinctrl-
> > > > single */
> > > > +	COMPAT_ALTERA_SOCFPGA_H2F_BRG,		/*
> > > > Arria10
> > > > hps2fpga bridge */
> > > > +	COMPAT_ALTERA_SOCFPGA_LWH2F_BRG,	/* Arria10
> > > > lwhps2fpga bridge */
> > > > +	COMPAT_ALTERA_SOCFPGA_F2H_BRG,		/*
> > > > Arria10
> > > > fpga2hps bridge */
> > > > +	COMPAT_ALTERA_SOCFPGA_F2SDR0,		/*
> > > > Arria10
> > > > fpga2SDRAM0 bridge */
> > > > +	COMPAT_ALTERA_SOCFPGA_F2SDR1,		/*
> > > > Arria10
> > > > fpga2SDRAM1 bridge */
> > > > +	COMPAT_ALTERA_SOCFPGA_F2SDR2,		/*
> > > > Arria10
> > > > fpga2SDRAM2 bridge */
> > > Is all of this needed ? You're only adding two entries in the
> > > FDTDEC
> > > below.
> > > 
> > This is to avoid compilation error, we have some functions ported
> > from
> > our internal branch, which using above COMPAT macro. Soon, in
> > upcoming
> > patches, we will need those functions and more entries will be
> > added
> > into FDTDEC below.
> You can add the compat strings when you really need them. Still, with
> DM, you shouldn't even need them AFAIK.
> 
We have some drivers in these series of patches contain some COMPAT
strings, without these compact strings, the compilation would fail due
to error compact string is not defined. I think having compact string
would giving us flexbility to put our nodes where we want without
worrying to break our existing codes?
> > 
> > > 
> > > > 
> > > > 
> > > >  
> > > >  	COMPAT_COUNT,
> > > >  };
> > > > diff --git a/lib/fdtdec.c b/lib/fdtdec.c
> > > > index 4defb90..09a1db4 100644
> > > > --- a/lib/fdtdec.c
> > > > +++ b/lib/fdtdec.c
> > > > @@ -66,6 +66,8 @@ static const char * const
> > > > compat_names[COMPAT_COUNT] = {
> > > >  	COMPAT(INTEL_BAYTRAIL_FSP_MDP, "intel,baytrail-fsp-
> > > > mdp"),
> > > >  	COMPAT(INTEL_IVYBRIDGE_FSP, "intel,ivybridge-fsp"),
> > > >  	COMPAT(COMPAT_SUNXI_NAND, "allwinner,sun4i-a10-nand"),
> > > > +	COMPAT(ALTERA_SOCFPGA_CLK, "altr,clk-mgr"),
> > > > +	COMPAT(ALTERA_SOCFPGA_PINCTRL_SINGLE, "pinctrl-
> > > > single"),
> > > >  };
> > > >  
> > > >  const char *fdtdec_get_compatible(enum fdt_compat_id id)
> > > > 
>