2008/9/5 David Sharp <[EMAIL PROTECTED]>
> geez, I had no idea you were including C files... Please, let's clean that
> up.
>
First, let me defend myself, I didn't create the original structure of the
driver, I just continued to build upon it until now.
>
> Can you send a patch to the list for us to review? That's easier than
> guessing what exactly those arrows mean in your diagram.
>
Sure,
I'll merge a new branch tonight.
Regards,
Erik
>
> 2008/9/4 Erik Andrén <[EMAIL PROTECTED]>:
> > Hi,
> >
> > The current driver structure with only one source file including other .c
> > files directly isn't pretty and causes unnecessary name-space collisions.
> > (The same define may not exist in several sensors, for instance.)
> > Also, the current structure isn't compatible when linux kernel
> integration.
> >
> > I've been playing with an alternative reorganization and has come up with
> > the following structure:
> >
> > The core functionality is stored in
> > m5602_core.c with the v4l2 specific functions remaining in m5602_v4l2.c
> >
> > The sensor code remain as before but includes a m5602_sensor.h header
> > containing sensor specific defines and data structures.
> >
> > m5602_sensor.h in turn calls m5602_bridge.h containing m5602 specific
> data
> > structures.
> >
> > The attached picture contains a dependency graph.
> >
> > If anyone has some comments regarding this structure, please voice them
> > soon, or else I will merge this change into the m5602 branch.
> >
> > Regards,
> > Erik
> >
> >
> >
> > -------------------------------------------------------------------------
> > This SF.Net email is sponsored by the Moblin Your Move Developer's
> challenge
> > Build the coolest Linux based applications with Moblin SDK & win great
> > prizes
> > Grand prize is a trip for two to an Open Source event anywhere in the
> world
> > http://moblin-contest.org/redirect.php?banner_id=100&url=/
> > _______________________________________________
> > M560x-driver-devel mailing list
> > [email protected]
> > https://lists.sourceforge.net/lists/listinfo/m560x-driver-devel
> >
> >
>
-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
M560x-driver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/m560x-driver-devel
