On Wed, Mar 1, 2023 at 2:19 PM Ralph Siemsen <ralph.siemsen@xxxxxxxxxx> wrote: > > > > The bulk of this patch converts the clock table to use struct regbit, > > > making use of the RB() helper macro. The conversion was automated by > > > script, and as a further verification, the compiled binary of the table > > > was compared before/after the change (with objdump -D). > > > > I will trust your tool on the conversion. > > I'm going to check again using objdump, just to make sure nothing slips through. So naturally, this test uncovered a difference. It seems that gcc orders bitfields differently on x86 and ARM. So the tables did not match exactly with the old packed u16 fomat. It is a harmless difference, but it can easily be eliminated, by swapping the fields within 'struct regbit'. Now the tables are really identical. Patch v2 to follow shortly. Ralph
