[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20160621104321.GJ22406@atomide.com>
Date: Tue, 21 Jun 2016 03:43:23 -0700
From: Tony Lindgren <tony@...mide.com>
To: Felipe Balbi <balbi@...nel.org>
Cc: Peter Chen <hzpeterchen@...il.com>, Roger Quadros <rogerq@...com>,
peter.chen@...escale.com, gregkh@...uxfoundation.org,
dan.j.williams@...el.com, mathias.nyman@...ux.intel.com,
Joao.Pinto@...opsys.com, sergei.shtylyov@...entembedded.com,
jun.li@...escale.com, grygorii.strashko@...com,
yoshihiro.shimoda.uh@...esas.com, robh@...nel.org, nsekhar@...com,
b-liu@...com, joe@...ches.com, linux-usb@...r.kernel.org,
linux-omap@...r.kernel.org, linux-kernel@...r.kernel.org,
devicetree@...r.kernel.org
Subject: Re: [PATCH v11 08/14] usb: otg: add OTG/dual-role core
* Felipe Balbi <balbi@...nel.org> [160621 03:06]:
> 8<----------------------------------------------------------------------
> vbus = read(VBUS_STATE); /* could be a gpio_get_value() */
> id = read(ID_STATE); /* could be a gpio_get_value() */
>
> set_role(id);
> set_vbus(vbus);
We should use regulator framework API for set_vbus() because of
the delays involved bringing it up. And we already have separate
PHY and charger chips where VBUS is provided by the charger chip.
Regards,
Tony
Powered by blists - more mailing lists