LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Matthias Kaehlcke <mka@chromium.org>
To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: "Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
	"Alan Stern" <stern@rowland.harvard.edu>,
	"Rob Herring" <robh+dt@kernel.org>,
	"Frank Rowand" <frowand.list@gmail.com>,
	"Mathias Nyman" <mathias.nyman@intel.com>,
	"Felipe Balbi" <balbi@kernel.org>,
	"Bastien Nocera" <hadess@hadess.net>,
	"Peter Chen" <peter.chen@kernel.org>,
	"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
	<devicetree@vger.kernel.org>,
	"Michal Simek" <michal.simek@xilinx.com>,
	"open list" <linux-kernel@vger.kernel.org>,
	"Douglas Anderson" <dianders@chromium.org>,
	"Krzysztof Kozlowski" <krzk@kernel.org>,
	linux-usb@vger.kernel.org,
	"Ravi Chandra Sadineni" <ravisadineni@chromium.org>,
	"Stephen Boyd" <swboyd@chromium.org>,
	"Al Cooper" <alcooperx@gmail.com>,
	"Andrey Zhizhikin" <andrey.zhizhikin@leica-geosystems.com>,
	"Andy Gross" <agross@kernel.org>,
	"Aswath Govindraju" <a-govindraju@ti.com>,
	"Bjorn Andersson" <bjorn.andersson@linaro.org>,
	"Catalin Marinas" <catalin.marinas@arm.com>,
	"Dmitry Osipenko" <digetx@gmail.com>,
	"Enric Balletbo i Serra" <enric.balletbo@collabora.com>,
	"Fabio Estevam" <festevam@gmail.com>,
	"Guido Günther" <agx@sigxcpu.org>,
	"Jagan Teki" <jagan@amarulasolutions.com>,
	"Nishanth Menon" <nm@ti.com>,
	"Pawel Laszczak" <pawell@cadence.com>,
	"Roger Quadros" <rogerq@kernel.org>,
	"Shawn Guo" <shawnguo@kernel.org>,
	"Vinod Koul" <vkoul@kernel.org>, "Will Deacon" <will@kernel.org>,
	"Linux ARM" <linux-arm-kernel@lists.infradead.org>,
	"open list:DRM DRIVER FOR MSM ADRENO GPU"
	<linux-arm-msm@vger.kernel.org>
Subject: Re: [PATCH v15 0/6] usb: misc: Add onboard_usb_hub driver
Date: Tue, 19 Oct 2021 15:54:18 -0700	[thread overview]
Message-ID: <YW9MmoSTouEDdpxa@google.com> (raw)
In-Reply-To: <CAA8EJpq55e+fk9oDi8+JXDWiPcXDXK5oz1DL5eqfx+FkT-xhnw@mail.gmail.com>

Hi Dmitry,

On Tue, Oct 19, 2021 at 07:24:41PM +0300, Dmitry Baryshkov wrote:
> On Tue, 27 Jul 2021 at 03:41, Matthias Kaehlcke <mka@chromium.org> wrote:
> >
> > This series adds:
> > - the onboard_usb_hub_driver
> > - glue in the xhci-plat driver to create and destroy the
> >   onboard_usb_hub platform devices if needed
> > - a device tree binding for the Realtek RTS5411 USB hub controller
> > - device tree changes that add RTS5411 entries for the QCA SC7180
> >   based boards trogdor and lazor
> > - a couple of stubs for platform device functions to avoid
> >   unresolved symbols with certain kernel configs
> >
> > The main issue the driver addresses is that a USB hub needs to be
> > powered before it can be discovered. For discrete onboard hubs (an
> > example for such a hub is the Realtek RTS5411) this is often solved
> > by supplying the hub with an 'always-on' regulator, which is kind
> > of a hack. Some onboard hubs may require further initialization
> > steps, like changing the state of a GPIO or enabling a clock, which
> > requires even more hacks. This driver creates a platform device
> > representing the hub which performs the necessary initialization.
> > Currently it only supports switching on a single regulator, support
> > for multiple regulators or other actions can be added as needed.
> > Different initialization sequences can be supported based on the
> > compatible string.
> 
> I have the feeling that you might want to check if you can use pwrseq
> subsystem being proposed at
> https://lore.kernel.org/linux-arm-msm/20211006035407.1147909-1-dmitry.baryshkov@linaro.org/.
> It has been created for exactly the same reason of handling complex
> power up/down requirements in a bus-neutral way. So instead of
> creating an onboard-usb-hub, you might want to populate the hub node
> with the reference to pwrseq device and make usb core call into
> pwrseq. How does that sound to you?

