[PATCH v1 2/5] arm64: add ARMv8.2 id_aa64mmfr2 boiler plate

On 03/02/16 15:31, James Morse wrote:

Hi Suzuki,

quoted

quoted

+static struct arm64_ftr_bits ftr_id_aa64mmfr2[] = {
+    ARM64_FTR_END,

It will be good to see all the defined fields in id_aa64mmfr2 added here in
this patch. You could also move the definition of UA0 field from the next patch
to here.

Okay, what are the rules for unsupported features?

UAO support isn't added until a later patch, I assumed it was best to add the
definition as part of that patch.

The table up there is also used for SANITY checks. So even if Linux doesn't use
the fields directly, the sanity checks could run on the fields and report issues which
might be problematic for the normal operation (if at all any).

Also since you are adding the information about the new register to the table,
its good to have the fields defined along with this patch to make the UAO patch
a bit more cleaner, which would make use of it eventually.


Cheers
Suzuki