<div dir="ltr"><div>Thanks Jonas. Tested and working OK. It would be nice if the patch is also backported to Chaos Calmer revision.</div><div><br></div><div>Regards.</div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-01-18 14:16 GMT+01:00 Jonas Gorski <span dir="ltr"><<a href="mailto:jogo@openwrt.org" target="_blank">jogo@openwrt.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<span class=""><br>
On 12 December 2015 at 14:10, dani <<a href="mailto:dgcbueu@gmail.com">dgcbueu@gmail.com</a>> wrote:<br>
> Currently ephy-reset, which uses a GPIO for enabling external ethernet phys, is broken.<br>
> This patch fix the problem.<br>
><br>
> This problem causes in boards with external phys with a reset pin connected to gpio, are<br>
> initialized without lan interfaces.<br>
><br>
> The line<br>
> ephy_reset.table[0].chip_label = gpio_chip_labels[hw_gpio / 32];<br>
> always returns bcm63xx-gpio.1. As a result of this, reset pins connected to gpios <32<br>
> will return messages "gpio X out of range", and ethernet fails to initialize.<br>
><br>
> The array *gpio_chip_labels[] should be initialized with different names for each element,<br>
> otherwise sprintf will copy the label of the gpio chip for both elements at the same time.<br>
> And the name of both elements of the array will be the same.<br>
><br>
> Using a different name on the second array element solves the problem.<br>
> Signed-off-by: Daniel Gonzalez <<a href="mailto:dgcbueu@gmail.com">dgcbueu@gmail.com</a>><br>
<br>
</span>I added a slightly different approach in r48303, can you verify it<br>
works for you?<br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
Jonas<br>
</font></span></blockquote></div><br></div>