From mboxrd@z Thu Jan  1 00:00:00 1970
From: Thomas Chou <thomas@wytron.com.tw>
Date: Wed, 07 Apr 2010 11:34:27 +0800
Subject: [U-Boot] [PATCH v2] net: add opencore 10/100 ethernet mac driver
In-Reply-To: <4BB983CB.9030707@gmail.com>
References: <1269411881-8850-1-git-send-email-thomas@wytron.com.tw>
	<1269437683-4550-1-git-send-email-thomas@wytron.com.tw>
	<4BB983CB.9030707@gmail.com>
Message-ID: <4BBBFD43.1000800@wytron.com.tw>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Ben,

Thanks.

On 04/05/2010 02:31 PM, Ben Warren wrote:
> Hi Thomas,
>
>> + */
>> +struct ethoc {
>> +    void *iobase;
> eth_device struct already has this.  If you also want it in the 
> private struct, please don't use void *.
OK. I will use eth_device and remove the private iobase.
>> +
>> +    unsigned int num_tx;
>> +    unsigned int cur_tx;
>> +    unsigned int dty_tx;
>> +
>> +    unsigned int num_rx;
>> +    unsigned int cur_rx;
>> +
>> +    u32 msg_enable;
> Please don't mix types like this.  Using 'u32' and friends globally is 
> preferred.
OK. I will use u32 and friends globally.
>>
>> +
>> +int ethoc_initialize(bd_t *bis, int base_addr)
> You don't use 'bis', so don't pass it in.  I'd prefer to see you pass 
> in the base address and an index in case somebody wants more than one 
> (mainly useful for debugging)
Do you mean adding dev_num as index?

int ethoc_initialize(u8 dev_num, int base_addr)

Best regards,
Thomas