2008-07-11 08:52:30

by Ohad Ben-Cohen

[permalink] [raw]
Subject: Re: [Bluez-devel] [PATCH] Add support for Texas Instruments' chips

On Fri, Jul 11, 2008 at 11:50 AM, Marcel Holtmann <[email protected]> wrote:
> done.

Marcel, thank you for your work.

BR,
Ohad.

>
> Regards
>
> Marcel
>
>
>
> -------------------------------------------------------------------------
> Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
> Studies have shown that voting for your favorite open source project,
> along with a healthy diet, reduces your potential for chronic lameness
> and boredom. Vote Now at http://www.sourceforge.net/community/cca08
> _______________________________________________
> Bluez-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/bluez-devel
>

-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
Bluez-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel

2008-07-11 08:50:03

by Marcel Holtmann

[permalink] [raw]
Subject: Re: [Bluez-devel] [PATCH] Add support for Texas Instruments' chips

Hi Ohad,

> On Fri, Jul 11, 2008 at 11:37 AM, Marcel Holtmann <[email protected]> wrote:
> > No I didn't. You sent it without any copyright note. Please check your
> > original patch.
>
> You are right, it was only on my second patch with the separate
> hciattach_ti.c file.
> Can you please add:
>
> Copyright (C) 2007-2008 Texas Instruments, Inc.
>
> to the hciattach_ti.c file ?

done.

Regards

Marcel



-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
Bluez-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel

2008-07-11 08:38:40

by Ohad Ben-Cohen

[permalink] [raw]
Subject: Re: [Bluez-devel] [PATCH] Add support for Texas Instruments' chips

Hi Marcel,

On Fri, Jul 11, 2008 at 11:37 AM, Marcel Holtmann <[email protected]> wrote:
> No I didn't. You sent it without any copyright note. Please check your
> original patch.

You are right, it was only on my second patch with the separate
hciattach_ti.c file.
Can you please add:

Copyright (C) 2007-2008 Texas Instruments, Inc.

to the hciattach_ti.c file ?

Thanks,
Ohad.

>
> Regards
>
> Marcel
>
>
>
> -------------------------------------------------------------------------
> Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
> Studies have shown that voting for your favorite open source project,
> along with a healthy diet, reduces your potential for chronic lameness
> and boredom. Vote Now at http://www.sourceforge.net/community/cca08
> _______________________________________________
> Bluez-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/bluez-devel
>

-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
Bluez-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel

2008-07-11 08:37:32

by Marcel Holtmann

[permalink] [raw]
Subject: Re: [Bluez-devel] [PATCH] Add support for Texas Instruments' chips

Hi Ohad,

> > I did the hard work and put this into CVS as I see it should be done.
> > Please double check if that is working for you.
>
> It is.

good.

> One note: you took away TI's Copyright line and I don't think lawyers here will like that. Can you please add it back ?

No I didn't. You sent it without any copyright note. Please check your
original patch.

Regards

Marcel



-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
Bluez-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel

2008-07-11 08:21:13

by Ben-cohen, Ohad

[permalink] [raw]
Subject: Re: [Bluez-devel] [PATCH] Add support for Texas Instruments' chips

Hi Marcel,

> I did the hard work and put this into CVS as I see it should be done.
> Please double check if that is working for you.

It is.

One note: you took away TI's Copyright line and I don't think lawyers here will like that. Can you please add it back ?

Thank you for your work!
Ohad.


-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
Bluez-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel

-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
Bluez-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel

2008-07-11 06:14:57

by Marcel Holtmann

[permalink] [raw]
Subject: Re: [Bluez-devel] [PATCH] Add support for Texas Instruments' chips

Hi Ohad,

> > please put all the TI code into hciattach_ti.c and then I gonna apply
> > this patch.
>
> I'm gonna have to put some common definitions in hciattach.h then
> (mainly struct uart_t).

I did the hard work and put this into CVS as I see it should be done.
Please double check if that is working for you.

Regards

Marcel



-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
Bluez-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel

2008-07-10 12:33:23

by Marcel Holtmann

[permalink] [raw]
Subject: Re: [Bluez-devel] [PATCH] Add support for Texas Instruments' chips

Hi Ohad,

> > please put all the TI code into hciattach_ti.c and then I gonna apply
> > this patch.
>
> I'm gonna have to put some common definitions in hciattach.h then
> (mainly struct uart_t).
>
> Is that ok ?

no. Do the same as I did for hciattach_st.c

Regards

Marcel



-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
Bluez-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel

2008-07-10 05:36:38

by Ohad Ben-Cohen

[permalink] [raw]
Subject: Re: [Bluez-devel] [PATCH] Add support for Texas Instruments' chips

Hi Marcel,

On Thu, Jul 10, 2008 at 7:07 AM, Marcel Holtmann <[email protected]> wrote:
> please put all the TI code into hciattach_ti.c and then I gonna apply
> this patch.

I'm gonna have to put some common definitions in hciattach.h then
(mainly struct uart_t).

Is that ok ?

(you previously objected to the creation of hciattach.h)

Thanks,
Ohad.

>
> Regards
>
> Marcel
>
>
>
> -------------------------------------------------------------------------
> Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
> Studies have shown that voting for your favorite open source project,
> along with a healthy diet, reduces your potential for chronic lameness
> and boredom. Vote Now at http://www.sourceforge.net/community/cca08
> _______________________________________________
> Bluez-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/bluez-devel
>

-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
Bluez-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel

2008-07-10 04:07:07

by Marcel Holtmann

[permalink] [raw]
Subject: Re: [Bluez-devel] [PATCH] Add support for Texas Instruments' chips

Hi Ohad,

> See attached.

please put all the TI code into hciattach_ti.c and then I gonna apply
this patch.

Regards

Marcel



-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
Bluez-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel