> > + ufs_1_phy: ufs0-phy@17f04000 { > > Node name: "phy" (or ufs-phy). The previous node also could be corrected. phy@17f04000 looks better. I'll change this and make an additional patch to correct previous ufs0 nodes. > > > + compatible = "samsung,exynosautov9-ufs-phy"; > > + reg = <0x17f04000 0xc00>; > > + reg-names = "phy-pma"; > > + samsung,pmu-syscon = <&pmu_system_controller 0x72c>; > > + #phy-cells = <0>; > > + clocks = <&xtcxo>; > > + clock-names = "ref_clk"; > > + status = "disabled"; > > + }; > > + > > ufs_0: ufs0@17e00000 { > > compatible ="samsung,exynosautov9-ufs"; > > > > @@ -400,6 +411,27 @@ ufs_0: ufs0@17e00000 { > > samsung,sysreg = <&syscon_fsys2 0x710>; > > status = "disabled"; > > }; > > + > > + ufs_1: ufs0@17f00000 { > > + compatible ="samsung,exynosautov9-ufs"; > > + > > + reg = <0x17f00000 0x100>, /* 0: HCI standard */ > > + <0x17f01100 0x410>, /* 1: Vendor-specific */ > > + <0x17f80000 0x8000>, /* 2: UNIPRO */ > > + <0x17de0000 0x2200>; /* 3: UFS protector */ > > Align these please with first <> entry. I'll apply this on the next patchset and fix the same on ufs0 node as well. Best Regards, Chanho Park