From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dan Carpenter Date: Mon, 10 Dec 2012 08:44:27 +0000 Subject: Re: [patch] x86, efi: precedence error in setup_efi_pci() Message-Id: <20121210084426.GQ22569@mwanda> List-Id: References: <20121207061649.GA18220@elgon.mountain> <50C2AE91.9060800@zytor.com> In-Reply-To: <50C2AE91.9060800-YMNOUZJC4hwAvxtiuMwx3w@public.gmane.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: "H. Peter Anvin" Cc: Matt Fleming , Matthew Garrett , Thomas Gleixner , Ingo Molnar , x86-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, kernel-janitors-u79uwXL29TY76Z2rM5mHXA@public.gmane.org On Fri, Dec 07, 2012 at 07:05:53PM -0800, H. Peter Anvin wrote: > On 12/06/2012 10:16 PM, Dan Carpenter wrote: > >Negate has higher precedence than bitwise AND so the condition is always > >false. > > > >Signed-off-by: Dan Carpenter > > > >diff --git a/arch/x86/boot/compressed/eboot.c b/arch/x86/boot/compressed/eboot.c > >index b1942e2..18e329c 100644 > >--- a/arch/x86/boot/compressed/eboot.c > >+++ b/arch/x86/boot/compressed/eboot.c > >@@ -302,7 +302,7 @@ static efi_status_t setup_efi_pci(struct boot_params *params) > > if (status != EFI_SUCCESS) > > continue; > > > >- if (!attributes & EFI_PCI_IO_ATTRIBUTE_EMBEDDED_ROM) > >+ if (!(attributes & EFI_PCI_IO_ATTRIBUTE_EMBEDDED_ROM)) > > continue; > > > > if (!pci->romimage || !pci->romsize) > > > > What is the baseline for this patch? > Sorry, this is a linux-next bug. Introduced in: commit dd5fc854de5fd37adfcef8a366cd21a55aa01d3d Author: Matthew Garrett Date: Wed Dec 5 14:33:26 2012 -0700 EFI: Stash ROMs if they're not in the PCI BAR regards, dan carpenter