Le 05/07/2010 13:03, Prafulla Wadaskar a écrit : > >> -----Original Message----- >> From: u-boot-boun...@lists.denx.de >> [mailto:u-boot-boun...@lists.denx.de] On Behalf Of Albert Aribaud >> Sent: Friday, July 02, 2010 10:23 PM >> To: u-boot@lists.denx.de >> Subject: [U-Boot] [PATCH 2/4] egiga: Remove functional >> dependency on kirkwood >> >> Set DRAM windows by using gd as other drivers do, >> instead of calling kirkwood-specific functions. >> >> Signed-off-by: Albert Aribaud<albert.arib...@free.fr> >> --- >> drivers/net/egiga.c | 6 ++++-- >> 1 files changed, 4 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/net/egiga.c b/drivers/net/egiga.c >> index 50e8ff3..cbe4748 100644 >> --- a/drivers/net/egiga.c >> +++ b/drivers/net/egiga.c >> @@ -38,6 +38,8 @@ >> #include<asm/arch/kirkwood.h> >> #include "egiga.h" >> >> +DECLARE_GLOBAL_DATA_PTR; >> + >> #define KIRKWOOD_PHY_ADR_REQUEST 0xee >> #define KWGBE_SMI_REG (((struct kwgbe_registers >> *)KW_EGIGA0_BASE)->smi) >> >> @@ -246,8 +248,8 @@ static void set_dram_access(struct >> kwgbe_registers *regs) >> win_param.access_ctrl = EWIN_ACCESS_FULL; >> win_param.high_addr = 0; >> /* Get bank base */ >> - win_param.base_addr = kw_sdram_bar(i); >> - win_param.size = kw_sdram_bs(i); /* Get >> bank size */ >> + win_param.base_addr = gd->bd->bi_dram[i].start; >> + win_param.size = gd->bd->bi_dram[i].size; >> if (win_param.size == 0) >> win_param.enable = 0; >> else > > this is good patch indeed to make the driver more generic, which is > independent of this activity > please post this patch for current version i.e. kirkwood_egiga.c > > Regards.. > Prafulla . .
There is also a similar fix (move from kw_sdram_{bar,bs} to gd) to be done on drivers/usb/host/ehci-kirkwood.c, apparently. As I wasn't planning supporting USB with this patchset, I'd ignored it, but if you want I can submit a separate patch for it too. Amicalement, -- Albert. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot