public inbox for linux-bluetooth@vger.kernel.org
 help / color / mirror / Atom feed
From: "Uwe Kleine-König" <u.kleine-koenig@baylibre.com>
To: linux-bluetooth@vger.kernel.org
Subject: Re: sdio: About pointers in sdio_device_id::driver_data
Date: Fri, 17 Apr 2026 17:20:51 +0200	[thread overview]
Message-ID: <aeJNDwFrs8R8BDQr@monoceros> (raw)
In-Reply-To: <69e23f9e.050a0220.17c124.ff26@mx.google.com>

[-- Attachment #1: Type: text/plain, Size: 1739 bytes --]

Hello,

some feedback for this mail:

On Fri, Apr 17, 2026 at 07:11:42AM -0700, bluez.test.bot@gmail.com wrote:
> This is an automated email and please do not reply to this email.

I hope it's ok to ignore this request here :-)

> While preparing the CI tests, the patches you submitted couldn't be applied to the current HEAD of the repository.

It would be great to point out the URL and commit hash of this current
HEAD to simplify acting on that and also be able to understand the
details of the mail in a few weeks when the repository probably got a
few more commits.

Having said that, the patch series fits on top of
git://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth.git's
HEAD currently at bc39a094730c ("Bluetooth: hci_sync: fix stack buffer
overflow in hci_le_big_create_sync"). But I guess "the repository" is
git://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git
that indeed contains a conflicting commit.

Assuming that the series being applicable to the bluetooth HEAD isn't
the only test done here, a great way to make the bot reply more useful
would be to honor

	base-commit: 028ef9c96e96197026887c0f092424679298aae8

from the cover letter and then provide more test results (after pointing
out that the wrong base was chosen).

> ----- Output -----
> 
> error: patch failed: drivers/bluetooth/btmtksdio.c:64
> error: drivers/bluetooth/btmtksdio.c: patch does not apply
> hint: Use 'git am --show-current-patch' to see the failed patch
> 
> Please resolve the issue and submit the patches again.

Given that the bluetooth parts are not ready for application now without
the first patch in the series, I won't do that now.

Best regards
Uwe

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

  reply	other threads:[~2026-04-17 15:20 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-17 13:10 [PATCH v1 0/6] sdio: About pointers in sdio_device_id::driver_data Uwe Kleine-König (The Capable Hub)
2026-04-17 13:10 ` [PATCH v1 1/6] sdio: Add syntactic sugar to store a pointer in sdio_driver_id Uwe Kleine-König (The Capable Hub)
2026-04-17 14:11   ` sdio: About pointers in sdio_device_id::driver_data bluez.test.bot
2026-04-17 15:20     ` Uwe Kleine-König [this message]
2026-04-20 20:31   ` [PATCH v1 1/6] sdio: Add syntactic sugar to store a pointer in sdio_driver_id Uwe Kleine-König (The Capable Hub)
2026-04-20 20:46     ` Luiz Augusto von Dentz
2026-04-21  8:12       ` Uwe Kleine-König (The Capable Hub)
2026-04-21  8:59         ` Johannes Berg
2026-04-21 14:28           ` Uwe Kleine-König (The Capable Hub)
2026-04-21 15:54             ` Johannes Berg
2026-04-17 13:10 ` [PATCH v1 2/6] Bluetooth: btmrvl_sdio: Make use of driver data pointer in sdio_device_id Uwe Kleine-König (The Capable Hub)
2026-04-17 13:10 ` [PATCH v1 3/6] Bluetooth: btmtksdio: " Uwe Kleine-König (The Capable Hub)

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=aeJNDwFrs8R8BDQr@monoceros \
    --to=u.kleine-koenig@baylibre.com \
    --cc=linux-bluetooth@vger.kernel.org \
    /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