Thanks for the pointer, it's good to see another attempt to sort out
power sequencing.

The pwrseq framework could potentially be used by the onboard_usb_hub
driver, but it probably can't replace it completely. Besides powering
the USB hub on before enumeration the driver also can optionally power
it off during system suspend when no wakeup capable USB devices are
connected, which can result in signifcant power savings on battery
powered devices. For this the driver needs knowledge about the USB
(hub) devices that are provided by a hub chip. That part is probably
best implemented by a driver under drivers/usb/.

It might be an option to have the USB core and the onboard_usb_hub
driver use the pwrseq framework, though we'd have to ensure that it
isn't a problem that the USB core turns power on (before
enumeration) and the onboard_usb_hub driver turns it off during
system suspend (and on again on resume).

      reply	other threads:[~2021-10-19 22:54 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-27  0:41 Matthias Kaehlcke
2021-07-27  0:41 ` [PATCH v15 1/6] " Matthias Kaehlcke
2021-07-27  0:41 ` [PATCH v15 2/6] of/platform: Add stubs for of_platform_device_create/destroy() Matthias Kaehlcke
2021-07-27  0:41 ` [PATCH v15 3/6] arm64: defconfig: Explicitly enable USB_XHCI_PLATFORM Matthias Kaehlcke
2021-07-27  0:41 ` [PATCH v15 4/6] usb: Specify dependencies on USB_XHCI_PLATFORM with 'depends on' Matthias Kaehlcke
2021-07-27  0:41 ` [PATCH v15 5/6] usb: host: xhci-plat: Create platform device for onboard hubs in probe() Matthias Kaehlcke
2021-07-27  0:41 ` [PATCH v15 6/6] arm64: dts: qcom: sc7180-trogdor: Add nodes for onboard USB hub Matthias Kaehlcke
2021-08-09 15:49 ` [PATCH v15 0/6] usb: misc: Add onboard_usb_hub driver Matthias Kaehlcke
2021-08-11 12:38   ` Roger Quadros
2021-08-11 12:56     ` Roger Quadros
2021-08-12 17:08       ` Matthias Kaehlcke
2021-08-13  6:35         ` Roger Quadros
2021-08-13 18:33           ` Matthias Kaehlcke
2021-10-19 16:24 ` Dmitry Baryshkov
2021-10-19 22:54   ` Matthias Kaehlcke [this message]

Reply instructions:

You may reply publicly 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=YW9MmoSTouEDdpxa@google.com \
    --to=mka@chromium.org \
    --cc=a-govindraju@ti.com \
    --cc=agross@kernel.org \
    --cc=agx@sigxcpu.org \
    --cc=alcooperx@gmail.com \
    --cc=andrey.zhizhikin@leica-geosystems.com \
    --cc=balbi@kernel.org \
    --cc=bjorn.andersson@linaro.org \
    --cc=catalin.marinas@arm.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dianders@chromium.org \
    --cc=digetx@gmail.com \
    --cc=dmitry.baryshkov@linaro.org \
    --cc=enric.balletbo@collabora.com \
    --cc=festevam@gmail.com \
    --cc=frowand.list@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=hadess@hadess.net \
    --cc=jagan@amarulasolutions.com \
    --cc=krzk@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.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=michal.simek@xilinx.com \
    --cc=nm@ti.com \
    --cc=pawell@cadence.com \
    --cc=peter.chen@kernel.org \
    --cc=ravisadineni@chromium.org \
    --cc=robh+dt@kernel.org \
    --cc=rogerq@kernel.org \
    --cc=shawnguo@kernel.org \
    --cc=stern@rowland.harvard.edu \
    --cc=swboyd@chromium.org \
    --cc=vkoul@kernel.org \
    --cc=will@kernel.org \
    --subject='Re: [PATCH v15 0/6] usb: misc: Add onboard_usb_hub driver' \
    /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

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).