From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] pci: Add the class_id support in pci probe Date: Thu, 19 May 2016 14:57:17 +0200 Message-ID: <243965117.o52RgkbZZP@xps13> References: <1462946895-49958-1-git-send-email-ziye.yang@intel.com> <1727235.C3RSOGSy8E@xps13> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org To: "Yang, Ziye" Return-path: Received: from mail-wm0-f42.google.com (mail-wm0-f42.google.com [74.125.82.42]) by dpdk.org (Postfix) with ESMTP id E26D69606 for ; Thu, 19 May 2016 14:57:28 +0200 (CEST) Received: by mail-wm0-f42.google.com with SMTP id a17so124897905wme.0 for ; Thu, 19 May 2016 05:57:28 -0700 (PDT) In-Reply-To: List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2016-05-19 12:18, Yang, Ziye: > From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] > 2016-05-11 14:08, Ziye Yang: > > + dev->id.class_id = (uint32_t)tmp && CLASS_ANY_ID; > > Should be a bitwise &. Why masking is needed? > [Ziye] Only 24bit info is needed. What are the other bits? Please put a comment in the code.