public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Mattijs Korpershoek <mkorpershoek@kernel.org>
To: Mattijs Korpershoek <mkorpershoek@kernel.org>,
	"Markus Schneider-Pargmann (TI.com)" <msp@baylibre.com>
Cc: u-boot@lists.denx.de, Simon Glass <sjg@chromium.org>,
	Tom Rini <trini@konsulko.com>, Marek Vasut <marex@denx.de>,
	Andrew Goodbody <andrew.goodbody@linaro.org>,
	Kory Maincent <kory.maincent@bootlin.com>,
	Svyatoslav Ryhel <clamor95@gmail.com>,
	Christian Marangi <ansuelsmth@gmail.com>,
	Dinesh Maniyam <dinesh.maniyam@altera.com>,
	Heiko Schocher <hs@nabladev.com>
Subject: Re: [PATCH 0/4] dm: core: Support same compatible in host/gadget musb drivers
Date: Thu, 08 Jan 2026 15:13:51 +0100	[thread overview]
Message-ID: <87ms2oyyxs.fsf@kernel.org> (raw)
In-Reply-To: <87pl7lzc5t.fsf@kernel.org>

On Wed, Jan 07, 2026 at 16:15, Mattijs Korpershoek <mkorpershoek@kernel.org> wrote:

> Hi Markus,
>
> On Wed, Jan 07, 2026 at 15:01, Mattijs Korpershoek <mkorpershoek@kernel.org> wrote:
>
>> Hi,
>>
>> On Wed, 26 Nov 2025 21:31:06 +0100, Markus Schneider-Pargmann (TI.com) wrote:
>>> musb currently uses a wrapper driver that binds on the parent device of
>>> the actual musb devices to manage the differentiation between gadget and
>>> host modes. However in the upstream devicetree this parent devicetree
>>> node can not be used to match the wrapper driver.
>>> 
>>> To be able to probe the musb devices in host/gadget mode directly, this
>>> series introduces support for returning -ENODEV in bind functions
>>> resulting in iterating the remaining drivers potentially binding to
>>> other drivers that match the compatible.
>>> 
>>> [...]
>>
>> Thanks, Applied to https://source.denx.de/u-boot/custodians/u-boot-dfu (u-boot-dfu)
>
> It seems this series broke the CI when testing sandbox:
> https://source.denx.de/u-boot/custodians/u-boot-dfu/-/jobs/1349214
>
> Can you have a look please?
>
> This can be reproduced locally using:
>
> $ ./test/py/test.py --bd sandbox --build -k test_bind
> [...]
> FAILED test/py/tests/test_bind.py::test_bind_unbind_with_node - AssertionError: assert 'Unable to bind. err:0' == ''
> FAILED test/py/tests/test_bind.py::test_bind_unbind_with_uclass - AssertionError: assert 2 == 1

We have discussed this on IRC (privately) and Markus has posted a v2
that should address the CI failures here:

https://lore.kernel.org/all/20260108-topic-musb-probing-v2026-01-v2-0-2a47c6e0e73a@baylibre.com/


>
>>
>> [1/4] dm: core: Support multiple drivers with same compatibles
>>       https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/292d809159b86ff3c07e4fea93927cd0d00d8b27
>> [2/4] test: dm: Add compatible multimatch test
>>       https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/b1f483a350a7df6130eb4880ecce4cfabfe5b9a9
>> [3/4] usb: musb-new: Relative ctrl_mod address parsing
>>       https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/dce67e20709bd89f4ec17135e242447b5e6570d4
>> [4/4] usb: musb-new: Add compatibles for ti,musb-am33xx
>>       https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/b51b489ae7c47d8372a6b73a4c0c5fa71291c4fd
>>
>> --
>> Mattijs

  reply	other threads:[~2026-01-08 14:14 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-11-26 20:31 [PATCH 0/4] dm: core: Support same compatible in host/gadget musb drivers Markus Schneider-Pargmann (TI.com)
2025-11-26 20:31 ` [PATCH 1/4] dm: core: Support multiple drivers with same compatibles Markus Schneider-Pargmann (TI.com)
2026-01-07 13:16   ` Mattijs Korpershoek
2025-11-26 20:31 ` [PATCH 2/4] test: dm: Add compatible multimatch test Markus Schneider-Pargmann (TI.com)
2026-01-07 13:35   ` Mattijs Korpershoek
2025-11-26 20:31 ` [PATCH 3/4] usb: musb-new: Relative ctrl_mod address parsing Markus Schneider-Pargmann (TI.com)
2026-01-07 13:44   ` Mattijs Korpershoek
2025-11-26 20:31 ` [PATCH 4/4] usb: musb-new: Add compatibles for ti,musb-am33xx Markus Schneider-Pargmann (TI.com)
2026-01-07 13:48   ` Mattijs Korpershoek
2026-01-07 14:01 ` [PATCH 0/4] dm: core: Support same compatible in host/gadget musb drivers Mattijs Korpershoek
2026-01-07 15:15   ` Mattijs Korpershoek
2026-01-08 14:13     ` Mattijs Korpershoek [this message]
2026-01-08 19:25       ` Markus Schneider-Pargmann

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=87ms2oyyxs.fsf@kernel.org \
    --to=mkorpershoek@kernel.org \
    --cc=andrew.goodbody@linaro.org \
    --cc=ansuelsmth@gmail.com \
    --cc=clamor95@gmail.com \
    --cc=dinesh.maniyam@altera.com \
    --cc=hs@nabladev.com \
    --cc=kory.maincent@bootlin.com \
    --cc=marex@denx.de \
    --cc=msp@baylibre.com \
    --cc=sjg@chromium.org \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox