* [PATCH v9 0/5] usb: xhci: Add support for Renesas USB controllers @ 2020-04-14 16:41 Vinod Koul 2020-04-14 16:41 ` [PATCH v9 1/5] usb: hci: add hc_driver as argument for usb_hcd_pci_probe Vinod Koul ` (6 more replies) 0 siblings, 7 replies; 11+ messages in thread From: Vinod Koul @ 2020-04-14 16:41 UTC (permalink / raw) To: Mathias Nyman, Greg Kroah-Hartman Cc: linux-arm-msm, Bjorn Andersson, Vinod Koul, Yoshihiro Shimoda, Christian Lamparter, John Stultz, Alan Stern, Andreas Böhler, linux-usb, linux-kernel This series add support for Renesas USB controllers uPD720201 and uPD720202. These require firmware to be loaded and in case devices have ROM those can also be programmed if empty. If ROM is programmed, it runs from ROM as well. This includes patches from Christian which supported these controllers w/o ROM and later my patches for ROM support and debugfs hook for rom erase and export of xhci-pci functions. Changes in v9: Make fw load a sync call and have single instance of probe execute, elimating probe/remove races Add quirk for renesas and use that for loading Changes in v8: Fix compile error reported by Kbuild-bot by making usb_hcd_pci_probe() take const struct hc_driver * as argument Changes in v7: Make a single module which removes issues with module loading Keep the renesas code in renesas file Add hc_driver as argument for usb_hcd_pci_probe and modify hdc drivers to pass this and not use driver_data Use driver data for fw name Remove code to check if we need to load firmware or not remove multiple fw version support, we can do that with symlink in userspace Changes in v6: Move the renesas code into a separate driver which invokes xhci-pci functions. Changes in v5: Added a debugfs rom erase patch, helps in debugging Squashed patch 1 & 2 as requested by Mathias Changes in v4: Rollback the delay values as we got device failures Changes in v3: Dropped patch 2 as discussed with Christian Removed aligned 8 bytes check Change order for firmware search from highest version to lowest Added entry for new firmware for device 0x14 as well Add tested by Christian Changes in v2: used macros for timeout count and delay removed renesas_fw_alive_check cleaned renesas_fw_callback removed recurion for renesas_fw_download added MODULE_FIRMWARE added comment for multiple fw order Christian Lamparter (1): usb: renesas-xhci: Add the renesas xhci driver Vinod Koul (4): usb: hci: add hc_driver as argument for usb_hcd_pci_probe usb: xhci: Add support for Renesas controller with memory usb: renesas-xhci: Add ROM loader for uPD720201 usb: xhci: provide a debugfs hook for erasing rom drivers/usb/core/hcd-pci.c | 7 +- drivers/usb/host/Makefile | 3 +- drivers/usb/host/ehci-pci.c | 6 +- drivers/usb/host/ohci-pci.c | 9 +- drivers/usb/host/uhci-pci.c | 8 +- drivers/usb/host/xhci-pci-renesas.c | 740 ++++++++++++++++++++++++++++ drivers/usb/host/xhci-pci.c | 47 +- drivers/usb/host/xhci-pci.h | 16 + drivers/usb/host/xhci.h | 1 + include/linux/usb/hcd.h | 3 +- 10 files changed, 817 insertions(+), 23 deletions(-) create mode 100644 drivers/usb/host/xhci-pci-renesas.c create mode 100644 drivers/usb/host/xhci-pci.h -- 2.25.1 ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v9 1/5] usb: hci: add hc_driver as argument for usb_hcd_pci_probe 2020-04-14 16:41 [PATCH v9 0/5] usb: xhci: Add support for Renesas USB controllers Vinod Koul @ 2020-04-14 16:41 ` Vinod Koul 2020-04-14 16:41 ` [PATCH v9 2/5] usb: renesas-xhci: Add the renesas xhci driver Vinod Koul ` (5 subsequent siblings) 6 siblings, 0 replies; 11+ messages in thread From: Vinod Koul @ 2020-04-14 16:41 UTC (permalink / raw) To: Mathias Nyman, Greg Kroah-Hartman Cc: linux-arm-msm, Bjorn Andersson, Vinod Koul, Yoshihiro Shimoda, Christian Lamparter, John Stultz, Alan Stern, Andreas Böhler, linux-usb, linux-kernel, Mathias Nyman usb_hcd_pci_probe expects users to call this with driver_data set as hc_driver, that limits the possibility of using the driver_data for driver data. Add hc_driver as argument to usb_hcd_pci_probe and modify the callers ehci/ohci/xhci/uhci to pass hc_driver as argument and freeup the driver_data used Tested xhci driver on Dragon-board RB3, compile tested ehci and ohci and uhci Suggested-by: Mathias Nyman <mathias.nyman@linux.intel.com> [For all but the xHCI parts] Acked-by: Alan Stern <stern@rowland.harvard.edu> [For the xhci part] Acked-by: Mathias Nyman <mathias.nyman@linux.intel.com> Signed-off-by: Vinod Koul <vkoul@kernel.org> --- drivers/usb/core/hcd-pci.c | 7 ++++--- drivers/usb/host/ehci-pci.c | 6 ++---- drivers/usb/host/ohci-pci.c | 9 ++++++--- drivers/usb/host/uhci-pci.c | 8 ++++++-- drivers/usb/host/xhci-pci.c | 14 +++++--------- include/linux/usb/hcd.h | 3 ++- 6 files changed, 25 insertions(+), 22 deletions(-) diff --git a/drivers/usb/core/hcd-pci.c b/drivers/usb/core/hcd-pci.c index f0a259937da8..1547aa6e5314 100644 --- a/drivers/usb/core/hcd-pci.c +++ b/drivers/usb/core/hcd-pci.c @@ -159,6 +159,7 @@ static void ehci_wait_for_companions(struct pci_dev *pdev, struct usb_hcd *hcd, * usb_hcd_pci_probe - initialize PCI-based HCDs * @dev: USB Host Controller being probed * @id: pci hotplug id connecting controller to HCD framework + * @driver: USB HC driver handle * Context: !in_interrupt() * * Allocates basic PCI resources for this USB host controller, and @@ -169,9 +170,9 @@ static void ehci_wait_for_companions(struct pci_dev *pdev, struct usb_hcd *hcd, * * Return: 0 if successful. */ -int usb_hcd_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) +int usb_hcd_pci_probe(struct pci_dev *dev, const struct pci_device_id *id, + const struct hc_driver *driver) { - struct hc_driver *driver; struct usb_hcd *hcd; int retval; int hcd_irq = 0; @@ -181,7 +182,7 @@ int usb_hcd_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) if (!id) return -EINVAL; - driver = (struct hc_driver *)id->driver_data; + if (!driver) return -EINVAL; diff --git a/drivers/usb/host/ehci-pci.c b/drivers/usb/host/ehci-pci.c index 1a48ab1bd3b2..3c3820ad9092 100644 --- a/drivers/usb/host/ehci-pci.c +++ b/drivers/usb/host/ehci-pci.c @@ -360,23 +360,21 @@ static int ehci_pci_probe(struct pci_dev *pdev, const struct pci_device_id *id) { if (is_bypassed_id(pdev)) return -ENODEV; - return usb_hcd_pci_probe(pdev, id); + return usb_hcd_pci_probe(pdev, id, &ehci_pci_hc_driver); } static void ehci_pci_remove(struct pci_dev *pdev) { pci_clear_mwi(pdev); - usb_hcd_pci_remove(pdev); + usb_hcd_pci_remove(pdev); } /* PCI driver selection metadata; PCI hotplugging uses this */ static const struct pci_device_id pci_ids [] = { { /* handle any USB 2.0 EHCI controller */ PCI_DEVICE_CLASS(PCI_CLASS_SERIAL_USB_EHCI, ~0), - .driver_data = (unsigned long) &ehci_pci_hc_driver, }, { PCI_VDEVICE(STMICRO, PCI_DEVICE_ID_STMICRO_USB_HOST), - .driver_data = (unsigned long) &ehci_pci_hc_driver, }, { /* end: all zeroes */ } }; diff --git a/drivers/usb/host/ohci-pci.c b/drivers/usb/host/ohci-pci.c index 22117a6aeb4a..585222af24ff 100644 --- a/drivers/usb/host/ohci-pci.c +++ b/drivers/usb/host/ohci-pci.c @@ -277,21 +277,24 @@ static const struct ohci_driver_overrides pci_overrides __initconst = { static const struct pci_device_id pci_ids[] = { { /* handle any USB OHCI controller */ PCI_DEVICE_CLASS(PCI_CLASS_SERIAL_USB_OHCI, ~0), - .driver_data = (unsigned long) &ohci_pci_hc_driver, }, { /* The device in the ConneXT I/O hub has no class reg */ PCI_VDEVICE(STMICRO, PCI_DEVICE_ID_STMICRO_USB_OHCI), - .driver_data = (unsigned long) &ohci_pci_hc_driver, }, { /* end: all zeroes */ } }; MODULE_DEVICE_TABLE (pci, pci_ids); +static int ohci_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) +{ + return usb_hcd_pci_probe(dev, id, &ohci_pci_hc_driver); +} + /* pci driver glue; this is a "new style" PCI driver module */ static struct pci_driver ohci_pci_driver = { .name = hcd_name, .id_table = pci_ids, - .probe = usb_hcd_pci_probe, + .probe = ohci_pci_probe, .remove = usb_hcd_pci_remove, .shutdown = usb_hcd_pci_shutdown, diff --git a/drivers/usb/host/uhci-pci.c b/drivers/usb/host/uhci-pci.c index 957c87efc746..9b88745d247f 100644 --- a/drivers/usb/host/uhci-pci.c +++ b/drivers/usb/host/uhci-pci.c @@ -287,17 +287,21 @@ static const struct hc_driver uhci_driver = { static const struct pci_device_id uhci_pci_ids[] = { { /* handle any USB UHCI controller */ PCI_DEVICE_CLASS(PCI_CLASS_SERIAL_USB_UHCI, ~0), - .driver_data = (unsigned long) &uhci_driver, }, { /* end: all zeroes */ } }; MODULE_DEVICE_TABLE(pci, uhci_pci_ids); +static int uhci_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) +{ + return usb_hcd_pci_probe(dev, id, &uhci_driver); +} + static struct pci_driver uhci_pci_driver = { .name = hcd_name, .id_table = uhci_pci_ids, - .probe = usb_hcd_pci_probe, + .probe = uhci_pci_probe, .remove = usb_hcd_pci_remove, .shutdown = uhci_shutdown, diff --git a/drivers/usb/host/xhci-pci.c b/drivers/usb/host/xhci-pci.c index 766b74723e64..b6c2f5c530e3 100644 --- a/drivers/usb/host/xhci-pci.c +++ b/drivers/usb/host/xhci-pci.c @@ -327,11 +327,8 @@ static int xhci_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) { int retval; struct xhci_hcd *xhci; - struct hc_driver *driver; struct usb_hcd *hcd; - driver = (struct hc_driver *)id->driver_data; - /* Prevent runtime suspending between USB-2 and USB-3 initialization */ pm_runtime_get_noresume(&dev->dev); @@ -341,7 +338,7 @@ static int xhci_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) * to say USB 2.0, but I'm not sure what the implications would be in * the other parts of the HCD code. */ - retval = usb_hcd_pci_probe(dev, id); + retval = usb_hcd_pci_probe(dev, id, &xhci_pci_hc_driver); if (retval) goto put_runtime_pm; @@ -349,8 +346,8 @@ static int xhci_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) /* USB 2.0 roothub is stored in the PCI device now. */ hcd = dev_get_drvdata(&dev->dev); xhci = hcd_to_xhci(hcd); - xhci->shared_hcd = usb_create_shared_hcd(driver, &dev->dev, - pci_name(dev), hcd); + xhci->shared_hcd = usb_create_shared_hcd(&xhci_pci_hc_driver, &dev->dev, + pci_name(dev), hcd); if (!xhci->shared_hcd) { retval = -ENOMEM; goto dealloc_usb2_hcd; @@ -544,10 +541,9 @@ static void xhci_pci_shutdown(struct usb_hcd *hcd) /*-------------------------------------------------------------------------*/ /* PCI driver selection metadata; PCI hotplugging uses this */ -static const struct pci_device_id pci_ids[] = { { +static const struct pci_device_id pci_ids[] = { /* handle any USB 3.0 xHCI controller */ - PCI_DEVICE_CLASS(PCI_CLASS_SERIAL_USB_XHCI, ~0), - .driver_data = (unsigned long) &xhci_pci_hc_driver, + { PCI_DEVICE_CLASS(PCI_CLASS_SERIAL_USB_XHCI, ~0), }, { /* end: all zeroes */ } }; diff --git a/include/linux/usb/hcd.h b/include/linux/usb/hcd.h index e12105ed3834..3dbb42c637c1 100644 --- a/include/linux/usb/hcd.h +++ b/include/linux/usb/hcd.h @@ -479,7 +479,8 @@ extern void usb_hcd_platform_shutdown(struct platform_device *dev); struct pci_dev; struct pci_device_id; extern int usb_hcd_pci_probe(struct pci_dev *dev, - const struct pci_device_id *id); + const struct pci_device_id *id, + const struct hc_driver *driver); extern void usb_hcd_pci_remove(struct pci_dev *dev); extern void usb_hcd_pci_shutdown(struct pci_dev *dev); -- 2.25.1 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH v9 2/5] usb: renesas-xhci: Add the renesas xhci driver 2020-04-14 16:41 [PATCH v9 0/5] usb: xhci: Add support for Renesas USB controllers Vinod Koul 2020-04-14 16:41 ` [PATCH v9 1/5] usb: hci: add hc_driver as argument for usb_hcd_pci_probe Vinod Koul @ 2020-04-14 16:41 ` Vinod Koul 2020-04-25 9:43 ` kbuild test robot 2020-04-14 16:41 ` [PATCH v9 3/5] usb: xhci: Add support for Renesas controller with memory Vinod Koul ` (4 subsequent siblings) 6 siblings, 1 reply; 11+ messages in thread From: Vinod Koul @ 2020-04-14 16:41 UTC (permalink / raw) To: Mathias Nyman, Greg Kroah-Hartman Cc: linux-arm-msm, Bjorn Andersson, Christian Lamparter, Yoshihiro Shimoda, John Stultz, Alan Stern, Andreas Böhler, linux-usb, linux-kernel, Vinod Koul From: Christian Lamparter <chunkeey@googlemail.com> This add a new driver for renesas xhci which is basically a firmware loader for uPD720201 and uPD720202 w/o ROM. It uses xhci-pci driver for most of the work. This is added in Makefile before the xhci-pci driver so that it first get probed for renesas devices before the xhci-pci driver has a chance to claim any device. This patch adds a firmware loader for the uPD720201K8-711-BAC-A and uPD720202K8-711-BAA-A variant. Both of these chips are listed in Renesas' R19UH0078EJ0500 Rev.5.00 "User's Manual: Hardware" as devices which need the firmware loader on page 2 in order to work as they "do not support the External ROM". The "Firmware Download Sequence" is describe in chapter "7.1 FW Download Interface" R19UH0078EJ0500 Rev.5.00 page 131. The firmware "K2013080.mem" is available from a USB3.0 Host to PCIe Adapter (PP2U-E card) "Firmware download" archive. An alternative version can be sourced from Netgear's WNDR4700 GPL archives. The release notes of the PP2U-E's "Firmware Download" ver 2.0.1.3 (2012-06-15) state that the firmware is for the following devices: - uPD720201 ES 2.0 sample whose revision ID is 2. - uPD720201 ES 2.1 sample & CS sample & Mass product, ID is 3. - uPD720202 ES 2.0 sample & CS sample & Mass product, ID is 2. Cc: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com> Signed-off-by: Christian Lamparter <chunkeey@googlemail.com> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> [vkoul: fixed comments: used macros for timeout count and delay removed renesas_fw_alive_check cleaned renesas_fw_callback removed recursion for renesas_fw_download add register defines and field names move to a separate file make fw loader as sync probe so that we execute in probe and prevent race] Signed-off-by: Vinod Koul <vkoul@kernel.org> --- drivers/usb/host/Makefile | 3 +- drivers/usb/host/xhci-pci-renesas.c | 366 ++++++++++++++++++++++++++++ drivers/usb/host/xhci-pci.h | 16 ++ 3 files changed, 384 insertions(+), 1 deletion(-) create mode 100644 drivers/usb/host/xhci-pci-renesas.c create mode 100644 drivers/usb/host/xhci-pci.h diff --git a/drivers/usb/host/Makefile b/drivers/usb/host/Makefile index b191361257cc..c3a79f626393 100644 --- a/drivers/usb/host/Makefile +++ b/drivers/usb/host/Makefile @@ -70,7 +70,8 @@ obj-$(CONFIG_USB_OHCI_HCD_DAVINCI) += ohci-da8xx.o obj-$(CONFIG_USB_UHCI_HCD) += uhci-hcd.o obj-$(CONFIG_USB_FHCI_HCD) += fhci.o obj-$(CONFIG_USB_XHCI_HCD) += xhci-hcd.o -obj-$(CONFIG_USB_XHCI_PCI) += xhci-pci.o +usb-xhci-pci-objs := xhci-pci.o xhci-pci-renesas.o +obj-$(CONFIG_USB_XHCI_PCI) += usb-xhci-pci.o obj-$(CONFIG_USB_XHCI_PLATFORM) += xhci-plat-hcd.o obj-$(CONFIG_USB_XHCI_HISTB) += xhci-histb.o obj-$(CONFIG_USB_XHCI_MTK) += xhci-mtk.o diff --git a/drivers/usb/host/xhci-pci-renesas.c b/drivers/usb/host/xhci-pci-renesas.c new file mode 100644 index 000000000000..4be838b05f06 --- /dev/null +++ b/drivers/usb/host/xhci-pci-renesas.c @@ -0,0 +1,366 @@ +// SPDX-License-Identifier: GPL-2.0 +/* Copyright (C) 2019-2020 Linaro Limited */ + +#include <linux/acpi.h> +#include <linux/firmware.h> +#include <linux/module.h> +#include <linux/pci.h> +#include <linux/slab.h> +#include <linux/unaligned/access_ok.h> + +#include "xhci.h" +#include "xhci-trace.h" +#include "xhci-pci.h" + +#define RENESAS_FW_VERSION 0x6C +#define RENESAS_ROM_CONFIG 0xF0 +#define RENESAS_FW_STATUS 0xF4 +#define RENESAS_FW_STATUS_MSB 0xF5 +#define RENESAS_ROM_STATUS 0xF6 +#define RENESAS_ROM_STATUS_MSB 0xF7 +#define RENESAS_DATA0 0xF8 +#define RENESAS_DATA1 0xFC + +#define RENESAS_FW_VERSION_FIELD GENMASK(23, 7) +#define RENESAS_FW_VERSION_OFFSET 8 + +#define RENESAS_FW_STATUS_DOWNLOAD_ENABLE BIT(0) +#define RENESAS_FW_STATUS_LOCK BIT(1) +#define RENESAS_FW_STATUS_RESULT GENMASK(6, 4) + #define RENESAS_FW_STATUS_INVALID 0 + #define RENESAS_FW_STATUS_SUCCESS BIT(4) + #define RENESAS_FW_STATUS_ERROR BIT(5) +#define RENESAS_FW_STATUS_SET_DATA0 BIT(8) +#define RENESAS_FW_STATUS_SET_DATA1 BIT(9) + +#define RENESAS_ROM_STATUS_ACCESS BIT(0) +#define RENESAS_ROM_STATUS_ERASE BIT(1) +#define RENESAS_ROM_STATUS_RELOAD BIT(2) +#define RENESAS_ROM_STATUS_RESULT GENMASK(6, 4) + #define RENESAS_ROM_STATUS_NO_RESULT 0 + #define RENESAS_ROM_STATUS_SUCCESS BIT(4) + #define RENESAS_ROM_STATUS_ERROR BIT(5) +#define RENESAS_ROM_STATUS_SET_DATA0 BIT(8) +#define RENESAS_ROM_STATUS_SET_DATA1 BIT(9) +#define RENESAS_ROM_STATUS_ROM_EXISTS BIT(15) + +#define RENESAS_ROM_ERASE_MAGIC 0x5A65726F +#define RENESAS_ROM_WRITE_MAGIC 0x53524F4D + +#define RENESAS_RETRY 10000 +#define RENESAS_DELAY 10 + +static int renesas_fw_download_image(struct pci_dev *dev, + const u32 *fw, + size_t step) +{ + size_t i; + int err; + u8 fw_status; + bool data0_or_data1; + + /* + * The hardware does alternate between two 32-bit pages. + * (This is because each row of the firmware is 8 bytes). + * + * for even steps we use DATA0, for odd steps DATA1. + */ + data0_or_data1 = (step & 1) == 1; + + /* step+1. Read "Set DATAX" and confirm it is cleared. */ + for (i = 0; i < RENESAS_RETRY; i++) { + err = pci_read_config_byte(dev, RENESAS_FW_STATUS_MSB, + &fw_status); + if (err) + return pcibios_err_to_errno(err); + if (!(fw_status & BIT(data0_or_data1))) + break; + + udelay(RENESAS_DELAY); + } + if (i == RENESAS_RETRY) { + dev_err(&dev->dev, "FW Load timedout"); + return -ETIMEDOUT; + } + + /* + * step+2. Write FW data to "DATAX". + * "LSB is left" => force little endian + */ + err = pci_write_config_dword(dev, data0_or_data1 ? + RENESAS_DATA1 : RENESAS_DATA0, + (__force u32)cpu_to_le32(fw[step])); + if (err) + return pcibios_err_to_errno(err); + + udelay(100); + + /* step+3. Set "Set DATAX". */ + err = pci_write_config_byte(dev, RENESAS_FW_STATUS_MSB, + BIT(data0_or_data1)); + if (err) + return pcibios_err_to_errno(err); + + return 0; +} + +static int renesas_fw_verify(const void *fw_data, + size_t length) +{ + u16 fw_version_pointer; + u16 fw_version; + + /* + * The Firmware's Data Format is describe in + * "6.3 Data Format" R19UH0078EJ0500 Rev.5.00 page 124 + */ + + /* + * The bootrom chips of the big brother have sizes up to 64k, let's + * assume that's the biggest the firmware can get. + */ + if (length < 0x1000 || length >= 0x10000) { + pr_err("firmware is size %zd is not (4k - 64k).", + length); + return -EINVAL; + } + + /* The First 2 bytes are fixed value (55aa). "LSB on Left" */ + if (get_unaligned_le16(fw_data) != 0x55aa) { + pr_err("no valid firmware header found."); + return -EINVAL; + } + + /* verify the firmware version position and print it. */ + fw_version_pointer = get_unaligned_le16(fw_data + 4); + if (fw_version_pointer + 2 >= length) { + pr_err("fw ver pointer is outside of the firmware image"); + return -EINVAL; + } + + fw_version = get_unaligned_le16(fw_data + fw_version_pointer); + pr_err("got firmware version: %02x.", fw_version); + + return 0; +} + +static int renesas_fw_check_running(struct pci_dev *pdev) +{ + int err; + u8 fw_state; + + /* + * Test if the device is actually needing the firmware. As most + * BIOSes will initialize the device for us. If the device is + * initialized. + */ + err = pci_read_config_byte(pdev, RENESAS_FW_STATUS, &fw_state); + if (err) + return pcibios_err_to_errno(err); + + /* + * Check if "FW Download Lock" is locked. If it is and the FW is + * ready we can simply continue. If the FW is not ready, we have + * to give up. + */ + if (fw_state & RENESAS_FW_STATUS_LOCK) { + dev_dbg(&pdev->dev, "FW Download Lock is engaged."); + + if (fw_state & RENESAS_FW_STATUS_SUCCESS) + return 0; + + dev_err(&pdev->dev, + "FW Download Lock is set and FW is not ready. Giving Up."); + return -EIO; + } + + /* + * Check if "FW Download Enable" is set. If someone (us?) tampered + * with it and it can't be reset, we have to give up too... and + * ask for a forgiveness and a reboot. + */ + if (fw_state & RENESAS_FW_STATUS_DOWNLOAD_ENABLE) { + dev_err(&pdev->dev, + "FW Download Enable is stale. Giving Up (poweroff/reboot needed)."); + return -EIO; + } + + /* Otherwise, Check the "Result Code" Bits (6:4) and act accordingly */ + switch (fw_state & RENESAS_FW_STATUS_RESULT) { + case 0: /* No result yet */ + dev_dbg(&pdev->dev, "FW is not ready/loaded yet."); + + /* tell the caller, that this device needs the firmware. */ + return 1; + + case RENESAS_FW_STATUS_SUCCESS: /* Success, device should be working. */ + dev_dbg(&pdev->dev, "FW is ready."); + return 0; + + case RENESAS_FW_STATUS_ERROR: /* Error State */ + dev_err(&pdev->dev, + "hardware is in an error state. Giving up (poweroff/reboot needed)."); + return -ENODEV; + + default: /* All other states are marked as "Reserved states" */ + dev_err(&pdev->dev, + "hardware is in an invalid state %lx. Giving up (poweroff/reboot needed).", + (fw_state & RENESAS_FW_STATUS_RESULT) >> 4); + return -EINVAL; + } +} + +static int renesas_fw_download(struct pci_dev *pdev, + const struct firmware *fw) +{ + const u32 *fw_data = (const u32 *)fw->data; + size_t i; + int err; + u8 fw_status; + + /* + * For more information and the big picture: please look at the + * "Firmware Download Sequence" in "7.1 FW Download Interface" + * of R19UH0078EJ0500 Rev.5.00 page 131 + */ + + /* + * 0. Set "FW Download Enable" bit in the + * "FW Download Control & Status Register" at 0xF4 + */ + err = pci_write_config_byte(pdev, RENESAS_FW_STATUS, + RENESAS_FW_STATUS_DOWNLOAD_ENABLE); + if (err) + return pcibios_err_to_errno(err); + + /* 1 - 10 follow one step after the other. */ + for (i = 0; i < fw->size / 4; i++) { + err = renesas_fw_download_image(pdev, fw_data, i); + if (err) { + dev_err(&pdev->dev, + "Firmware Download Step %zd failed at position %zd bytes with (%d).", + i, i * 4, err); + return err; + } + } + + /* + * This sequence continues until the last data is written to + * "DATA0" or "DATA1". Naturally, we wait until "SET DATA0/1" + * is cleared by the hardware beforehand. + */ + for (i = 0; i < RENESAS_RETRY; i++) { + err = pci_read_config_byte(pdev, RENESAS_FW_STATUS_MSB, + &fw_status); + if (err) + return pcibios_err_to_errno(err); + if (!(fw_status & (BIT(0) | BIT(1)))) + break; + + udelay(RENESAS_DELAY); + } + if (i == RENESAS_RETRY) + dev_warn(&pdev->dev, "Final Firmware Download step timed out."); + + /* + * 11. After finishing writing the last data of FW, the + * System Software must clear "FW Download Enable" + */ + err = pci_write_config_byte(pdev, RENESAS_FW_STATUS, 0); + if (err) + return pcibios_err_to_errno(err); + + /* 12. Read "Result Code" and confirm it is good. */ + for (i = 0; i < RENESAS_RETRY; i++) { + err = pci_read_config_byte(pdev, RENESAS_FW_STATUS, &fw_status); + if (err) + return pcibios_err_to_errno(err); + if (fw_status & RENESAS_FW_STATUS_SUCCESS) + break; + + udelay(RENESAS_DELAY); + } + if (i == RENESAS_RETRY) { + /* Timed out / Error - let's see if we can fix this */ + err = renesas_fw_check_running(pdev); + switch (err) { + case 0: /* + * we shouldn't end up here. + * maybe it took a little bit longer. + * But all should be well? + */ + break; + + case 1: /* (No result yet! */ + dev_err(&pdev->dev, "FW Load timedout"); + return -ETIMEDOUT; + + default: + return err; + } + } + + return 0; +} + +static int renesas_load_fw(struct pci_dev *pdev, const struct firmware *fw) +{ + int err = 0; + + err = renesas_fw_download(pdev, fw); + if (err) + dev_dbg(&pdev->dev, "firmware failed to download (%d).", err); + return err; +} + +static int renesas_xhci_check_request_fw(struct pci_dev *pdev, + const struct pci_device_id *id) +{ + struct xhci_driver_data *driver_data = + (struct xhci_driver_data *)id->driver_data; + const char *fw_name = driver_data->firmware; + const struct firmware *fw; + int err; + + err = renesas_fw_check_running(pdev); + /* Continue ahead, if the firmware is already running. */ + if (err == 0) + return 0; + + if (err != 1) + return err; + + pci_dev_get(pdev); + err = request_firmware(&fw, fw_name, &pdev->dev); + pci_dev_put(pdev); + if (err) { + dev_err(&pdev->dev, "request_firmware failed: %d\n", err); + return err; + } + + err = renesas_fw_verify(fw->data, fw->size); + if (err) + goto exit; + + err = renesas_load_fw(pdev, fw); +exit: + release_firmware(fw); + return err; +} + +int renesas_xhci_pci_probe(struct pci_dev *dev, + const struct pci_device_id *id) +{ + return renesas_xhci_check_request_fw(dev, id); +} + +int renesas_xhci_pci_remove(struct pci_dev *dev) +{ + if (renesas_fw_check_running(dev)) { + /* + * bail out early, if this was a renesas device w/o FW. + */ + return -EIO; + } + return 0; +} diff --git a/drivers/usb/host/xhci-pci.h b/drivers/usb/host/xhci-pci.h new file mode 100644 index 000000000000..623f925ee4aa --- /dev/null +++ b/drivers/usb/host/xhci-pci.h @@ -0,0 +1,16 @@ +/* SPDX-License-Identifier: GPL-2.0 */ +/* Copyright (C) 2019-2020 Linaro Limited */ + +#ifndef XHCI_PCI_H +#define XHCI_PCI_H + +int renesas_xhci_pci_probe(struct pci_dev *dev, + const struct pci_device_id *id); +int renesas_xhci_pci_remove(struct pci_dev *dev); + +struct xhci_driver_data { + u64 quirks; + const char *firmware; +}; + +#endif -- 2.25.1 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH v9 2/5] usb: renesas-xhci: Add the renesas xhci driver 2020-04-14 16:41 ` [PATCH v9 2/5] usb: renesas-xhci: Add the renesas xhci driver Vinod Koul @ 2020-04-25 9:43 ` kbuild test robot 0 siblings, 0 replies; 11+ messages in thread From: kbuild test robot @ 2020-04-25 9:43 UTC (permalink / raw) To: Vinod Koul, Mathias Nyman, Greg Kroah-Hartman Cc: kbuild-all, linux-arm-msm, Bjorn Andersson, Christian Lamparter, Yoshihiro Shimoda, John Stultz, Alan Stern, Andreas Böhler, linux-usb, linux-kernel [-- Attachment #1: Type: text/plain, Size: 23330 bytes --] Hi Vinod, I love your patch! Yet something to improve: [auto build test ERROR on usb/usb-testing] [also build test ERROR on v5.7-rc2 next-20200424] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system. BTW, we also suggest to use '--base' option to specify the base tree in git format-patch, please see https://stackoverflow.com/a/37406982] url: https://github.com/0day-ci/linux/commits/Vinod-Koul/usb-xhci-Add-support-for-Renesas-USB-controllers/20200415-044339 base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing config: ia64-allmodconfig (attached as .config) compiler: ia64-linux-gcc (GCC) 9.3.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day GCC_VERSION=9.3.0 make.cross ARCH=ia64 If you fix the issue, kindly add following tag as appropriate Reported-by: kbuild test robot <lkp@intel.com> All errors (new ones prefixed by >>): In file included from drivers/usb/host/xhci-pci-renesas.c:9: >> include/linux/unaligned/access_ok.h:8:28: error: redefinition of 'get_unaligned_le16' 8 | static __always_inline u16 get_unaligned_le16(const void *p) | ^~~~~~~~~~~~~~~~~~ In file included from arch/ia64/include/asm/unaligned.h:5, from arch/ia64/include/asm/io.h:23, from arch/ia64/include/asm/smp.h:21, from include/linux/smp.h:67, from include/linux/percpu.h:7, from include/linux/arch_topology.h:9, from include/linux/topology.h:30, from include/linux/gfp.h:9, from include/linux/xarray.h:14, from include/linux/radix-tree.h:18, from include/linux/idr.h:15, from include/linux/kernfs.h:13, from include/linux/sysfs.h:16, from include/linux/kobject.h:20, from include/linux/of.h:17, from include/linux/irqdomain.h:35, from include/linux/acpi.h:13, from drivers/usb/host/xhci-pci-renesas.c:4: include/linux/unaligned/le_struct.h:7:19: note: previous definition of 'get_unaligned_le16' was here 7 | static inline u16 get_unaligned_le16(const void *p) | ^~~~~~~~~~~~~~~~~~ In file included from drivers/usb/host/xhci-pci-renesas.c:9: >> include/linux/unaligned/access_ok.h:13:28: error: redefinition of 'get_unaligned_le32' 13 | static __always_inline u32 get_unaligned_le32(const void *p) | ^~~~~~~~~~~~~~~~~~ In file included from arch/ia64/include/asm/unaligned.h:5, from arch/ia64/include/asm/io.h:23, from arch/ia64/include/asm/smp.h:21, from include/linux/smp.h:67, from include/linux/percpu.h:7, from include/linux/arch_topology.h:9, from include/linux/topology.h:30, from include/linux/gfp.h:9, from include/linux/xarray.h:14, from include/linux/radix-tree.h:18, from include/linux/idr.h:15, from include/linux/kernfs.h:13, from include/linux/sysfs.h:16, from include/linux/kobject.h:20, from include/linux/of.h:17, from include/linux/irqdomain.h:35, from include/linux/acpi.h:13, from drivers/usb/host/xhci-pci-renesas.c:4: include/linux/unaligned/le_struct.h:12:19: note: previous definition of 'get_unaligned_le32' was here 12 | static inline u32 get_unaligned_le32(const void *p) | ^~~~~~~~~~~~~~~~~~ In file included from drivers/usb/host/xhci-pci-renesas.c:9: >> include/linux/unaligned/access_ok.h:18:28: error: redefinition of 'get_unaligned_le64' 18 | static __always_inline u64 get_unaligned_le64(const void *p) | ^~~~~~~~~~~~~~~~~~ In file included from arch/ia64/include/asm/unaligned.h:5, from arch/ia64/include/asm/io.h:23, from arch/ia64/include/asm/smp.h:21, from include/linux/smp.h:67, from include/linux/percpu.h:7, from include/linux/arch_topology.h:9, from include/linux/topology.h:30, from include/linux/gfp.h:9, from include/linux/xarray.h:14, from include/linux/radix-tree.h:18, from include/linux/idr.h:15, from include/linux/kernfs.h:13, from include/linux/sysfs.h:16, from include/linux/kobject.h:20, from include/linux/of.h:17, from include/linux/irqdomain.h:35, from include/linux/acpi.h:13, from drivers/usb/host/xhci-pci-renesas.c:4: include/linux/unaligned/le_struct.h:17:19: note: previous definition of 'get_unaligned_le64' was here 17 | static inline u64 get_unaligned_le64(const void *p) | ^~~~~~~~~~~~~~~~~~ In file included from drivers/usb/host/xhci-pci-renesas.c:9: >> include/linux/unaligned/access_ok.h:23:28: error: redefinition of 'get_unaligned_be16' 23 | static __always_inline u16 get_unaligned_be16(const void *p) | ^~~~~~~~~~~~~~~~~~ In file included from arch/ia64/include/asm/unaligned.h:6, from arch/ia64/include/asm/io.h:23, from arch/ia64/include/asm/smp.h:21, from include/linux/smp.h:67, from include/linux/percpu.h:7, from include/linux/arch_topology.h:9, from include/linux/topology.h:30, from include/linux/gfp.h:9, from include/linux/xarray.h:14, from include/linux/radix-tree.h:18, from include/linux/idr.h:15, from include/linux/kernfs.h:13, from include/linux/sysfs.h:16, from include/linux/kobject.h:20, from include/linux/of.h:17, from include/linux/irqdomain.h:35, from include/linux/acpi.h:13, from drivers/usb/host/xhci-pci-renesas.c:4: include/linux/unaligned/be_byteshift.h:41:19: note: previous definition of 'get_unaligned_be16' was here 41 | static inline u16 get_unaligned_be16(const void *p) | ^~~~~~~~~~~~~~~~~~ In file included from drivers/usb/host/xhci-pci-renesas.c:9: >> include/linux/unaligned/access_ok.h:28:28: error: redefinition of 'get_unaligned_be32' 28 | static __always_inline u32 get_unaligned_be32(const void *p) | ^~~~~~~~~~~~~~~~~~ In file included from arch/ia64/include/asm/unaligned.h:6, from arch/ia64/include/asm/io.h:23, from arch/ia64/include/asm/smp.h:21, from include/linux/smp.h:67, from include/linux/percpu.h:7, from include/linux/arch_topology.h:9, from include/linux/topology.h:30, from include/linux/gfp.h:9, from include/linux/xarray.h:14, from include/linux/radix-tree.h:18, from include/linux/idr.h:15, from include/linux/kernfs.h:13, from include/linux/sysfs.h:16, from include/linux/kobject.h:20, from include/linux/of.h:17, from include/linux/irqdomain.h:35, from include/linux/acpi.h:13, from drivers/usb/host/xhci-pci-renesas.c:4: include/linux/unaligned/be_byteshift.h:46:19: note: previous definition of 'get_unaligned_be32' was here 46 | static inline u32 get_unaligned_be32(const void *p) | ^~~~~~~~~~~~~~~~~~ In file included from drivers/usb/host/xhci-pci-renesas.c:9: >> include/linux/unaligned/access_ok.h:33:28: error: redefinition of 'get_unaligned_be64' 33 | static __always_inline u64 get_unaligned_be64(const void *p) | ^~~~~~~~~~~~~~~~~~ In file included from arch/ia64/include/asm/unaligned.h:6, from arch/ia64/include/asm/io.h:23, from arch/ia64/include/asm/smp.h:21, from include/linux/smp.h:67, from include/linux/percpu.h:7, from include/linux/arch_topology.h:9, from include/linux/topology.h:30, from include/linux/gfp.h:9, from include/linux/xarray.h:14, from include/linux/radix-tree.h:18, from include/linux/idr.h:15, from include/linux/kernfs.h:13, from include/linux/sysfs.h:16, from include/linux/kobject.h:20, from include/linux/of.h:17, from include/linux/irqdomain.h:35, from include/linux/acpi.h:13, from drivers/usb/host/xhci-pci-renesas.c:4: include/linux/unaligned/be_byteshift.h:51:19: note: previous definition of 'get_unaligned_be64' was here 51 | static inline u64 get_unaligned_be64(const void *p) | ^~~~~~~~~~~~~~~~~~ In file included from drivers/usb/host/xhci-pci-renesas.c:9: >> include/linux/unaligned/access_ok.h:38:29: error: redefinition of 'put_unaligned_le16' 38 | static __always_inline void put_unaligned_le16(u16 val, void *p) | ^~~~~~~~~~~~~~~~~~ In file included from arch/ia64/include/asm/unaligned.h:5, from arch/ia64/include/asm/io.h:23, from arch/ia64/include/asm/smp.h:21, from include/linux/smp.h:67, from include/linux/percpu.h:7, from include/linux/arch_topology.h:9, from include/linux/topology.h:30, from include/linux/gfp.h:9, from include/linux/xarray.h:14, from include/linux/radix-tree.h:18, from include/linux/idr.h:15, from include/linux/kernfs.h:13, from include/linux/sysfs.h:16, from include/linux/kobject.h:20, from include/linux/of.h:17, from include/linux/irqdomain.h:35, from include/linux/acpi.h:13, from drivers/usb/host/xhci-pci-renesas.c:4: include/linux/unaligned/le_struct.h:22:20: note: previous definition of 'put_unaligned_le16' was here 22 | static inline void put_unaligned_le16(u16 val, void *p) | ^~~~~~~~~~~~~~~~~~ In file included from drivers/usb/host/xhci-pci-renesas.c:9: >> include/linux/unaligned/access_ok.h:43:29: error: redefinition of 'put_unaligned_le32' 43 | static __always_inline void put_unaligned_le32(u32 val, void *p) | ^~~~~~~~~~~~~~~~~~ In file included from arch/ia64/include/asm/unaligned.h:5, from arch/ia64/include/asm/io.h:23, from arch/ia64/include/asm/smp.h:21, from include/linux/smp.h:67, from include/linux/percpu.h:7, from include/linux/arch_topology.h:9, from include/linux/topology.h:30, from include/linux/gfp.h:9, from include/linux/xarray.h:14, from include/linux/radix-tree.h:18, from include/linux/idr.h:15, from include/linux/kernfs.h:13, from include/linux/sysfs.h:16, from include/linux/kobject.h:20, from include/linux/of.h:17, from include/linux/irqdomain.h:35, from include/linux/acpi.h:13, from drivers/usb/host/xhci-pci-renesas.c:4: include/linux/unaligned/le_struct.h:27:20: note: previous definition of 'put_unaligned_le32' was here 27 | static inline void put_unaligned_le32(u32 val, void *p) | ^~~~~~~~~~~~~~~~~~ In file included from drivers/usb/host/xhci-pci-renesas.c:9: >> include/linux/unaligned/access_ok.h:48:29: error: redefinition of 'put_unaligned_le64' 48 | static __always_inline void put_unaligned_le64(u64 val, void *p) | ^~~~~~~~~~~~~~~~~~ In file included from arch/ia64/include/asm/unaligned.h:5, from arch/ia64/include/asm/io.h:23, from arch/ia64/include/asm/smp.h:21, from include/linux/smp.h:67, from include/linux/percpu.h:7, from include/linux/arch_topology.h:9, from include/linux/topology.h:30, from include/linux/gfp.h:9, from include/linux/xarray.h:14, from include/linux/radix-tree.h:18, from include/linux/idr.h:15, from include/linux/kernfs.h:13, from include/linux/sysfs.h:16, from include/linux/kobject.h:20, from include/linux/of.h:17, from include/linux/irqdomain.h:35, from include/linux/acpi.h:13, from drivers/usb/host/xhci-pci-renesas.c:4: include/linux/unaligned/le_struct.h:32:20: note: previous definition of 'put_unaligned_le64' was here 32 | static inline void put_unaligned_le64(u64 val, void *p) | ^~~~~~~~~~~~~~~~~~ In file included from drivers/usb/host/xhci-pci-renesas.c:9: >> include/linux/unaligned/access_ok.h:53:29: error: redefinition of 'put_unaligned_be16' 53 | static __always_inline void put_unaligned_be16(u16 val, void *p) | ^~~~~~~~~~~~~~~~~~ In file included from arch/ia64/include/asm/unaligned.h:6, from arch/ia64/include/asm/io.h:23, from arch/ia64/include/asm/smp.h:21, from include/linux/smp.h:67, from include/linux/percpu.h:7, from include/linux/arch_topology.h:9, from include/linux/topology.h:30, from include/linux/gfp.h:9, from include/linux/xarray.h:14, from include/linux/radix-tree.h:18, from include/linux/idr.h:15, from include/linux/kernfs.h:13, from include/linux/sysfs.h:16, from include/linux/kobject.h:20, from include/linux/of.h:17, from include/linux/irqdomain.h:35, from include/linux/acpi.h:13, from drivers/usb/host/xhci-pci-renesas.c:4: include/linux/unaligned/be_byteshift.h:56:20: note: previous definition of 'put_unaligned_be16' was here 56 | static inline void put_unaligned_be16(u16 val, void *p) | ^~~~~~~~~~~~~~~~~~ In file included from drivers/usb/host/xhci-pci-renesas.c:9: >> include/linux/unaligned/access_ok.h:58:29: error: redefinition of 'put_unaligned_be32' 58 | static __always_inline void put_unaligned_be32(u32 val, void *p) | ^~~~~~~~~~~~~~~~~~ In file included from arch/ia64/include/asm/unaligned.h:6, from arch/ia64/include/asm/io.h:23, from arch/ia64/include/asm/smp.h:21, from include/linux/smp.h:67, from include/linux/percpu.h:7, from include/linux/arch_topology.h:9, from include/linux/topology.h:30, from include/linux/gfp.h:9, from include/linux/xarray.h:14, from include/linux/radix-tree.h:18, from include/linux/idr.h:15, from include/linux/kernfs.h:13, from include/linux/sysfs.h:16, from include/linux/kobject.h:20, from include/linux/of.h:17, from include/linux/irqdomain.h:35, from include/linux/acpi.h:13, from drivers/usb/host/xhci-pci-renesas.c:4: include/linux/unaligned/be_byteshift.h:61:20: note: previous definition of 'put_unaligned_be32' was here 61 | static inline void put_unaligned_be32(u32 val, void *p) | ^~~~~~~~~~~~~~~~~~ In file included from drivers/usb/host/xhci-pci-renesas.c:9: >> include/linux/unaligned/access_ok.h:63:29: error: redefinition of 'put_unaligned_be64' 63 | static __always_inline void put_unaligned_be64(u64 val, void *p) | ^~~~~~~~~~~~~~~~~~ In file included from arch/ia64/include/asm/unaligned.h:6, from arch/ia64/include/asm/io.h:23, from arch/ia64/include/asm/smp.h:21, from include/linux/smp.h:67, from include/linux/percpu.h:7, from include/linux/arch_topology.h:9, from include/linux/topology.h:30, from include/linux/gfp.h:9, from include/linux/xarray.h:14, from include/linux/radix-tree.h:18, from include/linux/idr.h:15, from include/linux/kernfs.h:13, from include/linux/sysfs.h:16, from include/linux/kobject.h:20, from include/linux/of.h:17, from include/linux/irqdomain.h:35, from include/linux/acpi.h:13, from drivers/usb/host/xhci-pci-renesas.c:4: include/linux/unaligned/be_byteshift.h:66:20: note: previous definition of 'put_unaligned_be64' was here 66 | static inline void put_unaligned_be64(u64 val, void *p) | ^~~~~~~~~~~~~~~~~~ vim +/get_unaligned_le16 +8 include/linux/unaligned/access_ok.h 064106a91be5e7 Harvey Harrison 2008-04-29 7 e3bde9568d992c Denys Vlasenko 2016-03-17 @8 static __always_inline u16 get_unaligned_le16(const void *p) 064106a91be5e7 Harvey Harrison 2008-04-29 9 { 064106a91be5e7 Harvey Harrison 2008-04-29 10 return le16_to_cpup((__le16 *)p); 064106a91be5e7 Harvey Harrison 2008-04-29 11 } 064106a91be5e7 Harvey Harrison 2008-04-29 12 e3bde9568d992c Denys Vlasenko 2016-03-17 @13 static __always_inline u32 get_unaligned_le32(const void *p) 064106a91be5e7 Harvey Harrison 2008-04-29 14 { 064106a91be5e7 Harvey Harrison 2008-04-29 15 return le32_to_cpup((__le32 *)p); 064106a91be5e7 Harvey Harrison 2008-04-29 16 } 064106a91be5e7 Harvey Harrison 2008-04-29 17 e3bde9568d992c Denys Vlasenko 2016-03-17 @18 static __always_inline u64 get_unaligned_le64(const void *p) 064106a91be5e7 Harvey Harrison 2008-04-29 19 { 064106a91be5e7 Harvey Harrison 2008-04-29 20 return le64_to_cpup((__le64 *)p); 064106a91be5e7 Harvey Harrison 2008-04-29 21 } 064106a91be5e7 Harvey Harrison 2008-04-29 22 e3bde9568d992c Denys Vlasenko 2016-03-17 @23 static __always_inline u16 get_unaligned_be16(const void *p) 064106a91be5e7 Harvey Harrison 2008-04-29 24 { 064106a91be5e7 Harvey Harrison 2008-04-29 25 return be16_to_cpup((__be16 *)p); 064106a91be5e7 Harvey Harrison 2008-04-29 26 } 064106a91be5e7 Harvey Harrison 2008-04-29 27 e3bde9568d992c Denys Vlasenko 2016-03-17 @28 static __always_inline u32 get_unaligned_be32(const void *p) 064106a91be5e7 Harvey Harrison 2008-04-29 29 { 064106a91be5e7 Harvey Harrison 2008-04-29 30 return be32_to_cpup((__be32 *)p); 064106a91be5e7 Harvey Harrison 2008-04-29 31 } 064106a91be5e7 Harvey Harrison 2008-04-29 32 e3bde9568d992c Denys Vlasenko 2016-03-17 @33 static __always_inline u64 get_unaligned_be64(const void *p) 064106a91be5e7 Harvey Harrison 2008-04-29 34 { 064106a91be5e7 Harvey Harrison 2008-04-29 35 return be64_to_cpup((__be64 *)p); 064106a91be5e7 Harvey Harrison 2008-04-29 36 } 064106a91be5e7 Harvey Harrison 2008-04-29 37 e3bde9568d992c Denys Vlasenko 2016-03-17 @38 static __always_inline void put_unaligned_le16(u16 val, void *p) 064106a91be5e7 Harvey Harrison 2008-04-29 39 { 064106a91be5e7 Harvey Harrison 2008-04-29 40 *((__le16 *)p) = cpu_to_le16(val); 064106a91be5e7 Harvey Harrison 2008-04-29 41 } 064106a91be5e7 Harvey Harrison 2008-04-29 42 e3bde9568d992c Denys Vlasenko 2016-03-17 @43 static __always_inline void put_unaligned_le32(u32 val, void *p) 064106a91be5e7 Harvey Harrison 2008-04-29 44 { 064106a91be5e7 Harvey Harrison 2008-04-29 45 *((__le32 *)p) = cpu_to_le32(val); 064106a91be5e7 Harvey Harrison 2008-04-29 46 } 064106a91be5e7 Harvey Harrison 2008-04-29 47 e3bde9568d992c Denys Vlasenko 2016-03-17 @48 static __always_inline void put_unaligned_le64(u64 val, void *p) 064106a91be5e7 Harvey Harrison 2008-04-29 49 { 064106a91be5e7 Harvey Harrison 2008-04-29 50 *((__le64 *)p) = cpu_to_le64(val); 064106a91be5e7 Harvey Harrison 2008-04-29 51 } 064106a91be5e7 Harvey Harrison 2008-04-29 52 e3bde9568d992c Denys Vlasenko 2016-03-17 @53 static __always_inline void put_unaligned_be16(u16 val, void *p) 064106a91be5e7 Harvey Harrison 2008-04-29 54 { 064106a91be5e7 Harvey Harrison 2008-04-29 55 *((__be16 *)p) = cpu_to_be16(val); 064106a91be5e7 Harvey Harrison 2008-04-29 56 } 064106a91be5e7 Harvey Harrison 2008-04-29 57 e3bde9568d992c Denys Vlasenko 2016-03-17 @58 static __always_inline void put_unaligned_be32(u32 val, void *p) 064106a91be5e7 Harvey Harrison 2008-04-29 59 { 064106a91be5e7 Harvey Harrison 2008-04-29 60 *((__be32 *)p) = cpu_to_be32(val); 064106a91be5e7 Harvey Harrison 2008-04-29 61 } 064106a91be5e7 Harvey Harrison 2008-04-29 62 e3bde9568d992c Denys Vlasenko 2016-03-17 @63 static __always_inline void put_unaligned_be64(u64 val, void *p) 064106a91be5e7 Harvey Harrison 2008-04-29 64 { 064106a91be5e7 Harvey Harrison 2008-04-29 65 *((__be64 *)p) = cpu_to_be64(val); 064106a91be5e7 Harvey Harrison 2008-04-29 66 } 064106a91be5e7 Harvey Harrison 2008-04-29 67 :::::: The code at line 8 was first introduced by commit :::::: e3bde9568d992c5f985e6e30731a5f9f9bef7b13 include/linux/unaligned: force inlining of byteswap operations :::::: TO: Denys Vlasenko <dvlasenk@redhat.com> :::::: CC: Linus Torvalds <torvalds@linux-foundation.org> --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 56806 bytes --] ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v9 3/5] usb: xhci: Add support for Renesas controller with memory 2020-04-14 16:41 [PATCH v9 0/5] usb: xhci: Add support for Renesas USB controllers Vinod Koul 2020-04-14 16:41 ` [PATCH v9 1/5] usb: hci: add hc_driver as argument for usb_hcd_pci_probe Vinod Koul 2020-04-14 16:41 ` [PATCH v9 2/5] usb: renesas-xhci: Add the renesas xhci driver Vinod Koul @ 2020-04-14 16:41 ` Vinod Koul 2020-04-23 14:07 ` Mathias Nyman 2020-04-14 16:41 ` [PATCH v9 4/5] usb: renesas-xhci: Add ROM loader for uPD720201 Vinod Koul ` (3 subsequent siblings) 6 siblings, 1 reply; 11+ messages in thread From: Vinod Koul @ 2020-04-14 16:41 UTC (permalink / raw) To: Mathias Nyman, Greg Kroah-Hartman Cc: linux-arm-msm, Bjorn Andersson, Vinod Koul, Yoshihiro Shimoda, Christian Lamparter, John Stultz, Alan Stern, Andreas Böhler, linux-usb, linux-kernel Some rensas controller like uPD720201 and uPD720202 need firmware to be loaded. Add these devices in table and invoke renesas firmware loader functions to check and load the firmware into device memory when required. Signed-off-by: Vinod Koul <vkoul@kernel.org> --- drivers/usb/host/xhci-pci.c | 33 +++++++++++++++++++++++++++++++++ drivers/usb/host/xhci.h | 1 + 2 files changed, 34 insertions(+) diff --git a/drivers/usb/host/xhci-pci.c b/drivers/usb/host/xhci-pci.c index b6c2f5c530e3..11521e2e1720 100644 --- a/drivers/usb/host/xhci-pci.c +++ b/drivers/usb/host/xhci-pci.c @@ -15,6 +15,7 @@ #include "xhci.h" #include "xhci-trace.h" +#include "xhci-pci.h" #define SSIC_PORT_NUM 2 #define SSIC_PORT_CFG2 0x880c @@ -328,6 +329,21 @@ static int xhci_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) int retval; struct xhci_hcd *xhci; struct usb_hcd *hcd; + struct xhci_driver_data *driver_data; + + driver_data = (struct xhci_driver_data *)id->driver_data; + + if (driver_data && driver_data->quirks & XHCI_RENESAS_FW_QUIRK) { + retval = renesas_xhci_pci_probe(dev, id); + switch (retval) { + case 0: /* fw check success, continue */ + break; + case 1: /* fw will be loaded by async load */ + return 0; + default: /* error */ + return retval; + } + } /* Prevent runtime suspending between USB-2 and USB-3 initialization */ pm_runtime_get_noresume(&dev->dev); @@ -387,6 +403,11 @@ static int xhci_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) static void xhci_pci_remove(struct pci_dev *dev) { struct xhci_hcd *xhci; + int err; + + err = renesas_xhci_pci_remove(dev); + if (err) + return; xhci = hcd_to_xhci(pci_get_drvdata(dev)); xhci->xhc_state |= XHCI_STATE_REMOVING; @@ -540,14 +561,26 @@ static void xhci_pci_shutdown(struct usb_hcd *hcd) /*-------------------------------------------------------------------------*/ +static const struct xhci_driver_data reneses_data = { + .quirks = XHCI_RENESAS_FW_QUIRK, + .firmware = "renesas_usb_fw.mem", +}; + /* PCI driver selection metadata; PCI hotplugging uses this */ static const struct pci_device_id pci_ids[] = { + { PCI_DEVICE(0x1912, 0x0014), + .driver_data = (unsigned long)&reneses_data, + }, + { PCI_DEVICE(0x1912, 0x0015), + .driver_data = (unsigned long)&reneses_data, + }, /* handle any USB 3.0 xHCI controller */ { PCI_DEVICE_CLASS(PCI_CLASS_SERIAL_USB_XHCI, ~0), }, { /* end: all zeroes */ } }; MODULE_DEVICE_TABLE(pci, pci_ids); +MODULE_FIRMWARE("renesas_usb_fw.mem"); /* pci driver glue; this is a "new style" PCI driver module */ static struct pci_driver xhci_pci_driver = { diff --git a/drivers/usb/host/xhci.h b/drivers/usb/host/xhci.h index 3289bb516201..4047363c7423 100644 --- a/drivers/usb/host/xhci.h +++ b/drivers/usb/host/xhci.h @@ -1873,6 +1873,7 @@ struct xhci_hcd { #define XHCI_DEFAULT_PM_RUNTIME_ALLOW BIT_ULL(33) #define XHCI_RESET_PLL_ON_DISCONNECT BIT_ULL(34) #define XHCI_SNPS_BROKEN_SUSPEND BIT_ULL(35) +#define XHCI_RENESAS_FW_QUIRK BIT_ULL(36) unsigned int num_active_eps; unsigned int limit_active_eps; -- 2.25.1 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH v9 3/5] usb: xhci: Add support for Renesas controller with memory 2020-04-14 16:41 ` [PATCH v9 3/5] usb: xhci: Add support for Renesas controller with memory Vinod Koul @ 2020-04-23 14:07 ` Mathias Nyman 2020-04-23 14:31 ` Vinod Koul 0 siblings, 1 reply; 11+ messages in thread From: Mathias Nyman @ 2020-04-23 14:07 UTC (permalink / raw) To: Vinod Koul, Mathias Nyman, Greg Kroah-Hartman Cc: linux-arm-msm, Bjorn Andersson, Yoshihiro Shimoda, Christian Lamparter, John Stultz, Alan Stern, Andreas Böhler, linux-usb, linux-kernel On 14.4.2020 19.41, Vinod Koul wrote: > Some rensas controller like uPD720201 and uPD720202 need firmware to be > loaded. Add these devices in table and invoke renesas firmware loader > functions to check and load the firmware into device memory when > required. > > Signed-off-by: Vinod Koul <vkoul@kernel.org> > --- > drivers/usb/host/xhci-pci.c | 33 +++++++++++++++++++++++++++++++++ > drivers/usb/host/xhci.h | 1 + > 2 files changed, 34 insertions(+) > > diff --git a/drivers/usb/host/xhci-pci.c b/drivers/usb/host/xhci-pci.c > index b6c2f5c530e3..11521e2e1720 100644 > --- a/drivers/usb/host/xhci-pci.c > +++ b/drivers/usb/host/xhci-pci.c > @@ -15,6 +15,7 @@ > > #include "xhci.h" > #include "xhci-trace.h" > +#include "xhci-pci.h" > > #define SSIC_PORT_NUM 2 > #define SSIC_PORT_CFG2 0x880c > @@ -328,6 +329,21 @@ static int xhci_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) > int retval; > struct xhci_hcd *xhci; > struct usb_hcd *hcd; > + struct xhci_driver_data *driver_data; > + > + driver_data = (struct xhci_driver_data *)id->driver_data; > + > + if (driver_data && driver_data->quirks & XHCI_RENESAS_FW_QUIRK) { > + retval = renesas_xhci_pci_probe(dev, id); > + switch (retval) { > + case 0: /* fw check success, continue */ > + break; > + case 1: /* fw will be loaded by async load */ > + return 0; This is no longer true, right? To me it looks like renesas_xhci_pci_probe() returns 0 on success, both if firmware was already running or if successfully loaded, and negative on error While changing this the function name "renesas_xhci_pci_probe()" should be changed as well. This isn't anymore a separate firmware loading driver, just a a lot of renesas firmware loading code. You could call renesas_xhci_check_request_fw() directly instead: if (driver_data && driver_data->quirks & XHCI_RENESAS_FW_QUIRK) { retval = renesas_xhci_check_request_fw(dev, id); if (retval) return retval; } -Mathias ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v9 3/5] usb: xhci: Add support for Renesas controller with memory 2020-04-23 14:07 ` Mathias Nyman @ 2020-04-23 14:31 ` Vinod Koul 0 siblings, 0 replies; 11+ messages in thread From: Vinod Koul @ 2020-04-23 14:31 UTC (permalink / raw) To: Mathias Nyman Cc: Mathias Nyman, Greg Kroah-Hartman, linux-arm-msm, Bjorn Andersson, Yoshihiro Shimoda, Christian Lamparter, John Stultz, Alan Stern, Andreas Böhler, linux-usb, linux-kernel On 23-04-20, 17:07, Mathias Nyman wrote: > On 14.4.2020 19.41, Vinod Koul wrote: > > Some rensas controller like uPD720201 and uPD720202 need firmware to be > > loaded. Add these devices in table and invoke renesas firmware loader > > functions to check and load the firmware into device memory when > > required. > > > > Signed-off-by: Vinod Koul <vkoul@kernel.org> > > --- > > drivers/usb/host/xhci-pci.c | 33 +++++++++++++++++++++++++++++++++ > > drivers/usb/host/xhci.h | 1 + > > 2 files changed, 34 insertions(+) > > > > diff --git a/drivers/usb/host/xhci-pci.c b/drivers/usb/host/xhci-pci.c > > index b6c2f5c530e3..11521e2e1720 100644 > > --- a/drivers/usb/host/xhci-pci.c > > +++ b/drivers/usb/host/xhci-pci.c > > @@ -15,6 +15,7 @@ > > > > #include "xhci.h" > > #include "xhci-trace.h" > > +#include "xhci-pci.h" > > > > #define SSIC_PORT_NUM 2 > > #define SSIC_PORT_CFG2 0x880c > > @@ -328,6 +329,21 @@ static int xhci_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) > > int retval; > > struct xhci_hcd *xhci; > > struct usb_hcd *hcd; > > + struct xhci_driver_data *driver_data; > > + > > + driver_data = (struct xhci_driver_data *)id->driver_data; > > + > > + if (driver_data && driver_data->quirks & XHCI_RENESAS_FW_QUIRK) { > > + retval = renesas_xhci_pci_probe(dev, id); > > + switch (retval) { > > + case 0: /* fw check success, continue */ > > + break; > > + case 1: /* fw will be loaded by async load */ > > + return 0; > > This is no longer true, right? Correct. > To me it looks like renesas_xhci_pci_probe() returns 0 on success, both if > firmware was already running or if successfully loaded, and negative on error Yes now it does that and I will update this part.. > > While changing this the function name "renesas_xhci_pci_probe()" should be > changed as well. This isn't anymore a separate firmware loading driver, just a > a lot of renesas firmware loading code. > > You could call renesas_xhci_check_request_fw() directly instead: > > if (driver_data && driver_data->quirks & XHCI_RENESAS_FW_QUIRK) { > retval = renesas_xhci_check_request_fw(dev, id); > if (retval) > return retval; > } Yes I can remove this layer and directly invoke the internal function.. Thanks for the comments -- ~Vinod ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v9 4/5] usb: renesas-xhci: Add ROM loader for uPD720201 2020-04-14 16:41 [PATCH v9 0/5] usb: xhci: Add support for Renesas USB controllers Vinod Koul ` (2 preceding siblings ...) 2020-04-14 16:41 ` [PATCH v9 3/5] usb: xhci: Add support for Renesas controller with memory Vinod Koul @ 2020-04-14 16:41 ` Vinod Koul 2020-04-14 16:41 ` [PATCH v9 5/5] usb: xhci: provide a debugfs hook for erasing rom Vinod Koul ` (2 subsequent siblings) 6 siblings, 0 replies; 11+ messages in thread From: Vinod Koul @ 2020-04-14 16:41 UTC (permalink / raw) To: Mathias Nyman, Greg Kroah-Hartman Cc: linux-arm-msm, Bjorn Andersson, Vinod Koul, Yoshihiro Shimoda, Christian Lamparter, John Stultz, Alan Stern, Andreas Böhler, linux-usb, linux-kernel uPD720201 supports ROM and allows software to program the ROM and boot from it. Add support for detecting if ROM is present, if so load the ROM if not programmed earlier. Signed-off-by: Vinod Koul <vkoul@kernel.org> Cc: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com> Cc: Christian Lamparter <chunkeey@googlemail.com> --- drivers/usb/host/xhci-pci-renesas.c | 342 +++++++++++++++++++++++++++- 1 file changed, 341 insertions(+), 1 deletion(-) diff --git a/drivers/usb/host/xhci-pci-renesas.c b/drivers/usb/host/xhci-pci-renesas.c index 4be838b05f06..b321d55f1e24 100644 --- a/drivers/usb/host/xhci-pci-renesas.c +++ b/drivers/usb/host/xhci-pci-renesas.c @@ -50,6 +50,22 @@ #define RENESAS_RETRY 10000 #define RENESAS_DELAY 10 +#define ROM_VALID_01 0x2013 +#define ROM_VALID_02 0x2026 + +static int renesas_verify_fw_version(struct pci_dev *pdev, u32 version) +{ + switch (version) { + case ROM_VALID_01: + case ROM_VALID_02: + return 0; + default: + dev_err(&pdev->dev, "FW has invalid version :%d\n", version); + return 1; + } + return -EINVAL; +} + static int renesas_fw_download_image(struct pci_dev *dev, const u32 *fw, size_t step) @@ -144,10 +160,84 @@ static int renesas_fw_verify(const void *fw_data, return 0; } -static int renesas_fw_check_running(struct pci_dev *pdev) +static bool renesas_check_rom(struct pci_dev *pdev) { + u16 rom_status; + int retval; + + /* Check if external ROM exists */ + retval = pci_read_config_word(pdev, RENESAS_ROM_STATUS, &rom_status); + if (retval) + return false; + + rom_status &= RENESAS_ROM_STATUS_ROM_EXISTS; + if (rom_status) { + dev_dbg(&pdev->dev, "External ROM exists\n"); + return true; /* External ROM exists */ + } + + return false; +} + +static int renesas_check_rom_state(struct pci_dev *pdev) +{ + u16 rom_state; + u32 version; int err; + + /* check FW version */ + err = pci_read_config_dword(pdev, RENESAS_FW_VERSION, &version); + if (err) + return pcibios_err_to_errno(err); + + version &= RENESAS_FW_VERSION_FIELD; + version = version >> RENESAS_FW_VERSION_OFFSET; + + err = renesas_verify_fw_version(pdev, version); + if (err) + return err; + + /* + * Test if ROM is present and loaded, if so we can skip everything + */ + err = pci_read_config_word(pdev, RENESAS_ROM_STATUS, &rom_state); + if (err) + return pcibios_err_to_errno(err); + + if (rom_state & BIT(15)) { + /* ROM exists */ + dev_dbg(&pdev->dev, "ROM exists\n"); + + /* Check the "Result Code" Bits (6:4) and act accordingly */ + switch (rom_state & RENESAS_ROM_STATUS_RESULT) { + case RENESAS_ROM_STATUS_SUCCESS: + return 0; + + case RENESAS_ROM_STATUS_NO_RESULT: /* No result yet */ + return 0; + + case RENESAS_ROM_STATUS_ERROR: /* Error State */ + default: /* All other states are marked as "Reserved states" */ + dev_err(&pdev->dev, "Invalid ROM.."); + break; + } + } + + return -EIO; +} + +static int renesas_fw_check_running(struct pci_dev *pdev) +{ u8 fw_state; + int err; + + /* Check if device has ROM and loaded, if so skip everything */ + err = renesas_check_rom(pdev); + if (err) { /* we have rom */ + err = renesas_check_rom_state(pdev); + if (!err) + return err; + } /* * Test if the device is actually needing the firmware. As most @@ -303,11 +393,261 @@ static int renesas_fw_download(struct pci_dev *pdev, return 0; } +static void renesas_rom_erase(struct pci_dev *pdev) +{ + int retval, i; + u8 status; + + dev_dbg(&pdev->dev, "Performing ROM Erase...\n"); + retval = pci_write_config_dword(pdev, RENESAS_DATA0, + RENESAS_ROM_ERASE_MAGIC); + if (retval) { + dev_err(&pdev->dev, "ROM erase, magic word write failed: %d\n", + pcibios_err_to_errno(retval)); + return; + } + + retval = pci_read_config_byte(pdev, RENESAS_ROM_STATUS, &status); + if (retval) { + dev_err(&pdev->dev, "ROM status read failed: %d\n", + pcibios_err_to_errno(retval)); + return; + } + status |= RENESAS_ROM_STATUS_ERASE; + retval = pci_write_config_byte(pdev, RENESAS_ROM_STATUS, status); + if (retval) { + dev_err(&pdev->dev, "ROM erase set word write failed\n"); + return; + } + + /* sleep a bit while ROM is erased */ + msleep(20); + + for (i = 0; i < RENESAS_RETRY; i++) { + retval = pci_read_config_byte(pdev, RENESAS_ROM_STATUS, + &status); + status &= RENESAS_ROM_STATUS_ERASE; + if (!status) + break; + + mdelay(RENESAS_DELAY); + } + + if (i == RENESAS_RETRY) + dev_dbg(&pdev->dev, "Chip erase timedout: %x\n", status); + + dev_dbg(&pdev->dev, "ROM Erase... Done success\n"); +} + +static bool renesas_download_rom(struct pci_dev *pdev, + const u32 *fw, size_t step) +{ + bool data0_or_data1; + u8 fw_status; + size_t i; + int err; + + /* + * The hardware does alternate between two 32-bit pages. + * (This is because each row of the firmware is 8 bytes). + * + * for even steps we use DATA0, for odd steps DATA1. + */ + data0_or_data1 = (step & 1) == 1; + + /* Read "Set DATAX" and confirm it is cleared. */ + for (i = 0; i < RENESAS_RETRY; i++) { + err = pci_read_config_byte(pdev, RENESAS_ROM_STATUS_MSB, + &fw_status); + if (err) { + dev_err(&pdev->dev, "Read ROM Status failed: %d\n", + pcibios_err_to_errno(err)); + return false; + } + if (!(fw_status & BIT(data0_or_data1))) + break; + + udelay(RENESAS_DELAY); + } + if (i == RENESAS_RETRY) { + dev_err(&pdev->dev, "Timeout for Set DATAX step: %zd\n", step); + return false; + } + + /* + * Write FW data to "DATAX". + * "LSB is left" => force little endian + */ + err = pci_write_config_dword(pdev, data0_or_data1 ? + RENESAS_DATA1 : RENESAS_DATA0, + (__force u32)cpu_to_le32(fw[step])); + if (err) { + dev_err(&pdev->dev, "Write to DATAX failed: %d\n", + pcibios_err_to_errno(err)); + return false; + } + + udelay(100); + + /* Set "Set DATAX". */ + err = pci_write_config_byte(pdev, RENESAS_ROM_STATUS_MSB, + BIT(data0_or_data1)); + if (err) { + dev_err(&pdev->dev, "Write config for DATAX failed: %d\n", + pcibios_err_to_errno(err)); + return false; + } + + return true; +} + +static bool renesas_setup_rom(struct pci_dev *pdev, const struct firmware *fw) +{ + const u32 *fw_data = (const u32 *)fw->data; + int err, i; + u8 status; + + /* 2. Write magic word to Data0 */ + err = pci_write_config_dword(pdev, RENESAS_DATA0, + RENESAS_ROM_WRITE_MAGIC); + if (err) + return false; + + /* 3. Set External ROM access */ + err = pci_write_config_byte(pdev, RENESAS_ROM_STATUS, + RENESAS_ROM_STATUS_ACCESS); + if (err) + goto remove_bypass; + + /* 4. Check the result */ + err = pci_read_config_byte(pdev, RENESAS_ROM_STATUS, &status); + if (err) + goto remove_bypass; + status &= GENMASK(6, 4); + if (status) { + dev_err(&pdev->dev, + "setting external rom failed: %x\n", status); + goto remove_bypass; + } + + /* 5 to 16 Write FW to DATA0/1 while checking SetData0/1 */ + for (i = 0; i < fw->size / 4; i++) { + err = renesas_download_rom(pdev, fw_data, i); + if (!err) { + dev_err(&pdev->dev, + "ROM Download Step %d failed at position %d bytes\n", + i, i * 4); + goto remove_bypass; + } + } + + /* + * wait till DATA0/1 is cleared + */ + for (i = 0; i < RENESAS_RETRY; i++) { + err = pci_read_config_byte(pdev, RENESAS_ROM_STATUS_MSB, + &status); + if (err) + goto remove_bypass; + if (!(status & (BIT(0) | BIT(1)))) + break; + + udelay(RENESAS_DELAY); + } + if (i == RENESAS_RETRY) { + dev_err(&pdev->dev, "Final Firmware ROM Download step timed out\n"); + goto remove_bypass; + } + + /* 17. Remove bypass */ + err = pci_write_config_byte(pdev, RENESAS_ROM_STATUS, 0); + if (err) + return false; + + udelay(10); + + /* 18. check result */ + for (i = 0; i < RENESAS_RETRY; i++) { + err = pci_read_config_byte(pdev, RENESAS_ROM_STATUS, &status); + if (err) { + dev_err(&pdev->dev, "Read ROM status failed:%d\n", + pcibios_err_to_errno(err)); + return false; + } + status &= RENESAS_ROM_STATUS_RESULT; + if (status == RENESAS_ROM_STATUS_SUCCESS) { + dev_dbg(&pdev->dev, "Download ROM success\n"); + break; + } + udelay(RENESAS_DELAY); + } + if (i == RENESAS_RETRY) { /* Timed out */ + dev_err(&pdev->dev, + "Download to external ROM TO: %x\n", status); + return false; + } + + dev_dbg(&pdev->dev, "Download to external ROM succeeded\n"); + + /* Last step set Reload */ + err = pci_write_config_byte(pdev, RENESAS_ROM_STATUS, + RENESAS_ROM_STATUS_RELOAD); + if (err) { + dev_err(&pdev->dev, "Set ROM execute failed: %d\n", + pcibios_err_to_errno(err)); + return false; + } + + /* + * wait till Reload is cleared + */ + for (i = 0; i < RENESAS_RETRY; i++) { + err = pci_read_config_byte(pdev, RENESAS_ROM_STATUS, &status); + if (err) + return false; + if (!(status & RENESAS_ROM_STATUS_RELOAD)) + break; + + udelay(RENESAS_DELAY); + } + if (i == RENESAS_RETRY) { + dev_err(&pdev->dev, "ROM Exec timed out: %x\n", status); + return false; + } + + return true; + +remove_bypass: + pci_write_config_byte(pdev, RENESAS_ROM_STATUS, 0); + return false; +} + static int renesas_load_fw(struct pci_dev *pdev, const struct firmware *fw) { int err = 0; + bool rom; + + /* Check if the device has external ROM */ + rom = renesas_check_rom(pdev); + if (rom) { + /* perform chip erase first */ + renesas_rom_erase(pdev); + + /* lets try loading fw on ROM first */ + rom = renesas_setup_rom(pdev, fw); + if (!rom) { + dev_dbg(&pdev->dev, + "ROM load failed, falling back on FW load\n"); + } else { + dev_dbg(&pdev->dev, + "ROM load success\n"); + goto exit; + } + } err = renesas_fw_download(pdev, fw); + +exit: if (err) dev_dbg(&pdev->dev, "firmware failed to download (%d).", err); return err; -- 2.25.1 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH v9 5/5] usb: xhci: provide a debugfs hook for erasing rom 2020-04-14 16:41 [PATCH v9 0/5] usb: xhci: Add support for Renesas USB controllers Vinod Koul ` (3 preceding siblings ...) 2020-04-14 16:41 ` [PATCH v9 4/5] usb: renesas-xhci: Add ROM loader for uPD720201 Vinod Koul @ 2020-04-14 16:41 ` Vinod Koul 2020-04-14 22:27 ` [PATCH v9 0/5] usb: xhci: Add support for Renesas USB controllers John Stultz 2020-04-23 13:10 ` Vinod Koul 6 siblings, 0 replies; 11+ messages in thread From: Vinod Koul @ 2020-04-14 16:41 UTC (permalink / raw) To: Mathias Nyman, Greg Kroah-Hartman Cc: linux-arm-msm, Bjorn Andersson, Vinod Koul, Yoshihiro Shimoda, Christian Lamparter, John Stultz, Alan Stern, Andreas Böhler, linux-usb, linux-kernel run "echo 1 > /sys/kernel/debug/renesas-usb/rom_erase" to erase firmware when driver is loaded. Subsequent init of driver shall reload the firmware Signed-off-by: Vinod Koul <vkoul@kernel.org> --- drivers/usb/host/xhci-pci-renesas.c | 34 +++++++++++++++++++++++++++++ 1 file changed, 34 insertions(+) diff --git a/drivers/usb/host/xhci-pci-renesas.c b/drivers/usb/host/xhci-pci-renesas.c index b321d55f1e24..c7e18e70f0f8 100644 --- a/drivers/usb/host/xhci-pci-renesas.c +++ b/drivers/usb/host/xhci-pci-renesas.c @@ -2,6 +2,7 @@ /* Copyright (C) 2019-2020 Linaro Limited */ #include <linux/acpi.h> +#include <linux/debugfs.h> #include <linux/firmware.h> #include <linux/module.h> #include <linux/pci.h> @@ -160,6 +161,8 @@ static int renesas_fw_verify(const void *fw_data, return 0; } +static void debugfs_init(struct pci_dev *pdev); + static bool renesas_check_rom(struct pci_dev *pdev) { u16 rom_status; @@ -173,6 +176,7 @@ static bool renesas_check_rom(struct pci_dev *pdev) rom_status &= RENESAS_ROM_STATUS_ROM_EXISTS; if (rom_status) { dev_dbg(&pdev->dev, "External ROM exists\n"); + debugfs_init(pdev); return true; /* External ROM exists */ } @@ -439,6 +443,34 @@ static void renesas_rom_erase(struct pci_dev *pdev) dev_dbg(&pdev->dev, "ROM Erase... Done success\n"); } +static int debugfs_rom_erase(void *data, u64 value) +{ + struct pci_dev *pdev = data; + + if (value == 1) { + dev_dbg(&pdev->dev, "Userspace requested ROM erase\n"); + renesas_rom_erase(pdev); + return 0; + } + return -EINVAL; +} +DEFINE_DEBUGFS_ATTRIBUTE(rom_erase_ops, NULL, debugfs_rom_erase, "%llu\n"); + +static struct dentry *debugfs_root; + +static void debugfs_init(struct pci_dev *pdev) +{ + debugfs_root = debugfs_create_dir("renesas_usb", NULL); + + debugfs_create_file("rom_erase", 0200, debugfs_root, + pdev, &rom_erase_ops); +} + +static void debugfs_exit(void) +{ + debugfs_remove_recursive(debugfs_root); +} + static bool renesas_download_rom(struct pci_dev *pdev, const u32 *fw, size_t step) { @@ -696,6 +728,8 @@ int renesas_xhci_pci_probe(struct pci_dev *dev, int renesas_xhci_pci_remove(struct pci_dev *dev) { + debugfs_exit(); + if (renesas_fw_check_running(dev)) { /* * bail out early, if this was a renesas device w/o FW. -- 2.25.1 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH v9 0/5] usb: xhci: Add support for Renesas USB controllers 2020-04-14 16:41 [PATCH v9 0/5] usb: xhci: Add support for Renesas USB controllers Vinod Koul ` (4 preceding siblings ...) 2020-04-14 16:41 ` [PATCH v9 5/5] usb: xhci: provide a debugfs hook for erasing rom Vinod Koul @ 2020-04-14 22:27 ` John Stultz 2020-04-23 13:10 ` Vinod Koul 6 siblings, 0 replies; 11+ messages in thread From: John Stultz @ 2020-04-14 22:27 UTC (permalink / raw) To: Vinod Koul Cc: Mathias Nyman, Greg Kroah-Hartman, linux-arm-msm, Bjorn Andersson, Yoshihiro Shimoda, Christian Lamparter, Alan Stern, Andreas Böhler, Linux USB List, lkml On Tue, Apr 14, 2020 at 9:42 AM Vinod Koul <vkoul@kernel.org> wrote: > > This series add support for Renesas USB controllers uPD720201 and uPD720202. > These require firmware to be loaded and in case devices have ROM those can > also be programmed if empty. If ROM is programmed, it runs from ROM as well. > > This includes patches from Christian which supported these controllers w/o > ROM and later my patches for ROM support and debugfs hook for rom erase and > export of xhci-pci functions. > > Changes in v9: > Make fw load a sync call and have single instance of probe execute, > elimating probe/remove races > Add quirk for renesas and use that for loading Validated on db845c. Tested-by: John Stultz <john.stultz@linaro.org> thanks -john ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v9 0/5] usb: xhci: Add support for Renesas USB controllers 2020-04-14 16:41 [PATCH v9 0/5] usb: xhci: Add support for Renesas USB controllers Vinod Koul ` (5 preceding siblings ...) 2020-04-14 22:27 ` [PATCH v9 0/5] usb: xhci: Add support for Renesas USB controllers John Stultz @ 2020-04-23 13:10 ` Vinod Koul 6 siblings, 0 replies; 11+ messages in thread From: Vinod Koul @ 2020-04-23 13:10 UTC (permalink / raw) To: Mathias Nyman, Greg Kroah-Hartman Cc: linux-arm-msm, Bjorn Andersson, Yoshihiro Shimoda, Christian Lamparter, John Stultz, Alan Stern, Andreas Böhler, linux-usb, linux-kernel On 14-04-20, 22:11, Vinod Koul wrote: > This series add support for Renesas USB controllers uPD720201 and uPD720202. > These require firmware to be loaded and in case devices have ROM those can > also be programmed if empty. If ROM is programmed, it runs from ROM as well. > > This includes patches from Christian which supported these controllers w/o > ROM and later my patches for ROM support and debugfs hook for rom erase and > export of xhci-pci functions. Any feedback Mathias ? > > Changes in v9: > Make fw load a sync call and have single instance of probe execute, > elimating probe/remove races > Add quirk for renesas and use that for loading > > Changes in v8: > Fix compile error reported by Kbuild-bot by making usb_hcd_pci_probe() take > const struct hc_driver * as argument > > Changes in v7: > Make a single module which removes issues with module loading > Keep the renesas code in renesas file > Add hc_driver as argument for usb_hcd_pci_probe and modify hdc drivers to > pass this and not use driver_data > Use driver data for fw name > Remove code to check if we need to load firmware or not > remove multiple fw version support, we can do that with symlink in > userspace > > Changes in v6: > Move the renesas code into a separate driver which invokes xhci-pci functions. > > Changes in v5: > Added a debugfs rom erase patch, helps in debugging > Squashed patch 1 & 2 as requested by Mathias > > Changes in v4: > Rollback the delay values as we got device failures > > Changes in v3: > Dropped patch 2 as discussed with Christian > Removed aligned 8 bytes check > Change order for firmware search from highest version to lowest > Added entry for new firmware for device 0x14 as well > Add tested by Christian > > Changes in v2: > used macros for timeout count and delay > removed renesas_fw_alive_check > cleaned renesas_fw_callback > removed recurion for renesas_fw_download > added MODULE_FIRMWARE > added comment for multiple fw order > > Christian Lamparter (1): > usb: renesas-xhci: Add the renesas xhci driver > > Vinod Koul (4): > usb: hci: add hc_driver as argument for usb_hcd_pci_probe > usb: xhci: Add support for Renesas controller with memory > usb: renesas-xhci: Add ROM loader for uPD720201 > usb: xhci: provide a debugfs hook for erasing rom > > drivers/usb/core/hcd-pci.c | 7 +- > drivers/usb/host/Makefile | 3 +- > drivers/usb/host/ehci-pci.c | 6 +- > drivers/usb/host/ohci-pci.c | 9 +- > drivers/usb/host/uhci-pci.c | 8 +- > drivers/usb/host/xhci-pci-renesas.c | 740 ++++++++++++++++++++++++++++ > drivers/usb/host/xhci-pci.c | 47 +- > drivers/usb/host/xhci-pci.h | 16 + > drivers/usb/host/xhci.h | 1 + > include/linux/usb/hcd.h | 3 +- > 10 files changed, 817 insertions(+), 23 deletions(-) > create mode 100644 drivers/usb/host/xhci-pci-renesas.c > create mode 100644 drivers/usb/host/xhci-pci.h > > -- > 2.25.1 -- ~Vinod ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2020-04-25 9:43 UTC | newest] Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-04-14 16:41 [PATCH v9 0/5] usb: xhci: Add support for Renesas USB controllers Vinod Koul 2020-04-14 16:41 ` [PATCH v9 1/5] usb: hci: add hc_driver as argument for usb_hcd_pci_probe Vinod Koul 2020-04-14 16:41 ` [PATCH v9 2/5] usb: renesas-xhci: Add the renesas xhci driver Vinod Koul 2020-04-25 9:43 ` kbuild test robot 2020-04-14 16:41 ` [PATCH v9 3/5] usb: xhci: Add support for Renesas controller with memory Vinod Koul 2020-04-23 14:07 ` Mathias Nyman 2020-04-23 14:31 ` Vinod Koul 2020-04-14 16:41 ` [PATCH v9 4/5] usb: renesas-xhci: Add ROM loader for uPD720201 Vinod Koul 2020-04-14 16:41 ` [PATCH v9 5/5] usb: xhci: provide a debugfs hook for erasing rom Vinod Koul 2020-04-14 22:27 ` [PATCH v9 0/5] usb: xhci: Add support for Renesas USB controllers John Stultz 2020-04-23 13:10 ` Vinod Koul
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).