Re: Inclusion of the W996[87]CF driver in the kernel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]



> > There are some oddments that I'd move into headers,
> 
> For example? I tried to keep the header as small as possible including
> necessary defines and main structures only.

Linux people tend to put the macros and debug stuff into headers as
well. Its just style stuff - and not a claim there is a "right" style
just its the kernel style

> 	thread 1	-	read image
> 	thread 2 	-	change picture size
> 
> There's not a particular reason behind that. I used that procedure
> to avoid the settings to be changed if the requested frames aren't still
> captured. I can remove the incriminated lines of code (<5) if the policy
> has to be different.

Ah sorry. I was looking for locking, but in fact you defer the change.

> Other bits:
> For things that really can't happen we normally use BUG().
> 
> I don't understand, sorry..

If you've got something that cannot happen, we have a function called
BUG() which prints out the file/line and then a stack trace. Users tend
not to report "Odd.. this can't occur" messages, but do report stack
traces 8)

> What ahout the /proc interface? Can I keep it?

I don't see a problem for now. Gerd's final call not mine. The
sysfs stuff in 2.5 eventually should let people do far more, but 
lots has yet to migrate to it.

> I will send you the driver by email as soon as I totally "clean" it and
> the ovsensor module is included in the kernel 2.5.

Thanks

Alan





[Index of Archives]     [Linux DVB]     [Video Disk Recorder]     [Asterisk]     [Photo]     [DCCP]     [Netdev]     [Xorg]     [Util Linux NG]     [Xfree86]     [Free Photo Albums]     [Fedora Users]     [Fedora Women]     [ALSA Users]     [ALSA Devel]     [Linux USB]

Powered by Linux