Hi Luigi,
Sorry, I'm going to have to ask you to re-submit again :( There are a
few things that need to be addressed
Luigi 'Comio' Mantellini wrote:
> This patch rewrites the miiphybb ( Bit-banged MII bus driver ) in order to
> support an arbitrary number of mii buses. This feature is useful when your
> board uses different mii buses for different phys and all (or a part) of these
> buses are implemented via bit-banging mode.
>
> The driver requires that the following macros should be defined into the board
> configuration file:
>
> CONFIG_BITBANGMII - Enable the miiphybb driver
> CONFIG_BITBANGMII_MULTI - Enable the multi bus support
>
> If the CONFIG_BITBANGMII_MULTI is not defined, the board's config file needs
> to define at least the following macros:
>
> MII_INIT - Generic code to enable the MII bus (optional)
> MDIO_DECLARE - Declaration needed to access to the MDIO pin (optional)
> MDIO_ACTIVE - Activate the MDIO pin as out pin
> MDIO_TRISTATE - Activate the MDIO pin as input/tristate pin
> MDIO_READ - Read the MDIO pin
> MDIO(v) - Write v on the MDIO pin
> MDC_DECLARE - Declaration needed to access to the MDC pin (optional)
> MDC(v) - Write v on the MDC pin
>
> The previous macros make the driver compatible with the previous version
> (that didn't support the multi-bus).
>
> When the CONFIG_BITBANGMII_MULTI is also defined, the board code needs to fill
> the bb_miiphy_buses[] array with a record for each required bus and declare
> the bb_miiphy_buses_num variable with the number of mii buses.
> The record (struct bb_miiphy_bus) has the following fields/callbacks (see
> miiphy.h for details):
>
> char name[] - The symbolic name that must be equal to the MII bus
> registered name
> int (*init)() - Initialization function called at startup time (just
> before the Ethernet initialization)
> int (*mdio_active)() - Activate the MDIO pin as output
> int (*mdio_tristate)() - Activate the MDIO pin as input/tristate pin
> int (*set_mdio)() - Write the MDIO pin
> int (*get_mdio)() - Read the MDIO pin
> int (*set_mdc)() - Write the MDC pin
> int (*delay)() - Delay function
> void *priv - Private data used by board specific code
>
> The board code will look like:
>
> struct bb_miiphy_bus bb_miiphy_buses[] = {
> { .name = "miibus#1", .init = b1_init, .mdio_active = b1_mdio_active, ... },
> { .name = "miibus#2", .init = b2_init, .mdio_active = b2_mdio_active, ... },
> ...
> };
> int bb_miiphy_buses_num = sizeof(bb_miiphy_buses) /
> sizeof(bb_miiphy_buses[0]);
>
>
Can you please copy this stuff into a README file (doc/README.bitbangMII
maybe)? I'm afraid all this wonderful changelog information will be in
a place that people won't look.
If it wasn't for other problems with this patch set, I'd do it myself.
> Patch Changelog:
>
> v1 -- First (broken) release
> v2 -- Fix some typos and disable callbacks pointers relacation (if
> CONFIG_RELOC_FIXUP_WORKS is not defined)
> v3 -- Do not relocate NULL pointers
> v4 -- Code Cleanup.
>
> Luigi 'Comio' Mantellini (3):
> New Bit-banged MII driver (MIIPHYBB) implementation with multi-bus
> support.
> Add bb_miiphy_init call before any ethernet bring-up code.
> Update all board to support new bbmiiphy driver (with multibus
> support)
>
> drivers/net/phy/miiphybb.c | 346
> +++++++++++++++++++++++++++++-------------
> include/configs/ISPAN.h | 4 +
> include/configs/MPC8260ADS.h | 3 +
> include/configs/MPC8266ADS.h | 4 +
> include/configs/MPC8560ADS.h | 4 +
> include/configs/Rattler.h | 4 +
> include/configs/SBC8540.h | 4 +
> include/configs/TQM8272.h | 3 +
> include/configs/VoVPN-GW.h | 4 +
> include/configs/ZPC1900.h | 4 +
> include/configs/ep8248.h | 4 +
> include/configs/ep82xxm.h | 4 +
> include/configs/gw8260.h | 4 +
> include/configs/hymod.h | 12 ++
> include/configs/muas3001.h | 4 +
> include/configs/ppmc8260.h | 4 +
> include/configs/sacsng.h | 4 +
> include/configs/sbc8260.h | 4 +
> include/configs/sbc8560.h | 4 +
> include/miiphy.h | 25 +++-
> lib_arm/board.c | 7 +
> lib_avr32/board.c | 7 +
> lib_blackfin/board.c | 7 +
> lib_i386/board.c | 9 +-
> lib_m68k/board.c | 7 +
> lib_mips/board.c | 7 +
> lib_ppc/board.c | 7 +
> lib_sh/board.c | 7 +
> lib_sparc/board.c | 7 +
> 29 files changed, 408 insertions(+), 106 deletions(-)
>
> _______
Here are the real problems:
bwar...@bwarren-bldsrv$ CROSS_COMPILE=ppc_6xx- ./MAKEALL ppc
...
Configuring for ep8248 board...
board.c:87:20: error: miihpy.h: No such file or directory
board.c:87:20: error: miihpy.h: No such file or directory
board.c: In function 'board_init_r':
board.c:950: warning: implicit declaration of function 'bb_miiphy_init'
make[1]: *** [board.o] Error 1
make: *** [lib_ppc/libppc.a] Error 2
And all sorts of other PPC boards fail too. Looks like a typo.
regards,
Ben
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
