[U-Boot] [PATCH 2/2] net: sh_eth: add support for SH7757's ETHER

Yoshihiro Shimoda yoshihiro.shimoda.uh at renesas.com
Tue Jan 18 08:44:52 CET 2011


Hi, Iwamatsu-san,

2011/01/17 19:59, Nobuhiro Iwamatsu wrote:
> Hi, Shimoda-san.
> 
> On Mon, Jan 17, 2011 at 01:09:23PM +0900, Yoshihiro Shimoda wrote:
< snip >
>> +	val = 0x0;
> 
> Is this change necessary?
Oh, this is not necessary.

>>  	/* Check if full duplex mode is supported by the phy */
>>  	if (phy_status & (PHY_S_100X_F|PHY_S_10T_F)) {
>>  		printf("Full\n");
>> -		outl((ECMR_CHG_DM|ECMR_RE|ECMR_TE|ECMR_DM), ECMR(port));
>> +		outl(val | (ECMR_CHG_DM|ECMR_RE|ECMR_TE|ECMR_DM), ECMR(port));
>>  	} else {
>>  		printf("Half\n");
>> -		outl((ECMR_CHG_DM|ECMR_RE|ECMR_TE),  ECMR(port));
>> +		outl(val | (ECMR_CHG_DM|ECMR_RE|ECMR_TE),  ECMR(port));
>>  	}
> 
> Is this change necessary? 
> Because val is changed to 0 at the top, I do not think that this change is necessary.
Thank you for your point.
I will remove it.

Best regards,
Yoshihiro Shimoda


More information about the U-Boot mailing list