diff mbox

Beagle-X15, Debian and reproducible-builds

Message ID 57051EC5.3080006@ti.com
State New
Headers show

Commit Message

Roger Quadros April 6, 2016, 2:35 p.m. UTC 
On 04/04/16 10:08, Roger Quadros wrote:
> On 04/04/16 06:02, Vagrant Cascadian wrote:

>> On 2016-04-01, Roger Quadros <rogerq@ti.com> wrote:

>>> On 01/04/16 02:04, Vagrant Cascadian wrote:

>>>> On 2016-03-31, Nishanth Menon wrote:

>>>>> On 03/31/2016 01:04 PM, Vagrant Cascadian wrote:

>>>>>> On 2016-03-30, Nishanth Menon wrote:

>>>>>>> On 03/30/2016 03:08 PM, Vagrant Cascadian wrote:

>>

>>>>> testing latest master: v2016.03-523-g080c499df689

>>>>>

>>>>> Looks like SD card with fat partitions boots u-boot:

>>>>> http://pastebin.ubuntu.com/15571915/ with a newer rev of board.

>>>>>

>>>>> http://pastebin.ubuntu.com/15571902/ with the A.2 rev of the board.

>>>>

>>>> Attached should be a boot log on the BeagleBoard-X15 booting with u-boot

>>>> master (080c499df689e8c42df70de44502c0d71533dda8) and linux 4.6-rc1 with

>>>> the config provided earlier in the thread.

>>>>

>>>> No luck with Roger Quadros's patch removing the extra init_sata call in

>>>> u-boot, although I only tested that with u-boot 2016.03.

>>>>

>>>> Since it's possibe the initramfs doesn't have all the necessary modules

>>>

>>> you would need "phy_ti_pipe3" along with the AHCI and scsi modules.

>>

>> Yeah, that's present and loaded, even. along with ocp2scp and

>> phy_omap_control, the others that usually seem to be required as far as

>> I can tell.

>>

>>

>>>> to get eSATA working, I'll try testing with rootfs on microSD or eMMC

>>>> later today or tomorrow, so I can be confident it has all the modules

>>>> the kernel was built with when debugging the eSATA issues.

>>>

>>> That is a good idea.

>>

>> Unfortunately, even with rootfs on eMMC, with all the modules built with

>> the kernel available, it still doesn't detect the eSATA SSD.

>>

>>

>>> However SATA seems to be failing right at u-boot level from your log.

>>> That doesn't seem right.

>>>

>>> 2016-03-31_22:45:17.40524 SATA link 0 timeout.

>>> 2016-03-31_22:45:17.40530 AHCI 0001.0300 32 slots 1 ports 3 Gbps 0x1 impl SATA mode

>>> 2016-03-31_22:45:17.40534 flags: 64bit ncq stag pm led clo only pmp pio slum part ccc apst 

>>

>> I've not seen SATA work from mainline u-boot on the BeagleBoard-X15.

>>

>>

>>> Can you please try to power cycle board and hard drive always till we

>>> have got the eSATA issue figured out?

>>

>> Power cycling didn't help with my most recent test, using linux 4.6-rc2

>> and u-boot 2016.03 (with a small patch to support

>> config_distro_bootcmd).

>>

>>

>> The attached kernel config with linux 4.6-rc2 doesn't work with

>> OMAP_OCP2SCP, OMAP_CONTROL_PHY and TI_PIPE3 as modules, but works as

>> builtins.

> 

> OK. This is the magic configuration that is needed for SATA to work.

> When earlier I said that TI_PIPE3 must be built in, that would imply that

> OMCP_OCP2SCP and OMAP_CONTROL_PHY will be built in due to module dependency.

> 

>>

>> I did at one point get the impression that the eSATA port provides power

>> off of the USB rail... perhaps there is an ordering issue with bringup

>> of USB vs. eSATA when loaded as a module?

>>

> 

> We don't expect eSATA to provide power to the SATA drive. You will have to

> power it externally.


Correcting my earlier statement. We do support eSATA drives that can run
@5V/1A power.

I've send the below patch to linux-omap and that reduces USB dependency
on EXTCON and might fix your issue as well.

---
From: Roger Quadros <rogerq@ti.com>

Date: Wed, 6 Apr 2016 17:06:48 +0300
Subject: [PATCH] ARM: dts: am57xx-beagle-x15:: remove extcon_usb1

USB1 controller is hardwired to be used as Host only port so
we don't need to check ID pin state and can get rid of extcon_usb1.

This also reduces USB1 controller's and so eSATA power's dependency
with EXTCON.

Cc: Franklin S Cooper Jr. <fcooper@ti.com>
Cc: Vagrant Cascadian <vagrant@debian.org>
Signed-off-by: Roger Quadros <rogerq@ti.com>

---
 arch/arm/boot/dts/am57xx-beagle-x15.dts | 17 -----------------
 1 file changed, 17 deletions(-)

-- 
2.5.0

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/boot/dts/am57xx-beagle-x15.dts b/arch/arm/boot/dts/am57xx-beagle-x15.dts
index 0a5fc5d..4168eb9 100644
--- a/arch/arm/boot/dts/am57xx-beagle-x15.dts
+++ b/arch/arm/boot/dts/am57xx-beagle-x15.dts
@@ -99,13 +99,6 @@ 
 		#cooling-cells = <2>;
 	};
 
-	extcon_usb1: extcon_usb1 {
-		compatible = "linux,extcon-usb-gpio";
-		id-gpio = <&gpio7 25 GPIO_ACTIVE_HIGH>;
-		pinctrl-names = "default";
-		pinctrl-0 = <&extcon_usb1_pins>;
-	};
-
 	hdmi0: connector {
 		compatible = "hdmi-connector";
 		label = "hdmi";
@@ -349,12 +342,6 @@ 
 		>;
 	};
 
-	extcon_usb1_pins: extcon_usb1_pins {
-		pinctrl-single,pins = <
-			DRA7XX_CORE_IOPAD(0x37ec, PIN_INPUT_PULLUP | MUX_MODE14) /* uart1_rtsn.gpio7_25 */
-		>;
-	};
-
 	tpd12s015_pins: pinmux_tpd12s015_pins {
 		pinctrl-single,pins = <
 			DRA7XX_CORE_IOPAD(0x37b0, PIN_OUTPUT | MUX_MODE14)		/* gpio7_10 CT_CP_HPD */
@@ -706,10 +693,6 @@ 
 	pinctrl-0 = <&usb1_pins>;
 };
 
-&omap_dwc3_1 {
-	extcon = <&extcon_usb1>;
-};
-
 &omap_dwc3_2 {
 	extcon = <&extcon_usb2>;
 };