On 11/8/25 9:27 AM, Timur Tabi wrote: > On Sat, 2025-11-08 at 20:45 +0900, Alexandre Courbot wrote: >> To be precise, we need to supply this because of a shortcoming in the >> `register`` macro: it doesn't support read-only registers yet, and write >> support requires a `Default` implementation for its fields. This is >> subject to be fixed in the future but for now we need this little >> workaround. > > This definitely feels like something that needs a TODO comment.
I've drafted it this way, for the next patchset revision: /// Enum representation of the GPU generation. /// /// TODO: remove the `Default` trait implementation, and the `#[default]` /// attribute, once the register!() macro (which creates Architecture items) no /// longer requires it for read-only fields. thanks, -- John Hubbard
