| Message ID | 20210308070843.2096992-1-quanyang.wang@windriver.com |
|---|---|
| State | New |
| Headers | show |
| Series | arm64: dts: zynqmp: Add compatible strings for si5328 | expand |
Hi, On 3/8/21 9:35 AM, Laurent Pinchart wrote: > Hi Quanyang, > > Thank you for the patch. > > On Mon, Mar 08, 2021 at 03:08:43PM +0800, quanyang.wang@windriver.com wrote: >> From: Quanyang Wang <quanyang.wang@windriver.com> >> >> The function of_i2c_get_board_info will call of_modalias_node to check >> if a device_node contains "compatible" string. So let's assign the >> proper string "silabs,si5328" to clock-generator@69's compatible >> property to eliminate the error info as below: > > As far as I can tell, "silabs,si5328" isn't documented in > Documentation/devicetree/bindings/. We need DT bindings before making > use of the compatible string. > the second option is to remove/comment this node. Thanks, Michal
Hi Laurent, On 3/8/21 4:35 PM, Laurent Pinchart wrote: > Hi Quanyang, > > Thank you for the patch. > > On Mon, Mar 08, 2021 at 03:08:43PM +0800, quanyang.wang@windriver.com wrote: >> From: Quanyang Wang <quanyang.wang@windriver.com> >> >> The function of_i2c_get_board_info will call of_modalias_node to check >> if a device_node contains "compatible" string. So let's assign the >> proper string "silabs,si5328" to clock-generator@69's compatible >> property to eliminate the error info as below: > As far as I can tell, "silabs,si5328" isn't documented in > Documentation/devicetree/bindings/. We need DT bindings before making > use of the compatible string. Thank you for your review. I will send a new patch to delete these device nodes. Best Regards Quanyang > >> i2c i2c-10: of_i2c: modalias failure on /axi/i2c@ff030000/i2c-mux@74/i2c@4/clock-generator@69 >> i2c i2c-10: Failed to create I2C device for /axi/i2c@ff030000/i2c-mux@74/i2c@4/clock-generator@69 >> >> Fixes: 82a7ebf00224 ("arm64: dts: zynqmp: Add DT description for si5328 for zcu102/zcu106") >> Signed-off-by: Quanyang Wang <quanyang.wang@windriver.com> >> --- >> arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts | 1 + >> arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts | 1 + >> 2 files changed, 2 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts >> index 12e8bd48dc8c..a73841ad24ce 100644 >> --- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts >> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts >> @@ -590,6 +590,7 @@ si5328: clock-generator@69 {/* SI5328 - u20 */ >> #address-cells = <1>; >> #size-cells = <0>; >> #clock-cells = <1>; >> + compatible = "silabs,si5328"; >> clocks = <&refhdmi>; >> clock-names = "xtal"; >> clock-output-names = "si5328"; >> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts >> index 18771e868399..24ce64364d63 100644 >> --- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts >> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts >> @@ -591,6 +591,7 @@ si5328: clock-generator@69 {/* SI5328 - u20 */ >> #address-cells = <1>; >> #size-cells = <0>; >> #clock-cells = <1>; >> + compatible = "silabs,si5328"; >> clocks = <&refhdmi>; >> clock-names = "xtal"; >> clock-output-names = "si5328";
diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts index 12e8bd48dc8c..a73841ad24ce 100644 --- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts +++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts @@ -590,6 +590,7 @@ si5328: clock-generator@69 {/* SI5328 - u20 */ #address-cells = <1>; #size-cells = <0>; #clock-cells = <1>; + compatible = "silabs,si5328"; clocks = <&refhdmi>; clock-names = "xtal"; clock-output-names = "si5328"; diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts index 18771e868399..24ce64364d63 100644 --- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts +++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts @@ -591,6 +591,7 @@ si5328: clock-generator@69 {/* SI5328 - u20 */ #address-cells = <1>; #size-cells = <0>; #clock-cells = <1>; + compatible = "silabs,si5328"; clocks = <&refhdmi>; clock-names = "xtal"; clock-output-names = "si5328";