[PATCH v1 1/6] sdio: Add syntactic sugar to store a pointer in sdio_driver_id
Luiz Augusto von Dentz
luiz.dentz at gmail.com
Mon Apr 20 13:46:56 PDT 2026
Hi Uwe,
On Mon, Apr 20, 2026 at 4:31 PM Uwe Kleine-König (The Capable Hub)
<u.kleine-koenig at baylibre.com> wrote:
>
> Hello,
>
> On Fri, Apr 17, 2026 at 03:10:47PM +0200, Uwe Kleine-König (The Capable Hub) wrote:
> > On all current Linux architectures sizeof(long) == sizeof(void *) and
> > this is used a lot through the kernel. For example it enables the usual
> > practice to store pointers in sdio_driver_id's .driver_data member.
> >
> > This works fine, but involves casting and thus isn't type-safe.
> > Additionally with the CHERI architecture extension there are machines
> > with sizeof(void *) > sizeof(long) for with the traditional approach of
> > storing a pointer in .driver_data doesn't work.
> >
> > By replacing the plain unsigned long .driver_data by an anonymous union,
> > most of the casting can be dropped and it yields a working solution for
> > CHERI.
> >
> > All users of struct sdio_driver_id are initialized in a way that is
> > compatible with the new definition, so no adaptions are needed there.
>
> sashiko.dev found s/sdio_driver_id/sdio_device_id/ twice in the commit
> log and once in the short log. If you consider applying this patch
> please adapt the commit message accordingly.
No problem I can fix them up once applying.
> Many thanks to those who created sashiko.dev!
>
> Best regards
> Uwe
We only received 1-3 of the 6:
https://patchwork.kernel.org/project/bluetooth/list/?series=1082520
Or is this on purpose, and we should consider the set complete?
--
Luiz Augusto von Dentz
More information about the Linux-mediatek
mailing list