Thread (46 messages) 46 messages, 11 authors, 2025-06-16

Re: [PATCH 01/20] bitfield: introduce HWORD_UPDATE bitfield macros

From: Nicolas Frattaroli <nicolas.frattaroli@collabora.com>
Date: 2025-06-12 19:51:37
Also in: dri-devel, linux-clk, linux-media, linux-mmc, linux-pci, linux-phy, linux-pm, linux-rockchip, linux-sound, lkml, llvm

On Thursday, 12 June 2025 21:44:15 Central European Summer Time Jakub Kicinski wrote:
On Thu, 12 Jun 2025 20:56:03 +0200 Nicolas Frattaroli wrote:
quoted
Hardware of various vendors, but very notably Rockchip, often uses
32-bit registers where the upper 16-bit half of the register is a
write-enable mask for the lower half.
Please limit the spread of this weirdness to a rockchip or "hiword"
specific header. To a normal reader of bitfield.h these macros will
be equally confusing and useless.
That is how this change started out, and then a different maintainer told
me that this is a commonly used thing (see: the sunplus patch), and
Rockchip just happens to have a lot of these with consistent naming.

I believe normal readers of bitfield.h will be much more confused by the
undocumented concatenating macro soup at the end, but maybe that's just
me.

Best regards,
Nicolas Frattaroli

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help