From owner-freebsd-multimedia Thu Apr 17 13:20:55 1997 Return-Path: Received: (from root@localhost) by freefall.freebsd.org (8.8.5/8.8.5) id NAA02342 for multimedia-outgoing; Thu, 17 Apr 1997 13:20:55 -0700 (PDT) Received: from rah.star-gate.com (rah.star-gate.com [204.188.121.18]) by freefall.freebsd.org (8.8.5/8.8.5) with ESMTP id NAA02335 for ; Thu, 17 Apr 1997 13:20:51 -0700 (PDT) Received: from rah.star-gate.com (localhost.star-gate.com [127.0.0.1]) by rah.star-gate.com (8.8.5/8.7.3) with ESMTP id NAA00528; Thu, 17 Apr 1997 13:20:42 -0700 (PDT) Message-Id: <199704172020.NAA00528@rah.star-gate.com> X-Mailer: exmh version 1.6.9 8/22/96 To: Steve Passe cc: multimedia@freebsd.org Subject: Re: Fxtv 0.4 In-reply-to: Your message of "Thu, 17 Apr 1997 11:00:16 MDT." <199704171700.LAA21089@Ilsa.StevesCafe.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Date: Thu, 17 Apr 1997 13:20:42 -0700 From: Amancio Hasty Sender: owner-multimedia@freebsd.org X-Loop: FreeBSD.org Precedence: bulk Hi Steve, Good point, for sure the code needs a little bit more comments. All in all I am very happy with the way that the driver has turned out. >From The Desk Of Steve Passe : > Hi, > > >In the same routine there is this comment: > > > >/* FIXME: why set colorbars for 1 instant ??? */ > > bt848->color_ctl = BT848_COLOR_CTL_COLOR_BARS; > >/* FIXME: undone 4 lines later, why bother ??? */ > > bt848->color_ctl = BT848_COLOR_CTL_GAMMA; > > > >Careful here because some of this stuff was written after I read > >Brooktree's DOS demo. Some of these chipsets are cranky and yes > >we can experiment by taking the lines out. > > the "FIXME" protocal is mine, while doing my cleanup passes I added those > lines wherever I saw something that needed attention. In many cases > it may be that the code is valid, but that it is something arcane > (cranky chipset really wants that bit set for no apparrent reason!) Oh Steve, some of this PC chipset manufacturers don't do a good job at quality assurance and whenever they can they have soft work arounds over hardware fixes. Chips like the Bt848 are complicated and is not feasible to fully test the chip. > so in this case the "FIX" would be a comment that explains why its there. > This way someone like me can come along and not wonder "what the h*** is that > there for???". Remember that this driver will have a long lifetime, and peop le > may be dealing it long after we're not available/have forgotten ourselves > the little details. Point well taken... Regards, Amancio