Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   <Mario.Limonciello@dell.com>
To:     <heikki.krogerus@linux.intel.com>
CC:     <greg@kroah.com>, <pmenzel+linux-usb@molgen.mpg.de>,
        <linux-usb@vger.kernel.org>, <linux-kernel@vger.kernel.org>
Subject: RE: `ucsi_acpi: probe of USBC000:00 failed with error -12` on Dell
 XPS 13 9370
Thread-Topic: `ucsi_acpi: probe of USBC000:00 failed with error -12` on Dell
 XPS 13 9370
Thread-Index: AQHT7FnacruSscNvFUKixxVl6UQN9aQxRuSAgAANFYCAAP+ZAIAAQeIA///t/DCAAXJpAP//7vrw
Date:   Thu, 17 May 2018 13:01:20 +0000
Message-ID: <91a3c211cc8547ccb8eab844abd3b095@ausx13mpc120.AMER.DELL.COM>
References: <9365397f-7206-728e-ee76-146629bfd186@molgen.mpg.de>
 <20180515160047.GB26720@kroah.com>
 <69da4aa4-3d4b-0ffc-81ce-500713cd3743@molgen.mpg.de>
 <20180516080226.GC3536@kroah.com> <20180516115814.GB11469@kuha.fi.intel.com>
 <df6610738db046e9bd1db77d4346cfc3@ausx13mpc120.AMER.DELL.COM>
 <20180517085930.GD11469@kuha.fi.intel.com>
In-Reply-To: <20180517085930.GD11469@kuha.fi.intel.com>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

> -----Original Message-----
> From: Heikki Krogerus [mailto:heikki.krogerus@linux.intel.com]
> Sent: Thursday, May 17, 2018 4:00 AM
> To: Limonciello, Mario
> Cc: greg@kroah.com; pmenzel+linux-usb@molgen.mpg.de; linux-
> usb@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: Re: `ucsi_acpi: probe of USBC000:00 failed with error -12` on De=
ll XPS 13
> 9370
>=20
> Hi,
>=20
> On Wed, May 16, 2018 at 04:13:31PM +0000, Mario.Limonciello@dell.com wrot=
e:
> >
> >
> > > -----Original Message-----
> > > From: Heikki Krogerus [mailto:heikki.krogerus@linux.intel.com]
> > > Sent: Wednesday, May 16, 2018 6:58 AM
> > > To: Greg KH; Paul Menzel
> > > Cc: linux-usb@vger.kernel.org; linux-kernel@vger.kernel.org; Limoncie=
llo, Mario
> > > Subject: Re: `ucsi_acpi: probe of USBC000:00 failed with error -12` o=
n Dell XPS 13
> > > 9370
> > >
> > > Hi,
> > >
> > > On Wed, May 16, 2018 at 10:02:26AM +0200, Greg KH wrote:
> > > > On Tue, May 15, 2018 at 06:47:37PM +0200, Paul Menzel wrote:
> > > > > Dear Greg,
> > > > >
> > > > >
> > > > > As always, thank you for the prompt response.
> > > > >
> > > > >
> > > > > On 05/15/18 18:00, Greg KH wrote:
> > > > > > On Tue, May 15, 2018 at 04:34:03PM +0200, Paul Menzel wrote:
> > > > >
> > > > > > > Linux 4.17-rc5 shows the error below on the Dell XPS 13 9370 =
with Debian
> > > > > > > Sid/unstable.
> > > > > > >
> > > > > > > ```
> > > > > > > [???]
> > > > > > > [    0.440240] usb: port power management may be unreliable
> > > > > > > [    0.441358] usbcore: registered new interface driver usb-s=
torage
> > > > > > > [    0.441367] usbcore: registered new interface driver usbse=
rial_generic
> > > > > > > [    0.441369] usbserial: USB Serial support registered for g=
eneric
> > > > > > > [    0.441383] ioremap error for 0x3f799000-0x3f79a000, reque=
sted 0x2,
> got
> > > > > > > 0x0
> > > > > > > [    0.441518] ucsi_acpi: probe of USBC000:00 failed with err=
or -12
> > > > > > > [???]
> > > > > > > ```
> > > > > > >
> > > > > > > 1.  Are the ioremap and ucsi_acpi error related or is a separ=
ate report
> > > > > > > needed?
> > > > > >
> > > > > > The ioremap error is what causes ucsi_acpi to fail the probe ca=
ll (-12
> > > > > > is "out of memory".)
> > > > > >
> > > > > > > 2.  Do you know the reason for the ucsi_acpi error?
> > > > > >
> > > > > > the call to ioremap failed.
> > > > > >
> > > > > > Does this device really have a working typec connector?
> > > > >
> > > > > Just to avoid misunderstandings, no device was connected to the l=
aptop
> > > > > during my test.
> > > > >
> > > > > But, from other boots, the Dell docking station TB16 kind of work=
s with it,
> > > > > so I???d say the USB Type-C connector is working.
> > > >
> > > > Ok, good, this might just be the acpi tables not set up properly fo=
r
> > > > this type of connection.  Odd that the tables show it should work,
> > > > Heikki should know more about this.
> > >
> > > The firmware probable has not implemented UCSI on this board. I think
> > > Dell always supplies the ACPI device node for UCSI in their acpi
> > > tables. The _STA method in that device node is then used to inform th=
e
> > > OS if the interface exists or not. The return value for _STA comes
> > > probable from BIOS, so this is most likely a BIOS problem.
> >
> > Heikki,
> >
> > I confirmed with internal team that UCSI is implemented on XPS 9370
> > and was confirmed to be working properly with Windows 10 RS2+.
>=20
> Just to double check: "UCSI was confirmed working properly", so not
> "the Type-C ports were confirmed working properly"?

