Dual custom camera sensor device tree on Jetson TX2

Hello,

I am trying to develop two custom CAMERA SENSOR at the same time .
It was modified as follows with reference to the tegra186-camera-e3323-a00.dtsi file.

I want to use two sensors with the same i2c line and different addresses.

Is there any problem if I modify as below?

	i2c@3180000 {
		s5k5e8_c@10 {
			compatible = "nvidia,s5k5e8";
			/* I2C device address */
			reg = <0x10>;
			
			/* V4L2 device node location */
			devnode = "video0";

			/* Physical dimensions of sensor */
			physical_w = "4.675";
			physical_h = "3.633";

			/* Define any required hw resources needed by driver */
			/* ie. clocks, io pins, power sources */
			avdd-reg = "vana";
			iovdd-reg = "vif";
			dvdd-reg = "vdig";

			/* Sensor output flip settings */
			vertical-flip = "false";


			mode0 { // S5K5E8_MODE_2608X1960
				mclk_khz = "24000";
				……
				…...
				embedded_metadata_height = "0";
			};

			mode1 { //S5K5E8_MODE_1304X980
				mclk_khz = "24000";
				……
				……
				embedded_metadata_height = "0";
			};


			ports {
				#address-cells = <1>;
				#size-cells = <0>;

				port@0 {
					reg = <0>;
					e3327_s5k5e8_out0: endpoint {
						port-index = <2>;
						bus-width = <2>;
						remote-endpoint = <&e3327_csi_in0>;
					};
				};
			};
		};


		s5k5e8_d@18 {
			compatible = "nvidia,s5k5e8";
			/* I2C device address */
			reg = <0x18>;
			
			/* V4L2 device node location */
			devnode = "video1";

			/* Physical dimensions of sensor */
			physical_w = "4.675";
			physical_h = "3.633";

			/* Define any required hw resources needed by driver */
			/* ie. clocks, io pins, power sources */
			avdd-reg = "vana";
			iovdd-reg = "vif";
			dvdd-reg = "vdig";

			/* Sensor output flip settings */
			vertical-flip = "false";


			mode0 { // S5K5E8_MODE_2608X1960
				mclk_khz = "24000";
				……
				…...
				embedded_metadata_height = "0";
			};

			mode1 { //S5K5E8_MODE_1304X980
				mclk_khz = "24000";
				……
				…...
				embedded_metadata_height = "0";
			};


			ports {
				#address-cells = <1>;
				#size-cells = <0>;

				port@0 {
					reg = <0>;
					e3327_s5k5e8_out1: endpoint {
						port-index = <3>;
						bus-width = <2>;
						remote-endpoint = <&e3327_csi_in1>;
					};
				};
			};
		};

	};

No problem if i2c address not conflict.