All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dr. Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [U-Boot, v4, 3/4] usb: dwc3: add support for 16 bit UTMI+ interface
Date: Tue, 12 Sep 2017 11:09:07 +0200	[thread overview]
Message-ID: <569E2286-40C7-4AF6-B666-118BAADA27BB@theobroma-systems.com> (raw)
In-Reply-To: <f2444a79-650a-2686-c310-d1fda5b9df90@rock-chips.com>

Marek,

do you know whether there's an update of the DWC3 driver planned to get us back in sync with the kernel driver?

Thanks,
Philipp.


> On 12 Sep 2017, at 07:02, Kever Yang <kever.yang@rock-chips.com> wrote:
> 
> Philipp,
> 
>    I believe there are some request I think is not reasonable which stop me from update new patch sets.
> 
> The usb/dwc3 is mess up now, separate in host mode and gadget mode, one need init in board init
> 
> while another use DM. When I want to add a new feature like use 16bit UTMI+  instead of 8bit(which suppose to be very very simple),
> 
> I don't think the maintainer have an idea where it should go and how it should be, and the driver now
> 
> it's far away from upstream kernel.
> 
> I have send 4 patch set for this, everything should be clear, but...
> 
> 
> 
> Thanks,
> - Kever
> On 09/11/2017 09:30 PM, Philipp Tomsich wrote:
>> 
>> Kever,
>> 
>> What is the status on this patch (and the encapsulating series)?
>> This seems to be stuck in 'changes requested' for about a year now...
>> 
>> Thanks,
>> Philipp.
>> 
>> On Thu, 1 Sep 2016, Kever Yang wrote:
>> 
>>> The dwc3 controller is using 8 bit UTMI+ interface for USB2.0 PHY,
>>> add one variable in dwc3/dwc3_device struct to support 16 bit
>>> UTMI+ interface on some SoCs like Rockchip rk3399.
>>> 
>>> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
>>> ---
>>> 
>>> Changes in v4:
>>> - use 1 bit for usb2_phyif_utmi_width instead of 5bit
>>> 
>>> Changes in v3:
>>> - Parse the DT for utmi+ interface width in dwc3 driver
>>> 
>>> Changes in v2:
>>> - use a variable to identify utmi+ bus width instead of CONFIG MACRO
>>> 
>>> drivers/usb/dwc3/core.c | 19 +++++++++++++++++++
>>> drivers/usb/dwc3/core.h | 12 ++++++++++++
>>> 2 files changed, 31 insertions(+)
>>> 
>> 
> 
> 

  reply	other threads:[~2017-09-12  9:09 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-01  2:14 [U-Boot] [PATCH v4 0/4] rk3399: enable dwc3 gadget and fastboot Kever Yang
2016-09-01  2:14 ` [U-Boot] [PATCH v4 1/4] rk3399: add a empty "sys_proto.h" header file Kever Yang
2016-09-05 23:17   ` Simon Glass
2016-09-01  2:14 ` [U-Boot] [PATCH v4 2/4] board: evb-rk3399: add api to support dwc3 gadget Kever Yang
2016-09-01  8:59   ` Marek Vasut
2016-09-02  9:58     ` Kever Yang
2016-09-02 12:14       ` Marek Vasut
2016-09-01  2:14 ` [U-Boot] [PATCH v4 3/4] usb: dwc3: add support for 16 bit UTMI+ interface Kever Yang
2016-09-01  9:09   ` Marek Vasut
2016-09-02 10:00     ` Kever Yang
2017-09-11 13:30   ` [U-Boot] [U-Boot, v4, " Philipp Tomsich
2017-09-12  5:02     ` Kever Yang
2017-09-12  9:09       ` Dr. Philipp Tomsich [this message]
2017-09-12 10:22         ` Marek Vasut
2017-09-12 10:50           ` Dr. Philipp Tomsich
2016-09-01  2:14 ` [U-Boot] [PATCH v4 4/4] config: rk3399: add support for dwc3 gadget Kever Yang

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=569E2286-40C7-4AF6-B666-118BAADA27BB@theobroma-systems.com \
    --to=philipp.tomsich@theobroma-systems.com \
    --cc=u-boot@lists.denx.de \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.