On Tue, Mar 10, 2015 at 08:13:58AM -0400, Quan Xu wrote: > Signed-off-by: Quan Xu <quan...@intel.com> > --- > tools/firmware/hvmloader/acpi/build.c | 7 ++++--- > tools/libxl/libxl_create.c | 5 ++++- > 2 files changed, 8 insertions(+), 4 deletions(-) > > diff --git a/tools/firmware/hvmloader/acpi/build.c > b/tools/firmware/hvmloader/acpi/build.c > index 1431296..49f6772 100644 > --- a/tools/firmware/hvmloader/acpi/build.c > +++ b/tools/firmware/hvmloader/acpi/build.c > @@ -313,9 +313,10 @@ static int construct_secondary_tables(unsigned long > *table_ptrs, > > /* TPM TCPA and SSDT. */ > tis_hdr = (uint16_t *)0xFED40F00; > - if ( (tis_hdr[0] == tis_signature[0]) && > - (tis_hdr[1] == tis_signature[1]) && > - (tis_hdr[2] == tis_signature[2]) ) > + if (((tis_hdr[0] == tis_signature[0]) && > + (tis_hdr[1] == tis_signature[1]) &&
These two lines are not necessary. They are not functional change and violate coding style. Wei. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
