Re: [PATCH 6/7] ARM: dts: imx6q: add simple-bus compatible string for anatop
From: Richard Zhao <hidden>
Date: 2012-08-22 08:53:09
Also in:
linux-arm-kernel, lkml
On Wed, Aug 22, 2012 at 03:18:47PM +0800, Dong Aisheng wrote:
quoted hunk ↗ jump to hunk
From: Dong Aisheng <redacted> Originally the anatop regulator devices are populated by mfd anatop driver. Since mfd anatop driver will be deleted later, we change to populate the regulator devices by devicetree automatically. This will cause some warning messages as follows during boot due to device recreation: "vdd1p1: Failed to create debugfs directory" But it does not break any function. Later, we will remove mfd anatop driver which can get rid of this error message. Signed-off-by: Dong Aisheng <redacted> --- arch/arm/boot/dts/imx6q.dtsi | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)diff --git a/arch/arm/boot/dts/imx6q.dtsi b/arch/arm/boot/dts/imx6q.dtsi index 7076be0..426f735 100644 --- a/arch/arm/boot/dts/imx6q.dtsi +++ b/arch/arm/boot/dts/imx6q.dtsi@@ -379,7 +379,7 @@ }; anatop: anatop@020c8000 { - compatible = "fsl,imx6q-anatop", "fsl,imx-syscon"; + compatible = "fsl,imx6q-anatop", "fsl,imx-syscon", "simple-bus";
To prevent bisect break, it should merge with patch #4. It's really strange to use simple-bus, because it's not a bus. I like more the way how anatop driver handle it. Anatop driver populate devices in its code. Thanks Richard
reg = <0x020c8000 0x1000>; interrupts = <0 49 0x04 0 54 0x04 0 127 0x04>; -- 1.7.0.4