[linux-dvb] Re: PROC_FS patch and other stuff

[Florian Schirmer <jolt@tuxbox.org>]

Hi,

> Any thoughts on directory/file layout in procfs? Is /proc/dvb even a good
> place? Any pointers I should follow while moving all proc_fs code to
> seperate file(s)?

Create lets say dvb_proc.c and .h. Define a struct dvb_proc_fe struct in 
dvb_proc.h and place all the required proc var things inside this struct. 
Then add this struct to the frontend struct, wrapped by #ifdef DVB_PROC_FS. 
So you blow up the fe_struct just by 3 lines.

Next move all the register unregister proc code to the dvb_proc.c and in the 
fe_register just call dvb_proc_register_frontend(&fe) (again wrapped by ifdef 
DVB_RPOC_FS)

Maybe it's a good idea to put the proc register/unregister hooks inside the 
dvb_device_register. Since it will become generic and you get proc infos for 
all device, not just the fe's. But i haven't checked this idea yet.

Hope that helps a bit,
     Florian



--
Info:
To unsubscribe send a mail to listar@linuxtv.org with "unsubscribe linux-dvb" as subject.