Re: [GnomeMeeting-list] [PATCH] Add USE_CURRENT_HARDWARE_SIZE hint

[Damien Sandras <dsandras seconix com>]

I agree with what you said, I just want to point that even it is true
for too many camera drivers (I won't name any), it doesn't apply to the
PWC driver written by Nemosoft.

I have been using it since a few years now, and can say that it works
really fine and seems to comply with the spec.

However, the bug we are discussing, seems to be new (at least the
behavior is new) and is due to the fact that Philips doesn't want to
publixh the whole spec, people thus need a binary module to have full
functionnality. The driver just supposes that binary module is loaded
when returning max. capabilities. 

Another fix for that problem, Derek, in pwlib, would be to initialize
frameWidth and frameHeight to the minimum returned values and not to the
maximum values. It will always work and wouldn't require the hint in
that case. Moreover, It shouldn't break other drivers.

Le mar 23/09/2003 à 00:21, Derek Smithies a écrit :
> Damien, all,
> 
>  I am sorry, I have extended this thread into a flame against video
>  driver writers.
> 
> You see, the real problem is video driver writers who produce drivers
> that fail to meet the v4l spec.
> 
> 
> Derek.
> ================================
> On Mon, 22 Sep 2003, Damien Sandras wrote:
> 
> > Let's stop this thread. That is the second time a thread with you
> > degenerates in sort of a flame on this mailing list. 
> > I'll ignore futher mails, sorry but I have better things to do than to
> > repeat myself, and I've already explained several times the problems as
> > I was seeing them with your patch.
> > 
> > Le lun 22/09/2003 à 14:41, Chris Rankin a écrit :
> > > But you still think that I should recompile as UP for
> > > the sake of the pcwx module? This would "logically"
> > > remove the second CPU, if not physically. It's not as
> > > if an SMP version of pcwx has been provided, and
> > > there's no reason to suppose that it is safe to load
> > > the UP version into an SMP kernel. If it works for you
> > > then you are "lucky", nothing more.
> > > 
> > > The pcwx is an optional, non-GPL, binary-only module
> > > that will taint the kernel. Any one of these is a good
> > > reason not to load it. It is definitely not a good
> > > idea to hold pwc users hostage to it, which is
> > > basically what you're suggesting. Especially when
> > > gnomemeeting is more than capable of running without
> > > it.
> > > 
> > > However, I do agree that the pwc module should be
> > > modifed to report its capabilities more accurately
> > > when the decompressor is not loaded. But how long do
> > > you reckon it will be before any modified pwc module
> > > will be picked up by all the Linux distributors? I
> > > suspect that modified pwlib packages will be available
> > > a lot sooner.
> > > 
> > > Chris
> > > 
> > > 
> > > ________________________________________________________________________
> > > Want to chat instantly with your online friends?  Get the FREE Yahoo!
> > > Messenger http://mail.messenger.yahoo.co.uk
> > > _______________________________________________
> > > GnomeMeeting-list mailing list
> > > GnomeMeeting-list gnome org
> > > http://mail.gnome.org/mailman/listinfo/gnomemeeting-list
> > 
-- 
 _	Damien Sandras
(o-	GnomeMeeting: http://www.gnomemeeting.org/
//\	FOSDEM 2003:  http://www.fosdem.org
v_/_	H.323 phone:  callto://ils.seconix.com/dsandras seconix com
echo
'12245692587856285105409351sn[ln0=aln256%Pln256/snlbx]sb[q]salbxq'|dc