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@xxxxxx> 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@xxxxxx> 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@xxxxxx> Cc: Vagrant Cascadian <vagrant@xxxxxxxxxx> Signed-off-by: Roger Quadros <rogerq@xxxxxx> --- arch/arm/boot/dts/am57xx-beagle-x15.dts | 17 ----------------- 1 file changed, 17 deletions(-) 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>; }; -- 2.5.0 -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