UCSI was confirmed working properly.  FWIW it's a certification requirement
in Windows.

>=20
> > The reason that _STA is responding on this device node now but wasn't
> > previously is it wasn't exposed in Linux until 4.16 when the Win 10 RS2
> > OSI string started to respond.
>=20
> OK.
>=20
> > Intel should internally have some XPS 9370 you can remotely access if
> > you would like to poke around ACPI tables some.
>=20
> I will try get access to XPS 9370, but with the acpi tables, if
> somebody could just send me acpidump, that would be enough:
>=20
>         % acpidump -o xps9370_acpi.dump
>=20
> > > Please note that UCSI will only supply status information to the
> > > operating system, so the USB Type-C ports will function normally even
> > > without it. The ports are handled in firmware on these platforms.
> > >
> > > Paul, do you have the latest BIOS?
> > >
> > >
> > > > > > Does normal USB devices work with it?
> > > > >
> > > > > Sorry for being ignorant, but could you please tell me what norma=
l USB
> > > > > devices are?
> > > >
> > > > If you plug a USB typeC device into this port, does it work?  A doc=
king
> > > > station is a little bit "different" in that it usually uses the PCI=
e
> > > > connection, not the USB connectors.  Or at least that's how my Dell
> > > > docking station works last time I tried it[1]
> >
> > I think the best description here is "Non-Thunderbolt" USB type C devic=
e.
> > Some examples:
> > There are Dell docking stations with Thunderbolt (TB16) or without (WD1=
5).
> >
> > You can also pick up little dongles for ethernet or combo dongles for
> > ethernet/VGA/HDMI/etc.
> >
> > Anything non-Thunderbolt would satisfy what Greg was looking for.
>=20
> Anything non-Thunderbolt and non-display.
>=20
> With the display adapters you would be in DisplayPort alternate mode,
> and you would again not be testing a normal USB device.
>=20
> It never hurts to check that, but I think it's safe to assume that the
> ports are functioning normally if the Thunderbolt dock was working.
> Unless I'm mistaken, even the xHCI USB host controller behind the
> USB Type-C (thunderbolt) ports is actually part of the thunderbolt
> controller.

In this laptop yes that's true, but isn't a general statement. It's possibl=
e
to have USB split mode (ie XHCI comes from PCH).