From patchwork Fri Sep 28 09:37:23 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Bj=C3=B8rn_Mork?= X-Patchwork-Id: 10619221 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 1C48615E8 for ; Fri, 28 Sep 2018 09:37:49 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 0DF112B016 for ; Fri, 28 Sep 2018 09:37:49 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 01A192B03B; Fri, 28 Sep 2018 09:37:48 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.0 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id C12032B016 for ; Fri, 28 Sep 2018 09:37:47 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729124AbeI1QAk (ORCPT ); Fri, 28 Sep 2018 12:00:40 -0400 Received: from canardo.mork.no ([148.122.252.1]:52313 "EHLO canardo.mork.no" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729008AbeI1QAj (ORCPT ); Fri, 28 Sep 2018 12:00:39 -0400 Received: from miraculix.mork.no ([IPv6:2a02:2121:284:6dba:c40f:c1ff:fecc:90f1]) (authenticated bits=0) by canardo.mork.no (8.15.2/8.15.2) with ESMTPSA id w8S9bhTH029223 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO); Fri, 28 Sep 2018 11:37:44 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=mork.no; s=b; t=1538127464; bh=zgPK/Y13kn92dH67QECZvo+fpz9LrNEv96Gk/2eDeEA=; h=From:To:Cc:Subject:Date:Message-Id:From; b=Vd4BNfpAP+x8euFvQwjxjfbR5QogzqoCZY81pph9E7QTNywSqUX52iLOrYwll56Jl dWeQr/eScwPIM2Ui+MyEuwtUnkZdU5WfLuKQqfVhzHh2nOPTrte23oI/BToaS9Dfjv XJcogieIQHtHaHs0KbCbU+1uyLEe2kVif3GHiyfQ= Received: from bjorn by miraculix.mork.no with local (Exim 4.89) (envelope-from ) id 1g5pDG-0002SE-0R; Fri, 28 Sep 2018 11:37:30 +0200 From: =?utf-8?q?Bj=C3=B8rn_Mork?= To: linux-usb@vger.kernel.org Cc: =?utf-8?q?Bj=C3=B8rn_Mork?= Subject: [PATCH] usb: export firmware port location in sysfs Date: Fri, 28 Sep 2018 11:37:23 +0200 Message-Id: <20180928093723.9330-1-bjorn@mork.no> X-Mailer: git-send-email 2.11.0 MIME-Version: 1.0 X-Virus-Scanned: clamav-milter 0.100.1 at canardo X-Virus-Status: Clean Sender: linux-usb-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-usb@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP The platform firmware "location" data is used to find port peer relationships. But firmware is an unreliable source. Errors in ACPI tables can lead to missing or wrong peer relationships. Debugging this is currently hard. Exporting the location attribute makes it easier to spot mismatches between the firmware data and the real world. Signed-off-by: Bjørn Mork --- This patch got stuck in one of my debugging branches. It proved very useful to me while trying to figure out why the "peer" link was useless on a specific host. And if it was useful to me once, then maybe it will be to someone else as well... tl;dr; The full use case for anyone interested: Some current LTE modems with USB3 SS support come with a bootloader supporting USB2 only. The application and bootloader modes are provided by different softwares running on the modem (which of course is just another Linux system with a UDC). None of the descriptors are therefore guaranteed to be identical, or even similar. Doing a firmware upgrade of such a device involves - some preparation in application mode (USB3), - rebooting into bootloader mode (USB2), and - finally booting into the upgraded application firmware (USB3) to verify the upgrade. The firmware upgrade tool should make sure it talks to the same physical device in all three phases. The only semi-reliable way to do that is to look for "new" devices in the expected mode, connected to the same physical USB port. But the locical port will change due to the USB2/3 switch, and all we are left with is the "peer" link. Which can, and do, fail due to buggy ACPI tables. This patch won't solve that problem, but it makes it a lot easier to detect. Documentation/ABI/testing/sysfs-bus-usb | 15 ++++++++++++--- drivers/usb/core/port.c | 9 +++++++++ 2 files changed, 21 insertions(+), 3 deletions(-) diff --git a/Documentation/ABI/testing/sysfs-bus-usb b/Documentation/ABI/testing/sysfs-bus-usb index 0bd731cbb50c..738bcccc420d 100644 --- a/Documentation/ABI/testing/sysfs-bus-usb +++ b/Documentation/ABI/testing/sysfs-bus-usb @@ -173,14 +173,14 @@ Description: The file will be present for all speeds of USB devices, and will always read "no" for USB 1.1 and USB 2.0 devices. -What: /sys/bus/usb/devices/.../(hub interface)/portX +What: /sys/bus/usb/devices/.../(hub interface)/usbY-portX Date: August 2012 Contact: Lan Tianyu Description: The /sys/bus/usb/devices/.../(hub interface)/portX is usb port device's sysfs directory. -What: /sys/bus/usb/devices/.../(hub interface)/portX/connect_type +What: /sys/bus/usb/devices/.../(hub interface)/usbY-portX/connect_type Date: January 2013 Contact: Lan Tianyu Description: @@ -189,7 +189,16 @@ Description: The file will read "hotplug", "wired" and "not used" if the information is available, and "unknown" otherwise. -What: /sys/bus/usb/devices/.../(hub interface)/portX/usb3_lpm_permit +What: /sys/bus/usb/devices/.../(hub interface)/usbY-portX/location +Date: October 2018 +Contact: Bjørn Mork +Description: + Some platforms provide usb port physical location through + firmware. This is used by the kernel to pair up logical ports + mapping to the same physical connector. The attribute exposes the + raw location value as a hex integer. + +What: /sys/bus/usb/devices/.../(hub interface)/usbY-portX/usb3_lpm_permit Date: November 2015 Contact: Lu Baolu Description: diff --git a/drivers/usb/core/port.c b/drivers/usb/core/port.c index 1a01e9ad3804..baa35ba410b6 100644 --- a/drivers/usb/core/port.c +++ b/drivers/usb/core/port.c @@ -16,6 +16,15 @@ static int usb_port_block_power_off; static const struct attribute_group *port_dev_group[]; +static ssize_t location_show(struct device *dev, + struct device_attribute *attr, char *buf) +{ + struct usb_port *port_dev = to_usb_port(dev); + + return sprintf(buf, "0x%08x\n", port_dev->location); +} +static DEVICE_ATTR_RO(location); + static ssize_t connect_type_show(struct device *dev, struct device_attribute *attr, char *buf) {