All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mattijs Korpershoek <mkorpershoek@baylibre.com>
To: Marek Vasut <marex@denx.de>, Zixun LI <admin@hifiphile.com>
Cc: Tom Rini <trini@konsulko.com>, Lukasz Majewski <lukma@denx.de>,
	u-boot@lists.denx.de
Subject: Re: [PATCH 1/1] usb: gadget: ether: Disable USB pullup in eth probe
Date: Thu, 29 Aug 2024 09:02:25 +0200	[thread overview]
Message-ID: <87bk1bstfi.fsf@baylibre.com> (raw)
In-Reply-To: <4e154bda-6d7d-45b8-bb7d-ba4d0623a932@denx.de>

On lun., août 26, 2024 at 16:40, Marek Vasut <marex@denx.de> wrote:

> On 8/26/24 11:12 AM, Zixun LI wrote:
>> On Sun, Aug 25, 2024 at 2:13 AM Marek Vasut <marex@denx.de> wrote:
>>>
>>>>
>>>> I am looking at the usb_add_gadget_udc_release() and
>>>> usb_gadget_probe_driver() and I am thinking, sigh, all that code is
>>>> ready to be replaced by something cleaner which does not use this
>>>> special struct usb_udc and udc_list, but instead only uses U-Boot DM
>>>> udevices . But I am reluctant to ask you to do all that rework, because
>>>> that is WAY out of scope of the problem you are trying to fix.
>>>
>> 
>> Thank you for looking into this, refactor udc core could make things clear.
>
> Sigh, yeah, I think you ran into part of the UDC code which is just ... 
> not great ... sorry about that.
>
>>> Maybe this could be a start (compile tested only):
>>>
>>> https://source.denx.de/u-boot/custodians/u-boot-sh/-/commits/usb-udc-udevice
>> 
>> Tested no regression with usba_udc.
>
> Thanks for testing, patches posted.

For reference, the series was posted here:

https://lore.kernel.org/r/20240826143851.8020-1-marek.vasut+renesas@mailbox.org

      reply	other threads:[~2024-08-29  7:02 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-08-21 21:22 [PATCH 1/1] usb: gadget: ether: Disable USB pullup in eth probe Zixun LI
2024-08-21 21:43 ` Marek Vasut
2024-08-21 21:58   ` Zixun LI
2024-08-21 23:21     ` Marek Vasut
2024-08-22  9:11       ` Zixun LI
2024-08-22 15:19         ` Marek Vasut
2024-08-22 15:53           ` Zixun LI
2024-08-23  1:23             ` Marek Vasut
2024-08-23  9:22               ` Zixun LI
2024-08-24 23:23                 ` Marek Vasut
2024-08-25  0:13                   ` Marek Vasut
2024-08-26  9:12                     ` Zixun LI
2024-08-26 14:40                       ` Marek Vasut
2024-08-29  7:02                         ` Mattijs Korpershoek [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=87bk1bstfi.fsf@baylibre.com \
    --to=mkorpershoek@baylibre.com \
    --cc=admin@hifiphile.com \
    --cc=lukma@denx.de \
    --cc=marex@denx.de \
    --cc=trini@konsulko.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.