[PATCH 07/12] sbbr/madt: Add initial tests to madt acpi table as per sbbr.

Leif Lindholm leif.lindholm at linaro.org
Tue Mar 7 08:11:22 UTC 2017


On Mon, Mar 06, 2017 at 11:59:36AM -0600, Supreeth Venkatesh wrote:
> On Thu, 2017-03-02 at 23:39 +0000, Colin Ian King wrote:
> > This seems to be derived from the other fwts MADT test, so we've got
> > code duplication here. Any specific reasons?
> > 
> Yes. there is some code re-use. But this removes x86 specific Interrupt
> controller tests and retains only GIC tests with additional tests
> added.

This separation should be done conditionally, based on whether the
--sbbr option has been specified, or at compile time per architecture
for any tables that are architecture specific.

Not by duplicating code (and hence the maintenance burden).

/
    Leif



More information about the fwts-devel mailing list