Linux-ARM-MSM Archive on lore.kernel.org
 help / color / Atom feed
From: Vinod Koul <vkoul@kernel.org>
To: Christian Lamparter <chunkeey@gmail.com>
Cc: Mathias Nyman <mathias.nyman@intel.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	linux-arm-msm@vger.kernel.org,
	Bjorn Andersson <bjorn.andersson@linaro.org>,
	Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>,
	Christian Lamparter <chunkeey@googlemail.com>,
	linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 0/5]  usb: xhci: Add support for Renesas USB controllers
Date: Sun, 23 Jun 2019 22:07:37 +0530
Message-ID: <20190623163737.GU2962@vkoul-mobl> (raw)
In-Reply-To: <11747914.jfR0NPdACA@debian64>

On 21-06-19, 21:41, Christian Lamparter wrote:
> On Friday, June 21, 2019 10:59:08 AM CEST 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 two patches from Christian which supported these controllers
> > w/o ROM and later my patches for ROM support and multiple firmware versions.
> > 
> > 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 (2):
> >   usb: xhci: add firmware loader for uPD720201 and uPD720202 w/o ROM
> >   usb: xhci: handle uPD720201 and uPD720202 w/o ROM
> > 
> > Vinod Koul (3):
> >   usb: xhci: Use register defined and field names
> >   usb: xhci: Add ROM loader for uPD720201
> >   usb: xhci: allow multiple firmware versions
> > 
> >  drivers/usb/host/xhci-pci.c | 942 ++++++++++++++++++++++++++++++++++++
> >  1 file changed, 942 insertions(+)
> > 
> 
> bootlog from a Netgear WNDR4700 with a 32GiB USB3 Stick attached to the 
> Renesas uPD720202K8-711-BAA-A chip.
> 
> [   13.523094] xhci_hcd 0000:45:00.0: firmware size is not a multiple of 8.
> [   13.868071] xhci_hcd 0000:45:00.0: xHCI Host Controller
> [   13.873325] xhci_hcd 0000:45:00.0: new USB bus registered, assigned bus number 2
> [   13.886296] xhci_hcd 0000:45:00.0: hcc params 0x014051cf hci version 0x100 quirks 0x0000000101000090
> [   13.902719] usbcore: registered new interface driver usb-storage
> [   13.909277] kmodloader: done loading kernel modules from /etc/modules-boot.d/*
> [   13.917336] hub 2-0:1.0: USB hub found
> [   13.921432] init: - preinit -
> [   13.930356] hub 2-0:1.0: 2 ports detected
> [   13.942869] xhci_hcd 0000:45:00.0: xHCI Host Controller
> [   13.948122] xhci_hcd 0000:45:00.0: new USB bus registered, assigned bus number 3
> [   13.955514] xhci_hcd 0000:45:00.0: Host supports USB 3.0  SuperSpeed
> [   13.981227] usb usb3: We don't know the algorithms for LPM for this host, disabling LPM.
> [   13.997141] hub 3-0:1.0: USB hub found
> [   14.004375] hub 3-0:1.0: 2 ports detected
> [   14.375644] usb-storage 1-1:1.0: USB Mass Storage device detected
> [   14.382247] scsi host1: usb-storage 1-1:1.0
> [   14.438959] usb 3-2: new SuperSpeed Gen 1 USB device number 2 using xhci_hcd
> [   14.467746] usb-storage 3-2:1.0: USB Mass Storage device detected
> [   14.474241] scsi host2: usb-storage 3-2:1.0
> [   15.394562] scsi 1:0:0:0: Direct-Access     Generic  STORAGE DEVICE   9454 PQ: 0 ANSI: 0
> [   15.405183] sd 1:0:0:0: [sda] Attached SCSI removable disk
> [   15.522448] scsi 2:0:0:0: Direct-Access     SanDisk  Ultra            1.00 PQ: 0 ANSI: 6
> [   15.531952] sd 2:0:0:0: [sdb] 60063744 512-byte logical blocks: (30.8 GB/28.6 GiB)
> [   15.541007] sd 2:0:0:0: [sdb] Write Protect is off
> [   15.546808] sd 2:0:0:0: [sdb] Write cache: disabled, read cache: enabled, doesn't support DPO or FUA
> [   15.574567]  sdb: sdb1
> [   15.580581] sd 2:0:0:0: [sdb] Attached SCSI removable disk
> 
> This is working.
> 
> Tested-by: Christian Lamparter <chunkeey@gmail.com>

Thanks for the verification Christian.

Regards
-- 
~Vinod

      reply index

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-06-21  8:59 Vinod Koul
2019-06-21  8:59 ` [PATCH v2 1/5] usb: xhci: add firmware loader for uPD720201 and uPD720202 w/o ROM Vinod Koul
2019-06-21 19:59   ` Christian Lamparter
2019-06-23 16:33     ` Vinod Koul
2019-06-21  8:59 ` [PATCH v2 2/5] usb: xhci: handle " Vinod Koul
2019-06-21 19:53   ` Christian Lamparter
2019-06-23 16:35     ` Vinod Koul
2019-06-21  8:59 ` [PATCH v2 3/5] usb: xhci: Use register defined and field names Vinod Koul
2019-06-21  8:59 ` [PATCH v2 4/5] usb: xhci: Add ROM loader for uPD720201 Vinod Koul
2019-06-21  8:59 ` [PATCH v2 5/5] usb: xhci: allow multiple firmware versions Vinod Koul
2019-06-21 19:46   ` Christian Lamparter
2019-06-23 16:36     ` Vinod Koul
2019-06-21 19:41 ` [PATCH v2 0/5] usb: xhci: Add support for Renesas USB controllers Christian Lamparter
2019-06-23 16:37   ` Vinod Koul [this message]

Reply instructions:

You may reply publically to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190623163737.GU2962@vkoul-mobl \
    --to=vkoul@kernel.org \
    --cc=bjorn.andersson@linaro.org \
    --cc=chunkeey@gmail.com \
    --cc=chunkeey@googlemail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=mathias.nyman@intel.com \
    --cc=yoshihiro.shimoda.uh@renesas.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Linux-ARM-MSM Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-arm-msm/0 linux-arm-msm/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-arm-msm linux-arm-msm/ https://lore.kernel.org/linux-arm-msm \
		linux-arm-msm@vger.kernel.org linux-arm-msm@archiver.kernel.org
	public-inbox-index linux-arm-msm


Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-arm-msm


AGPL code for this site: git clone https://public-inbox.org/ public-inbox