> I am okay if it is reasonable below in file > arch/arm64/boot/dts/amlogic/meson-g12a.dtsi. I2c controller node just uses > axg's compatible. > > i2c0: i2c@1f000 { > compatible = "amlogic,meson-axg-i2c"; Actually, you should have compatible = "amlogic,meson-g12a-i2c", "amlogic,meson-axg-i2c"; in the DT to have support for future SoC specific additions. And then, patch 1 is needed. Or do you handle this differently? I'd think this is DT standard.