Re: [PATCH v1 0/8] zorro: Improve handling of pointers in zorro_device_id::driver_data
Uwe Kleine-König (The Capable Hub) <[email protected]>
| Newsgroups | org.kernel.vger.linux-m68k,org.freedesktop.lists.dri-devel,org.kernel.vger.linux-fbdev,org.kernel.vger.linux-i2c,org.kernel.vger.linux-ide,org.kernel.vger.linux-kernel,org.kernel.vger.linux-scsi,org.kernel.vger.netdev |
|---|---|
| Message-ID | <ahXMGtBv_pQXM7YS@monoceros> |
On Tue, May 26, 2026 at 05:01:48PM +0200, Geert Uytterhoeven wrote: > Hi Uwe, > > On Tue, 26 May 2026 at 16:17, Uwe Kleine-König (The Capable Hub) > <[email protected]> wrote: > > this series is about improving the handling of pointers in struct > > zorro_device_id's driver_data. > > > > While it's ok on all current Linux platforms to store a pointer in an > > unsigned long variable, it involves casting that loses type information. > > This can be nicely seen in patch #7 where after profiting from patch #6 > > the compiler notices a missing const. > > > > Preparing for that change, all zorro_device_ids are converted to use > > named initializers, which is also a nice cleanup that could stand for > > itself, as it improves readability for humans. (That is necessary > > because an anonymous union can be initialized by name, but not using a > > list initializer.) > > > > My motivation for this series is the CHERI hardware extension. With that > > pointers are bigger than longs and thus you cannot store pointers in > > zorro_device_id::driver_data. So this series is also about getting > > support for CHERI into the mainline, but I hope the clean up effects > > mentioned above are justification enough to accept this series. > > Thanks for your series! > > > The dependencies in this series are as follows: > > > > - Patch #5 depends on #1, #2 > > s/5/6/? > > > - Patches #7 and #8 depend on patch #6. > > > > So if the ata maintainers agreed to merge their patch #1 via scsi, and > > Geert agrees to patch #5 and that it's also merged via scsi, patches #1, > > s/5/6/? Yes, indeed. And I tried so hard to get all the numbers right :-\ Best regards Uwe
signature.asc
(application/pgp-signature, 488 B)
-----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEP4GsaTp6HlmJrf7Tj4D7WH0S/k4FAmoVzHQACgkQj4D7WH0S /k4BXggAoAuTCgxPow5/rQYRxSTj+kNc4XmxhIlX39wkjq4KJoBOkfK1zOhQISV1 4HHCqWa9m9dov4bu6Kg8LwbKa2UChzh7HIDanvg0FuTjuR2DvZAqQ1h3KlZHQj4H Dh1fxxwA9H2mAUN8tHgeBxUVZ6qajflOjKFHpsQbdgTyoj2LkR1szY77e5P4QdmS 0UJbs5rWuTPA+ErMqqdl4SkAygYx8SEw84ch7byHO4yx9rpUCp8zJk95dKJCgAir L38wVLJUZNR7sQVkPKakWtv34niUdKWjqIXa8yB4GHOQFX65uFKEzrvodPCyE2J/ vgh7fbUvdt3OqTL+DpSduQS2l1F1qA== =W3Nr -----END PGP SIGNATURE-----