From owner-p4-projects@FreeBSD.ORG Wed Apr 12 22:12:24 2006 Return-Path: X-Original-To: p4-projects@freebsd.org Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id 5303716A404; Wed, 12 Apr 2006 22:12:24 +0000 (UTC) X-Original-To: perforce@freebsd.org Delivered-To: perforce@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 0E2E616A401; Wed, 12 Apr 2006 22:12:24 +0000 (UTC) (envelope-from marcel@xcllnt.net) Received: from ns1.xcllnt.net (209-128-86-226.BAYAREA.NET [209.128.86.226]) by mx1.FreeBSD.org (Postfix) with ESMTP id AC20443D45; Wed, 12 Apr 2006 22:12:23 +0000 (GMT) (envelope-from marcel@xcllnt.net) Received: from [192.168.5.6] (dhcp06.vpn.xcllnt.net [192.168.5.6]) by ns1.xcllnt.net (8.13.4/8.13.4) with ESMTP id k3CMCMN1074703; Wed, 12 Apr 2006 15:12:22 -0700 (PDT) (envelope-from marcel@xcllnt.net) In-Reply-To: <200604121202.29736.jhb@freebsd.org> References: <200604121450.k3CEoUJh071640@repoman.freebsd.org> <200604121202.29736.jhb@freebsd.org> Mime-Version: 1.0 (Apple Message framework v749.3) Content-Type: text/plain; charset=US-ASCII; format=flowed Message-Id: <014B1689-1A9A-4AC3-BE76-AF8E35FF0A9C@xcllnt.net> Content-Transfer-Encoding: 7bit From: Marcel Moolenaar Date: Wed, 12 Apr 2006 14:11:16 -0700 To: John Baldwin X-Mailer: Apple Mail (2.749.3) Cc: Perforce Change Reviews Subject: Re: PERFORCE change 95068 for review X-BeenThere: p4-projects@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: p4 projects tree changes List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 12 Apr 2006 22:12:24 -0000 On Apr 12, 2006, at 9:02 AM, John Baldwin wrote: > On Wednesday 12 April 2006 10:50, Marcel Moolenaar wrote: >> http://perforce.freebsd.org/chv.cgi?CH=95068 >> >> Change 95068 by marcel@marcel_nfs on 2006/04/12 14:49:30 >> >> Replace the rmask and rval arrays with vendor, device, subvendor >> and subdevice variables. When subvendor is 0xffff, don't test the >> subvendor and subdevice values. This strips 24 bytes from the >> description. >> While here, sort the list on vendor, device, subvendor and >> subdevice. > > I guess you expanded any entries that were using masks other than > 0xffff? (I think there were only a handful if any). Not yet. I looked at the latest pucdata.c in NetBSD and more entries with masks other than 0xffff have been added. While the handful we had looked like mistakes (was 0xfffc, should probably be 0xfffe), the new ones don't. I need to investigate, because if the mask is used as a quick and dirty way to add multiple boards with a single entry then I'm not sure it's a good thing. For different boards, different descriptions are better. In that case I will expand. Otherwise I may end up bringing back the mask. The entries in question have a /* MASK */ comment as a reminder. -- Marcel Moolenaar USPA: A-39004 marcel@xcllnt.net