[U-Boot] [PATCH v7 3/7] mips: ath79: add support for QCA953x SOCs
Marek Vasut
marex at denx.de
Fri Jan 22 15:49:23 CET 2016
On Friday, January 22, 2016 at 10:17:25 AM, Wills Wang wrote:
> On Sunday, January 17, 2016 03:33 AM, Marek Vasut wrote:
> > On Saturday, January 16, 2016 at 07:13:49 PM, Wills Wang wrote:
> >> This patch enable work for qca953x SOC.
> >>
> >> Signed-off-by: Wills Wang <wills.wang at live.com>
> >> ---
> >>
> >> Changes in v7:
> >> - Use CKSEGxADDR instead of KSEGxADDR for qca953x
> >>
> >> Changes in v6: None
> >> Changes in v5: None
> >> Changes in v4: None
> >> Changes in v3: None
> >> Changes in v2: None
> >
> > [...]
> >
> >> +int get_serial_clock(void)
> >> +{
> >> + return qca953x_get_xtal();
> >> +}
> >> +
> >> +int get_clocks(void)
> >> +{
> >> + void __iomem *regs;
> >> + u32 val, ctrl, xtal, pll, div;
> >
> > This looks like a copy of the same code in patch 2/7 .
>
> There are different in detail.
>
> >> + regs = map_physmem(AR71XX_PLL_BASE, AR71XX_PLL_SIZE,
> >> + MAP_NOCACHE);
> >> +
> >> + xtal = qca953x_get_xtal();
> >> + ctrl = readl(regs + QCA953X_PLL_CLK_CTRL_REG);
> >> + val = readl(regs + QCA953X_PLL_CPU_CONFIG_REG);
> >
> > [...]
> >
> >> +void ddr_init(void)
> >> +{
> >> + void __iomem *regs;
> >> + u32 val;
> >
> > This looks like a copy of the same code in patch 2/7 .
>
> There are different in detail, the bit fields of register are not identical
> copies of one another.
I ran a brief diff, it seems the register file differences can be easily
handled. The differences in the code are not that great afterward.
> there is no way to extract a common code.
>
> >> + regs = map_physmem(AR71XX_DDR_CTRL_BASE, AR71XX_DDR_CTRL_SIZE,
> >> + MAP_NOCACHE);
> >> + val = get_bootstrap();
> >> + if (val & QCA953X_BOOTSTRAP_DDR1) {
> >> + writel(DDR_CTL_CONFIG_VAL, regs + QCA953X_DDR_REG_CTL_CONF);
> >> + udelay(10);
> >> +
> >> + /* For 16-bit DDR */
> >> + writel(0xffff, regs + AR71XX_DDR_REG_RD_CYCLE);
> >> + udelay(100);
> >> +
> >> + /* Burst size */
> >> + writel(DDR_BURST_VAL, regs + QCA953X_DDR_REG_BURST);
> >> + udelay(100);
> >> + writel(DDR_BURST2_VAL, regs + QCA953X_DDR_REG_BURST2);
> >> + udelay(100);
> >> +
> >> + /* AHB maximum timeout */
> >> + writel(0xfffff, regs + QCA953X_DDR_REG_TIMEOUT_MAX);
> >> + udelay(100);
> >> +
> >> + /* DRAM timing */
> >> + writel(DDR1_CONF_REG_VAL, regs + AR71XX_DDR_REG_CONFIG);
> >> + udelay(100);
> >> + writel(DDR1_CONF2_REG_VAL, regs + AR71XX_DDR_REG_CONFIG2);
> >> + udelay(100);
> >> + writel(DDR1_CONF3_REG_VAL, regs + QCA953X_DDR_REG_CONFIG3);
> >> + udelay(100);
> >
> > [...]
More information about the U-Boot
mailing list