Hi Harald,

On Wed, Apr 15, 2020 at 12:54 PM Harald Seiler <h...@denx.de> wrote:

> +/ {
> +       fec_vio: regulator-fec {
> +               compatible = "regulator-fixed";
> +
> +               regulator-name = "fec-vio";
> +               gpio = <&gpio1 7 GPIO_ACTIVE_HIGH>;

By looking at your board code, this should be GPIO_ACTIVE_LOW instead.

> +               regulator-always-on;

This one could be removed since it has the FEC as a consumer.

Reply via email to