From owner-freebsd-multimedia Thu Apr 17 10:00:29 1997 Return-Path: Received: (from root@localhost) by freefall.freebsd.org (8.8.5/8.8.5) id KAA20999 for multimedia-outgoing; Thu, 17 Apr 1997 10:00:29 -0700 (PDT) Received: from Ilsa.StevesCafe.com (Ilsa.StevesCafe.com [205.168.119.129]) by freefall.freebsd.org (8.8.5/8.8.5) with ESMTP id KAA20987 for ; Thu, 17 Apr 1997 10:00:25 -0700 (PDT) Received: from Ilsa.StevesCafe.com (localhost [127.0.0.1]) by Ilsa.StevesCafe.com (8.8.5/8.8.5) with ESMTP id LAA21089; Thu, 17 Apr 1997 11:00:16 -0600 (MDT) Message-Id: <199704171700.LAA21089@Ilsa.StevesCafe.com> X-Mailer: exmh version 2.0gamma 1/27/96 From: Steve Passe To: Amancio Hasty cc: multimedia@freebsd.org Subject: Re: Fxtv 0.4 In-reply-to: Your message of "Thu, 17 Apr 1997 00:25:20 PDT." <199704170725.AAA00325@rah.star-gate.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Date: Thu, 17 Apr 1997 11:00:16 -0600 Sender: owner-multimedia@freebsd.org X-Loop: FreeBSD.org Precedence: bulk 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!) 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 people may be dealing it long after we're not available/have forgotten ourselves the little details. -- Steve Passe | powered by smp@csn.net | Symmetric MultiProcessor FreeBSD