Re: [PATCH 03/11] mux: consumer.h: Add MUX_USB_* state constant defines

2017-09-02 Thread Hans de Goede
Hi, On 02-09-17 21:06, Guenter Roeck wrote: On Sat, Sep 02, 2017 at 05:59:14PM +0200, Hans de Goede wrote: Hi, On 02-09-17 16:59, Guenter Roeck wrote: On 09/01/2017 02:48 PM, Hans de Goede wrote: Add MUX_USB_* state constant defines, which can be used by USB device/host and Type-C

Re: [PATCH 02/11] mux: core: Add support for getting a mux controller on a non DT platform

2017-09-02 Thread sathya
Hi, On 09/01/2017 02:48 PM, Hans de Goede wrote: On non DT platforms we cannot get the mux_chip by pnode. Other subsystems (regulator, clock, pwm) have the same problem and solve this by allowing platform / board-setup code to add entries to a lookup table and then use this table to look

Re: [PATCH 03/11] mux: consumer.h: Add MUX_USB_* state constant defines

2017-09-02 Thread Guenter Roeck
On Sat, Sep 02, 2017 at 05:59:14PM +0200, Hans de Goede wrote: > Hi, > > On 02-09-17 16:59, Guenter Roeck wrote: > >On 09/01/2017 02:48 PM, Hans de Goede wrote: > >>Add MUX_USB_* state constant defines, which can be used by USB > >>device/host and Type-C polarity/role/altmode mux drivers and

Re: [PATCH] usb: gadget: pch_udc: add checks for dma mapping errors

2017-09-02 Thread Alexey Khoroshilov
On 24.08.2017 04:02, Jack Pham wrote: > On Thu, Aug 24, 2017 at 01:47:08AM +0300, Alexey Khoroshilov wrote: >> There are no checks for dma mapping errors in pch_udc. >> Tha patch adds the checks and error handling code. >> Compile tested only. >> >> Found by Linux Driver Verification project

Re: [PATCH 03/11] mux: consumer.h: Add MUX_USB_* state constant defines

2017-09-02 Thread Hans de Goede
Hi, On 02-09-17 16:59, Guenter Roeck wrote: On 09/01/2017 02:48 PM, Hans de Goede wrote: Add MUX_USB_* state constant defines, which can be used by USB device/host and Type-C polarity/role/altmode mux drivers and consumers to ensure that they agree on the meaning of the mux_control_select()

Re: [PATCH 03/11] mux: consumer.h: Add MUX_USB_* state constant defines

2017-09-02 Thread Guenter Roeck
On 09/01/2017 02:48 PM, Hans de Goede wrote: Add MUX_USB_* state constant defines, which can be used by USB device/host and Type-C polarity/role/altmode mux drivers and consumers to ensure that they agree on the meaning of the mux_control_select() state argument. Signed-off-by: Hans de Goede

Re: [PATCH 03/11] mux: consumer.h: Add MUX_USB_* state constant defines

2017-09-02 Thread Hans de Goede
Hi, On 02-09-17 12:10, Andy Shevchenko wrote: On Sat, Sep 2, 2017 at 12:48 AM, Hans de Goede wrote: Add MUX_USB_* state constant defines, which can be used by USB device/host and Type-C polarity/role/altmode mux drivers and consumers to ensure that they agree on the

Re: [PATCH 11/11] platform/x86: intel_cht_int33fe: Add mux mappings for the Type-C port

2017-09-02 Thread Andy Shevchenko
On Sat, Sep 2, 2017 at 12:48 AM, Hans de Goede wrote: > We need to add mappings for the mux subsys to be able to find the > muxes for the fusb302 driver to be able to control the PI3USB30532 > Type-C mux and the device/host mux integrated in the CHT SoC. > I suppose it will

Re: [PATCH 07/11] extcon: intel-int3496: Add support for controlling the USB-role mux

2017-09-02 Thread Andy Shevchenko
On Sat, Sep 2, 2017 at 12:48 AM, Hans de Goede wrote: > Cherry Trail SoCs have a built-in USB-role mux for switching between > the host and device controllers, rather then using an external mux > controller by a GPIO. > > There is a driver using the mux-subsys to control this

Re: [PATCH 05/11] mux: Add Intel Cherrytrail USB mux driver

2017-09-02 Thread Dan Carpenter
On Sat, Sep 02, 2017 at 01:19:05PM +0300, Andy Shevchenko wrote: > > +static void intel_cht_usb_mux_vbus_work(struct work_struct *work) > > +{ > > + struct intel_cht_usb_mux *mux = > > + container_of(work, struct intel_cht_usb_mux, vbus_work); > > + bool vbus_present =

Re: [PATCH 05/11] mux: Add Intel Cherrytrail USB mux driver

2017-09-02 Thread Andy Shevchenko
On Sat, Sep 2, 2017 at 12:48 AM, Hans de Goede wrote: > Intel Cherrytrail SoCs have an internal USB mux for muxing the otg-port > USB data lines between the xHCI host controller and the dwc3 gadget > controller. On some Cherrytrail systems this mux is controlled through > AML

Re: [PATCH 03/11] mux: consumer.h: Add MUX_USB_* state constant defines

2017-09-02 Thread Andy Shevchenko
On Sat, Sep 2, 2017 at 12:48 AM, Hans de Goede wrote: > Add MUX_USB_* state constant defines, which can be used by USB > device/host and Type-C polarity/role/altmode mux drivers and consumers > to ensure that they agree on the meaning of the mux_control_select() > state