[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