From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.dev.rtsoft.ru (unknown [85.21.88.2]) by ozlabs.org (Postfix) with SMTP id A0DC0685DB for ; Tue, 25 Oct 2005 01:10:09 +1000 (EST) Message-ID: <435CF94E.5060102@ru.mvista.com> Date: Mon, 24 Oct 2005 19:10:06 +0400 From: Vitaly Bordug MIME-Version: 1.0 To: info References: <1130165338$29160$83019474@info@moreframe.com> In-Reply-To: <1130165338$29160$83019474@info@moreframe.com> Content-Type: text/plain; charset=GB2312 Cc: linuxppc-embedded Subject: Re: [PATCH] 85xx PHY Platform Update List-Id: Linux on Embedded PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , info wrote: > Andy, Kumar > > I try this patch for RMC-G8500 from GDAtech, error message below: > --------------------------------------------------------------------------------- > [root@ppclinux linux-2.6.13]# make uImage > CHK include/linux/version.h > CHK include/linux/compile.h > CHK usr/initramfs_list > CC drivers/net/gianfar.o > drivers/net/gianfar.c: In function `gfar_probe': > drivers/net/gianfar.c:217: error: structure has no member named `phy_reg_addr' > drivers/net/gianfar.c: In function `init_phy': > drivers/net/gianfar.c:452: error: structure has no member named `phyid' > drivers/net/gianfar.c: In function `stop_gfar': > drivers/net/gianfar.c:636: error: structure has no member named `interruptPHY' > drivers/net/gianfar.c: In function `gfar_phy_startup_timer': > drivers/net/gianfar.c:1818: error: structure has no member named `interruptPHY' > drivers/net/gianfar.c:1826: error: structure has no member named `interruptPHY' > make[2]: *** [drivers/net/gianfar.o] Error 1 > make[1]: *** [drivers/net] Error 2 > make: *** [drivers] Error 2 > > --------------------------------------------------------------------------------- > because??? This patch implies that PAL (Phy Abstraction layer) is utilized within the gianfar driver, but this change is just queued upstream (AFAIK - or in the netdev tree) right now. In order to use this you should wait a little... > > /* Internal interrupts are all Level Sensitive, and Positive Polarity */ > diff --git a/include/linux/fsl_devices.h b/include/linux/fsl_devices.h > --- a/include/linux/fsl_devices.h > +++ b/include/linux/fsl_devices.h > @@ -47,14 +47,19 @@ > struct gianfar_platform_data { > /* device specific information */ > u32 device_flags; > - u32 phy_reg_addr; > > /* board specific information */ > u32 board_flags; > - u32 phy_flags; > - u32 phyid; > - u32 interruptPHY; > + const char *bus_id; > u8 mac_addr[6]; > +}; > + > > > ------------------------------------------------------------------------------------------- > < > > > < > < > <--- a/arch/ppc/platforms/85xx/mpc8540_ads.c > <+++ b/arch/ppc/platforms/85xx/mpc8540_ads.c > <@@ -53,6 +53,10 @@ > < > < #include > < > <+static const char *GFAR_PHY_0 = "phy0:0"; > <+static const char *GFAR_PHY_1 = "phy0:1"; > <+static const char *GFAR_PHY_3 = "phy0:3"; > <+ > < /* ************************************************************************ > < * > < * Setup the architecture > <@@ -64,6 +68,7 @@ mpc8540ads_setup_arch(void) > > > > _______________________________________________ > Linuxppc-embedded mailing list > Linuxppc-embedded@ozlabs.org > https://ozlabs.org/mailman/listinfo/linuxppc-embedded > > -- Sincerely, Vitaly