From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753319AbZD3TYl (ORCPT ); Thu, 30 Apr 2009 15:24:41 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752347AbZD3TYb (ORCPT ); Thu, 30 Apr 2009 15:24:31 -0400 Received: from netrider.rowland.org ([192.131.102.5]:50530 "HELO netrider.rowland.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1751815AbZD3TYa (ORCPT ); Thu, 30 Apr 2009 15:24:30 -0400 Date: Thu, 30 Apr 2009 15:24:30 -0400 (EDT) From: Alan Stern X-X-Sender: stern@netrider.rowland.org To: Sergei Shtylyov cc: David VomLehn , , , Subject: Re: [PATCH 2/5] USB: Provide USB boot device availability notification In-Reply-To: <49F9C8E0.30808@ru.mvista.com> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 30 Apr 2009, Sergei Shtylyov wrote: > Alan Stern wrote: > > >>>+/* The mask of possible USB boot devices depends on what drivers and > >>>+ * options have been configured into the kernel. There are too many > >>>+ * USB network config options to list here, so just assume it is always > >>>+ * possible to have a USB network device. > >>>+ */ > >>>+static int usb_bootdev_mask = 0 > >>>+#ifdef CONFIG_USB_SERIAL_CONSOLE > >>>+ | BOOTDEV_CONSOLE_MASK > >>>+#endif > >>>+#if defined(CONFIG_USB_STORAGE) | defined(CONFIG_BLK_DEV_UB) > > >> It should be ||, not bitwise | here, no? > > > No, it should be bitwise |. The value being constructed is a union of > > disjoint mask bits. > > I meant the #if line only. Although it shouldn't matter really... Oh, sorry. Yes, you're right about that (and you're right that it doesn't really matter...). David, you can go ahead and change the patch, but don't bother to submit a new version for such a small thing. The next time this gets pushed it'll have the right operator. Alan Stern