From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Mon, 17 Nov 2008 09:13:54 +0100 Subject: [U-Boot] [PATCH 4/5] powerpc: keymile: Add a check for the PIGGY debug board In-Reply-To: <491C644B.3030508@gmail.com> References: <491A98A0.6070107@denx.de> <491B15DF.1070805@gmail.com> <491BDA95.2060306@invitel.hu> <491C644B.3030508@gmail.com> Message-ID: <492127C2.9050906@invitel.hu> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Ben, Ben Warren wrote: > Heiko Schocher wrote: > >> Hello Ben >> >> Ben Warren wrote: >> >> >>> Heiko Schocher wrote: >>> >>> >>> >>>> Check the presence of the PIGGY on the keymile boards mgcoge, >>>> mgsuvd and kmeter1. If the PIGGY is not present, dont register >>>> this Ethernet device. >>>> >>>> Signed-off-by: Heiko Schocher >>>> --- >>>> >>>> >>>> >>> This looks like useful stuff to have, but I'd prefer that you put the >>> check logic in board_eth_init() rather than adding to the individual >>> device drivers. I know the 8260 SCC driver is the older style, which >>> precludes the use of board_eth_init, but I'll convert it if you're able >>> to test. >>> >>> >>> >> Yes, I could test such a change for you, but hmm... I am not sure, if >> board_eth_init () is the right place for my purpose. >> I need for every Ethernet device a selection, if this device is present or >> not. >> Correct me if I am wrong, but it looks like board_eth_init () >> is not made for this purpose. (Ok, I can do a specific device init >> in board_eth_init (), but then we must do something, that prevents >> that the device is again initialized in eth_initialize () ... >> >> >> > board_eth_init() was introduced for exactly this sort of thing. Have a > look at the net repo (I've sent a pull request to Wolfgang so the > current changes will make it into the 12.2008 release). There aren't > any device initializations left in eth_initialize(), so there's no issue > of a device being initialized twice. The goal is for all devices to be > started by cpu_eth_int() or board_eth_init(). > Ahh... now I see it. I adjust my patch, thanks. bye Heiko