[U-Boot] [PATCH v5 08/19] arm: socfpga: Fix CLKMGR_INTOSC_HZ to 400MHz

Ley Foon Tan lftan.linux at gmail.com
Fri Oct 25 08:47:23 UTC 2019


On Wed, Oct 23, 2019 at 2:31 AM Simon Goldschmidt
<simon.k.r.goldschmidt at gmail.com> wrote:
>
> Am 11.10.2019 um 11:52 schrieb Ley Foon Tan:
> > CLKMGR_INTOSC_HZ should be 400MHz, instead of 460MHz.
>
> Having this in a separate patch is clearly better.
>
> >
> > Signed-off-by: Ley Foon Tan <ley.foon.tan at intel.com>
> > ---
> >   arch/arm/mach-socfpga/include/mach/clock_manager_soc64.h | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/mach-socfpga/include/mach/clock_manager_soc64.h b/arch/arm/mach-socfpga/include/mach/clock_manager_soc64.h
> > index 3b4bb62ca5..1a134b89ba 100644
> > --- a/arch/arm/mach-socfpga/include/mach/clock_manager_soc64.h
> > +++ b/arch/arm/mach-socfpga/include/mach/clock_manager_soc64.h
> > @@ -14,7 +14,7 @@ const unsigned int cm_get_intosc_clk_hz(void);
> >   const unsigned int cm_get_fpga_clk_hz(void);
> >
> >   #define CLKMGR_EOSC1_HZ             25000000
> > -#define CLKMGR_INTOSC_HZ     460000000
> > +#define CLKMGR_INTOSC_HZ     400000000
> >   #define CLKMGR_FPGA_CLK_HZ  50000000
>
> What are these defines anyway? CLKMGR_EOSC1_HZ and CLKMGR_FPGA_CLK_HZ
> seem unused? Can they be removed (that comment would probably have been
> better in 8/19...)?
They are unused now. Will remove them.

Thanks.

Regards
Ley Foon
>
> >
> >   /* Clock configuration accessors */
> >
>


More information about the U-Boot mailing list