--- On Mon, 4/20/09, Mauro Carvalho Chehab <[email protected]> wrote:
> From: Mauro Carvalho Chehab <[email protected]>
> Subject: Re: [PATCH] [0904_1] Siano: core header - update license and include
> files
> To: "Uri Shkolnik" <[email protected]>
> Cc: [email protected]
> Date: Monday, April 20, 2009, 8:01 PM
> On Mon, 20 Apr 2009 09:40:42 -0700
> (PDT)
> Uri Shkolnik <[email protected]>
> wrote:
>
> >
> >
> >
> > --- On Mon, 4/20/09, Mauro Carvalho Chehab <[email protected]>
> wrote:
> >
> > > From: Mauro Carvalho Chehab <[email protected]>
> > > Subject: Re: [PATCH] [0904_1] Siano: core header
> - update license and include files
> > > To: "Uri Shkolnik" <[email protected]>
> > > Cc: [email protected]
> > > Date: Monday, April 20, 2009, 5:42 PM
> > > On Sun, 5 Apr 2009 01:09:16 -0700
> > > (PDT)
> > > Uri Shkolnik <[email protected]>
> > > wrote:
> > >
> > > >
> > > > # HG changeset patch
> > > > # User Uri Shkolnik <[email protected]>
> > > > # Date 1238689930 -10800
> > > > # Node ID
> c3f0f50d46058f07fb355d8e5531f35cfd0ca37e
> > > > # Parent
> > > 7311d23c3355629b617013cd51223895a2423770
> > > > [PATCH] [0904_1] Siano: core header - update
> license
> > > and included files
> > > >
> > > > From: Uri Shkolnik <[email protected]>
> > > >
> > > > This patch does not include any
> implementation
> > > changes.
> > > > It update the smscoreapi.h license to be
> identical to
> > >
> > > > other Siano's headers and the #include files
> list.
> > >
> > > s/update/updates/
> > >
> > > > #include <linux/version.h>
> > > > #include <linux/device.h>
> > > > @@ -28,15 +28,23 @@
> > > > #include <linux/mm.h>
> > > > #include <linux/scatterlist.h>
> > > > #include <linux/types.h>
> > > > +#include <linux/mutex.h>
> > > > +#include <linux/compat.h>
> > > > +#include <linux/wait.h>
> > > > +#include <linux/timer.h>
> > > > +
> > > > #include <asm/page.h>
> > > > -#include <linux/mutex.h>
> > > > -#include "compat.h"
> > >
> > > Hmm... Why do you need the above changes? Also,
> #include
> > > "compat.h" is
> > > required, in order to compile inside the
> out-of-tree kernel
> > > tree.
> > >
> > > Also, the header changes should be on a
> different
> > > changeset, since they aren't
> > > related to what's described, e. g. this has
> nothing to do
> > > with licensing change.
> > >
> > >
> > > Cheers,
> > > Mauro
> > >
> >
> > 1) "compat.h" became <linux/compat.h> as result
> of old ML review
> > ---> +#include <linux/compat.h>
>
> I have no idea when do you need to include linux/compat.h.
> However, as
> compilation is currently fine, I see no reasons why to add
> it. I also don't
> have any idea why do you need to add other include files,
> since it is properly
> compiling without adding any other header.
>
> In the case of "compat.h", this is local to the out-of-tree
> compilation, having
> some needed defines to compile against older kernel
> versions. This header it is
> automatically stripped from upstream changes.
>
> > 2) There were a mail exchanged, back in mid-summer
> 2008, regarding the license. One template has been approved
> both by Siano and the reviewers back then, and the patch
> comes the align this particular file with that old
> decision.
>
> This seems fine to my eyes.
>
> > Regarding the change-set - since there were no
> implementation changes (only license text modification and
> re-arranging the include files list (I hadn't counted
> "compat.h" --> <linux/compat.h> as an
> implementation change) I decided to put them in one patch.
> If higher resolution is needed, I'll do so,
>
> If all you're doing is rearranging, it would be fine to add
> it at the same
> changeset, but you should explicitly mention this at the
> description.
>
> Also, fyi, the proper include sequence is:
>
> 1) Include all kernel headers that aren't at -hg (no
> particular order here - I
> generally use some alphabetic order, but this is just my
> personal preference);
>
> 2) #include "compat.h"
>
> 3) The other v4l/dvb core headers and local headers.
>
> Cheers,
> Mauro
>
Just to make sure (sorry to be a little nagger about it...)
Should I ignore the old request to replace "compat.h" with <linux/compat.h>,
and stay with "compat.h" ?
10x,
Uri
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
