devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Anand Moon <moon.linux@yahoo.com>
To: Rob Herring <robh+dt@kernel.org>, Pawel Moll <pawel.moll@arm.com>,
	Mark Rutland <mark.rutland@arm.com>,
	Ian Campbell <ijc+devicetree@hellion.org.uk>,
	Kumar Gala <galak@codeaurora.org>,
	Russell King <linux@arm.linux.org.uk>,
	Kukjin Kim <kgene@kernel.org>
Cc: devicetree@vger.kernel.org, Anand Moon <moon.linux@yahoo.com>,
	linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: dts: exynos5420 Fixes -ELOOP (Too many levels of symbolic links)
Date: Thu, 12 Mar 2015 12:59:51 +0530	[thread overview]
Message-ID: <1426145391-30089-1-git-send-email-moon.linux@yahoo.com> (raw)

BUG of recursive symbolic link and be observed by using below powerdebug
        git://git.linaro.org/tools/powerdebug.git.

This utility populates the Clock, Regulators, Sensors and Gpio.
Some how the regulators were not getting populated by this utility.
After debugging this powerdebug application using 'strace' I can across following.

Below output is trunkated.

stat64("/sys/class/regulator/regulator.0/usb@12000000-vdd10/
iommu_group/devices/usb@12000000/iommu_group/devices/usb@12000000/
iommu_group/devices/usb@12000000/iommu_group/devices/usb@12000000/
iommu_group/devices/usb@12000000/iommu_group/devices/usb@12000000/
iommu_group/devices/usb@12000000/iommu_group/devices/usb@12000000/
iommu_group/devices/usb@12000000/iommu_group/devices/usb@12000000/
iommu_group/devices/usb@12000000/iommu_group/devices/usb@12000000/
iommu_group/devices/usb@12000000/iommu_group/devices/usb@12000000/
iommu_group/devices/usb@12000000/iommu_group/devices/usb@12000000/
iommu_group/devices/usb@12000000/iommu_group/devices/usb@12000000/
iommu_group/devices/usb@12000000/iommu_group", 0xbec45fb8) = -1 ELOOP (Too many levels of symbolic links)
close(63)                               = 0
close(62)                               = 0

These changes depend on following series of commit.
 Pull iommu fixes from Joerg Roedel:
     "The patches contain:

       - fix multiple ARM IOMMU drivers to behave well when the hardware is
         not present

       - mark MSM driver as broken

       - fix build errors with the new ARM generic io-page-table code"

    * tag 'iommu-fixes-v4.0-rc3' of git://git.kernel.org/pub/scm/linux/kernel/git/joro/iommu:
      iommu/io-pgtable-arm: Add built time dependency
      iommu/msm: Mark driver BROKEN
      iommu/rockchip: Play nice in multi-platform builds
      iommu/omap: Play nice in multi-platform builds
      iommu/exynos: Play nice in multi-platform builds
      iommu/io-pgtable-arm: Fix self-test WARNs on i386

After these changes powerdebug will populate regulators.

Tested on OdroidXU3 board.

Signed-off-by: Anand Moon <moon.linux@yahoo.com>
---
 arch/arm/boot/dts/exynos5420.dtsi | 28 ++++++++++++++--------------
 1 file changed, 14 insertions(+), 14 deletions(-)

diff --git a/arch/arm/boot/dts/exynos5420.dtsi b/arch/arm/boot/dts/exynos5420.dtsi
index 9dc2e97..a5f13a3 100644
--- a/arch/arm/boot/dts/exynos5420.dtsi
+++ b/arch/arm/boot/dts/exynos5420.dtsi
@@ -840,14 +840,14 @@
 		#address-cells = <1>;
 		#size-cells = <1>;
 		ranges;
+	};
 
-		usbdrd_dwc3_0: dwc3 {
-			compatible = "snps,dwc3";
-			reg = <0x12000000 0x10000>;
-			interrupts = <0 72 0>;
-			phys = <&usbdrd_phy0 0>, <&usbdrd_phy0 1>;
-			phy-names = "usb2-phy", "usb3-phy";
-		};
+	usbdrd_dwc3_0: dwc3 {
+		compatible = "snps,dwc3";
+		reg = <0x12000000 0x10000>;
+		interrupts = <0 72 0>;
+		phys = <&usbdrd_phy0 0>, <&usbdrd_phy0 1>;
+		phy-names = "usb2-phy", "usb3-phy";
 	};
 
 	usbdrd_phy0: phy@12100000 {
@@ -866,14 +866,14 @@
 		#address-cells = <1>;
 		#size-cells = <1>;
 		ranges;
+	};
 
-		usbdrd_dwc3_1: dwc3 {
-			compatible = "snps,dwc3";
-			reg = <0x12400000 0x10000>;
-			interrupts = <0 73 0>;
-			phys = <&usbdrd_phy1 0>, <&usbdrd_phy1 1>;
-			phy-names = "usb2-phy", "usb3-phy";
-		};
+	usbdrd_dwc3_1: dwc3 {
+		compatible = "snps,dwc3";
+		reg = <0x12400000 0x10000>;
+		interrupts = <0 73 0>;
+		phys = <&usbdrd_phy1 0>, <&usbdrd_phy1 1>;
+		phy-names = "usb2-phy", "usb3-phy";
 	};
 
 	usbdrd_phy1: phy@12500000 {
-- 
2.1.0

                 reply	other threads:[~2015-03-12  7:29 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1426145391-30089-1-git-send-email-moon.linux@yahoo.com \
    --to=moon.linux@yahoo.com \
    --cc=devicetree@vger.kernel.org \
    --cc=galak@codeaurora.org \
    --cc=ijc+devicetree@hellion.org.uk \
    --cc=kgene@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=mark.rutland@arm.com \
    --cc=pawel.moll@arm.com \
    --cc=robh+dt@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).