[U-Boot] [PATCH v7 4/4] Kirkwood: add lschlv2 and lsxhl board support

[Michael Walle]

Hi Luka,

On Fri, June 1, 2012 01:07, Luka Perkov wrote:
>> +#ifdef CONFIG_RESET_PHY_R
>> +/* Configure and enable MV88E1118 PHY */
>> +void reset_phy(void)
>> +{
>> +	u16 devadr;
>> +	char *name = "egiga1";
>> +
>> +	if (miiphy_set_current_dev(name))
>> +		return;
>> +
>> +	/* command to read PHY dev address */
>> +	if (miiphy_read(name, 0xEE, 0xEE, (u16 *) &devadr)) {
>> +		printf("Err..%s could not read PHY dev address\n", __func__);
>> +		return;
>> +	}
>> +
>> +	/* reset the phy */
>> +	miiphy_reset(name, devadr);
>> +}
>> +#endif /* CONFIG_RESET_PHY_R */
>
> Can you please test without this part if your network will work?

Could you provide some more background why this should be superfluous? Eg.
what happens if an operating system changes some phy settings and reboots
the system?

-- 
michael