Re: [PATCH net-next v4 3/8] net: txgbe: Register I2C platform device

[Andi Shyti <andi.shyti@xxxxxxxxxx>]

Hi Jiawen,

[...]

> +	ret = txgbe_i2c_register(txgbe);
> +	if (ret) {
> +		wx_err(txgbe->wx, "failed to init i2c interface: %d\n", ret);
> +		goto err_unregister_swnode;
> +	}
> +
>  	return 0;
> +
> +err_unregister_swnode:
> +	software_node_unregister_node_group(txgbe->nodes.group);
> +
> +	return ret;

no need for the goto here... in my opinion it's easier if you put
software_node_unregister_node_group() under the if and return
ret.

Andi