> +&i2c2 {
> +	pinctrl-0 = <&i2c2_pins>;
> +	pinctrl-names = "default";
> +
> +	status = "okay";
> +};

I think we should add the bus speed here explicitly and not rely on a
default:

	clock-frequency = <100000>;

Maybe 400000 if all connected devices support that.