All of lore.kernel.org
 help / color / mirror / Atom feed
* Upstreaming SAA716x driver to the media_tree
@ 2014-01-07 11:58 Luis Alves
  2014-01-07 13:10 ` Manu Abraham
  2014-01-07 16:12 ` Andreas Regel
  0 siblings, 2 replies; 17+ messages in thread
From: Luis Alves @ 2014-01-07 11:58 UTC (permalink / raw)
  To: linux-media
  Cc: andreas.regel, updatelee, abraham.manu, crazycat69,
	Mauro Carvalho Chehab, Antti Palosaari

Hi,

I'm finishing a new frontend driver for one of my dvb cards, but the
pcie bridge uses the (cursed) saa716x.
As far as I know the progress to upstream Manu's driver to the
media_tree has stalled.

In CC I've placed some of the people that I found working on it
lately, supporting a few dvb cards.

It would be good if we could gather everything in one place and send a
few patchs to get this upstreamed for once...

Manu, do you see any inconvenience in sending your driver to the
linux_media tree?
I'm available to place some effort on this task.

Regards,
Luis Alves

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: Upstreaming SAA716x driver to the media_tree
  2014-01-07 11:58 Upstreaming SAA716x driver to the media_tree Luis Alves
@ 2014-01-07 13:10 ` Manu Abraham
  2014-01-07 15:59   ` Luis Alves
       [not found]   ` <CALzAhNW=uCLsu4sdrGyycxRoOdaRTFNM8yVj=Y+ahSZrWqrABg@mail.gmail.com>
  2014-01-07 16:12 ` Andreas Regel
  1 sibling, 2 replies; 17+ messages in thread
From: Manu Abraham @ 2014-01-07 13:10 UTC (permalink / raw)
  To: Luis Alves
  Cc: linux-media, Andreas Regel, Chris Lee, crazycat69,
	Mauro Carvalho Chehab, Antti Palosaari

Hi Luis,


On Tue, Jan 7, 2014 at 5:28 PM, Luis Alves <ljalvs@gmail.com> wrote:
> Hi,
>
> I'm finishing a new frontend driver for one of my dvb cards, but the
> pcie bridge uses the (cursed) saa716x.
> As far as I know the progress to upstream Manu's driver to the
> media_tree has stalled.
>
> In CC I've placed some of the people that I found working on it
> lately, supporting a few dvb cards.
>
> It would be good if we could gather everything in one place and send a
> few patchs to get this upstreamed for once...
>
> Manu, do you see any inconvenience in sending your driver to the
> linux_media tree?
> I'm available to place some effort on this task.


I can push the 716x driver and whatever additional changes that I have
later on this weekend, if that's okay with you.


Regards,

Manu

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: Upstreaming SAA716x driver to the media_tree
  2014-01-07 13:10 ` Manu Abraham
@ 2014-01-07 15:59   ` Luis Alves
       [not found]   ` <CALzAhNW=uCLsu4sdrGyycxRoOdaRTFNM8yVj=Y+ahSZrWqrABg@mail.gmail.com>
  1 sibling, 0 replies; 17+ messages in thread
From: Luis Alves @ 2014-01-07 15:59 UTC (permalink / raw)
  To: Manu Abraham
  Cc: linux-media, Andreas Regel, Chris Lee, crazycat69,
	Mauro Carvalho Chehab, Antti Palosaari

Hi Manu,

That would be great.
Let me know if you need a hand!

Regards,
Luis


On Tue, Jan 7, 2014 at 1:10 PM, Manu Abraham <abraham.manu@gmail.com> wrote:
> Hi Luis,
>
>
> On Tue, Jan 7, 2014 at 5:28 PM, Luis Alves <ljalvs@gmail.com> wrote:
>> Hi,
>>
>> I'm finishing a new frontend driver for one of my dvb cards, but the
>> pcie bridge uses the (cursed) saa716x.
>> As far as I know the progress to upstream Manu's driver to the
>> media_tree has stalled.
>>
>> In CC I've placed some of the people that I found working on it
>> lately, supporting a few dvb cards.
>>
>> It would be good if we could gather everything in one place and send a
>> few patchs to get this upstreamed for once...
>>
>> Manu, do you see any inconvenience in sending your driver to the
>> linux_media tree?
>> I'm available to place some effort on this task.
>
>
> I can push the 716x driver and whatever additional changes that I have
> later on this weekend, if that's okay with you.
>
>
> Regards,
>
> Manu

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: Upstreaming SAA716x driver to the media_tree
  2014-01-07 11:58 Upstreaming SAA716x driver to the media_tree Luis Alves
  2014-01-07 13:10 ` Manu Abraham
@ 2014-01-07 16:12 ` Andreas Regel
  2014-01-07 16:33   ` Luis Alves
  1 sibling, 1 reply; 17+ messages in thread
From: Andreas Regel @ 2014-01-07 16:12 UTC (permalink / raw)
  To: Luis Alves, linux-media
  Cc: updatelee, abraham.manu, crazycat69, Mauro Carvalho Chehab,
	Antti Palosaari

Hi Luis,

Am 07.01.2014 12:58, schrieb Luis Alves:
> Hi,
> 
> I'm finishing a new frontend driver for one of my dvb cards, but the
> pcie bridge uses the (cursed) saa716x.
> As far as I know the progress to upstream Manu's driver to the
> media_tree has stalled.
> 
> In CC I've placed some of the people that I found working on it
> lately, supporting a few dvb cards.
> 
> It would be good if we could gather everything in one place and send a
> few patchs to get this upstreamed for once...
> 
> Manu, do you see any inconvenience in sending your driver to the
> linux_media tree?
> I'm available to place some effort on this task.

which repository of the saa761x is your work based on?

Regards,
Andreas


^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: Upstreaming SAA716x driver to the media_tree
  2014-01-07 16:12 ` Andreas Regel
@ 2014-01-07 16:33   ` Luis Alves
  2014-01-07 17:31     ` Konstantin Dimitrov
  0 siblings, 1 reply; 17+ messages in thread
From: Luis Alves @ 2014-01-07 16:33 UTC (permalink / raw)
  To: Andreas Regel
  Cc: linux-media, Chris Lee, Manu Abraham, crazycat69,
	Mauro Carvalho Chehab, Antti Palosaari

HI Andreas,

My initial commit is based on:
http://powarman.dyndns.org/hgwebdir.cgi/v4l-dvb-saa716x/
(I think it's your repo with some commits from Soeren Moch)

The difference to my working area is that I have the driver placed in
"drivers/media/pci/saa716x" (instead of
"drivers/media/common/saa716x") and everything is rebased on the
latest media_tree.
On top of that I just have 2 commits: one to be able to build FF cards
and another to fix some i2c issues.

You can check my repo here:
https://github.com/ljalves/linux_media/commits/saa716x

Regards,
Luis


On Tue, Jan 7, 2014 at 4:12 PM, Andreas Regel <andreas.regel@gmx.de> wrote:
> Hi Luis,
>
> Am 07.01.2014 12:58, schrieb Luis Alves:
>> Hi,
>>
>> I'm finishing a new frontend driver for one of my dvb cards, but the
>> pcie bridge uses the (cursed) saa716x.
>> As far as I know the progress to upstream Manu's driver to the
>> media_tree has stalled.
>>
>> In CC I've placed some of the people that I found working on it
>> lately, supporting a few dvb cards.
>>
>> It would be good if we could gather everything in one place and send a
>> few patchs to get this upstreamed for once...
>>
>> Manu, do you see any inconvenience in sending your driver to the
>> linux_media tree?
>> I'm available to place some effort on this task.
>
> which repository of the saa761x is your work based on?
>
> Regards,
> Andreas
>

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: Upstreaming SAA716x driver to the media_tree
  2014-01-07 16:33   ` Luis Alves
@ 2014-01-07 17:31     ` Konstantin Dimitrov
  2014-01-07 20:23       ` Luis Alves
  2014-01-07 23:42       ` CrazyCat
  0 siblings, 2 replies; 17+ messages in thread
From: Konstantin Dimitrov @ 2014-01-07 17:31 UTC (permalink / raw)
  To: Luis Alves
  Cc: Andreas Regel, linux-media, Chris Lee, Manu Abraham, crazycat69,
	Mauro Carvalho Chehab, Antti Palosaari

Luis,

can you explain to us all here how exactly you came up to those
particular I2C fixes:

https://github.com/ljalves/linux_media/commit/be7cd1ff82cc20578b805ad508d089f818ae726d

because essentially they are the same as what i did years ago -
included as source code in drivers i made for some TBS cards (source
code is available all over online) or we just have the exact same case
with you as before:

http://www.spinics.net/lists/linux-media/msg65888.html

http://www.spinics.net/lists/linux-media/msg65889.html

and you're continue to taking credit for patches i made or basically
stealing them.

if they were some trivial patches i won't mind, but even they are
small, they are nothing like trivial.

so, i believe you will have really hard time to explain "your" I2C
fixes, because for example the SDA hold time value of 0x14 is needed
for only one particular SAA716x-based card (other such cards can work
with wide range of SDA hold time settings) and i'm sure you don't know
that card and cannot cite its model or any technical details why
that's needed, because you don't have it, as well it takes quite an
effort and good knowledge of I2C signaling with oscilloscope to figure
out that value, as well that exactly that value needs changing.

why you didn't use for example 0x16 or 0x13 for SDA hold time in
"your" I2C patch?!

so, one time, like the previous time, excuse that you just didn't know
who the author of that work is may fly, but second time, especially
considering that the SAA716x code base from which i'm sure you took
(not to use stole) those settings contains my name as copyright,
because i actually added to that code base new code i developed from
scratch like for example saa716x_input.[c|h], is another thing you
cannot explain.

so, i was waiting Manu to upstream his SAA716x driver code some day
and then submit the improvements i made to it. yet again you're trying
to take that from me and again, conveniently you included many people
on CC, but not me.

in my opinion what you're doing is not right, because that patch is
not clean-room reverse-engineering, you just took those changes from
another open-source base and if nothing else it's at least common
courtesy in open-source community when you didn't make them to not
submit them as "your" patches.

i also think with your actions you're actually hurting the community,
because people like me, that do actually have the technical
understanding and can help and contribute further improvements are
driven away from the community, because
effectively the community accepting behavior like yours is encouraging
code stealing!!

--konstantin

On Tue, Jan 7, 2014 at 6:33 PM, Luis Alves <ljalvs@gmail.com> wrote:
> HI Andreas,
>
> My initial commit is based on:
> http://powarman.dyndns.org/hgwebdir.cgi/v4l-dvb-saa716x/
> (I think it's your repo with some commits from Soeren Moch)
>
> The difference to my working area is that I have the driver placed in
> "drivers/media/pci/saa716x" (instead of
> "drivers/media/common/saa716x") and everything is rebased on the
> latest media_tree.
> On top of that I just have 2 commits: one to be able to build FF cards
> and another to fix some i2c issues.
>
> You can check my repo here:
> https://github.com/ljalves/linux_media/commits/saa716x
>
> Regards,
> Luis
>
>
> On Tue, Jan 7, 2014 at 4:12 PM, Andreas Regel <andreas.regel@gmx.de> wrote:
>> Hi Luis,
>>
>> Am 07.01.2014 12:58, schrieb Luis Alves:
>>> Hi,
>>>
>>> I'm finishing a new frontend driver for one of my dvb cards, but the
>>> pcie bridge uses the (cursed) saa716x.
>>> As far as I know the progress to upstream Manu's driver to the
>>> media_tree has stalled.
>>>
>>> In CC I've placed some of the people that I found working on it
>>> lately, supporting a few dvb cards.
>>>
>>> It would be good if we could gather everything in one place and send a
>>> few patchs to get this upstreamed for once...
>>>
>>> Manu, do you see any inconvenience in sending your driver to the
>>> linux_media tree?
>>> I'm available to place some effort on this task.
>>
>> which repository of the saa761x is your work based on?
>>
>> Regards,
>> Andreas
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: Upstreaming SAA716x driver to the media_tree
  2014-01-07 17:31     ` Konstantin Dimitrov
@ 2014-01-07 20:23       ` Luis Alves
  2014-01-08 21:21         ` Konstantin Dimitrov
  2014-01-07 23:42       ` CrazyCat
  1 sibling, 1 reply; 17+ messages in thread
From: Luis Alves @ 2014-01-07 20:23 UTC (permalink / raw)
  To: Konstantin Dimitrov
  Cc: Andreas Regel, linux-media, Chris Lee, Manu Abraham, crazycat69,
	Mauro Carvalho Chehab, Antti Palosaari

Hi Konstantin,
(here we go again...)

What the hell are you talking about? I didn't submit any I2C patch.
This email is just about Manu sending his SAA716x driver to the media_tree.

And where do you see any "saa716x_input.[c|h]" in my repo?

Anyway, since you asked I took some pics:
https://plus.google.com/photos/105602732859464871628/albums/5966247612074668305?authkey=CN3jl9mWhrDhQg
(SDA_HOLD setting in the comment)

For SDA_HOLD = 0x19 the next clock rising edge is really close to the
data line release.
Even 0x14 is too close, so I will use a smaller value so maybe 0x10 is
fine for the 400kHZ clk speed.

And I do have a TBS card that doesn't work with Manu default setting (0x19).


Your email besides being offending, is empty.
If someone's actions hurt the community are your own.

Not going to enter in personal discussions in here nor going to waist
time answering to anymore of your offending emails.
If you want to exchange some thoughts, mail me personally.

Regards,
Luis







On Tue, Jan 7, 2014 at 5:31 PM, Konstantin Dimitrov
<kosio.dimitrov@gmail.com> wrote:
> Luis,
>
> can you explain to us all here how exactly you came up to those
> particular I2C fixes:
>
> https://github.com/ljalves/linux_media/commit/be7cd1ff82cc20578b805ad508d089f818ae726d
>
> because essentially they are the same as what i did years ago -
> included as source code in drivers i made for some TBS cards (source
> code is available all over online) or we just have the exact same case
> with you as before:
>
> http://www.spinics.net/lists/linux-media/msg65888.html
>
> http://www.spinics.net/lists/linux-media/msg65889.html
>
> and you're continue to taking credit for patches i made or basically
> stealing them.
>
> if they were some trivial patches i won't mind, but even they are
> small, they are nothing like trivial.
>
> so, i believe you will have really hard time to explain "your" I2C
> fixes, because for example the SDA hold time value of 0x14 is needed
> for only one particular SAA716x-based card (other such cards can work
> with wide range of SDA hold time settings) and i'm sure you don't know
> that card and cannot cite its model or any technical details why
> that's needed, because you don't have it, as well it takes quite an
> effort and good knowledge of I2C signaling with oscilloscope to figure
> out that value, as well that exactly that value needs changing.
>
> why you didn't use for example 0x16 or 0x13 for SDA hold time in
> "your" I2C patch?!
>
> so, one time, like the previous time, excuse that you just didn't know
> who the author of that work is may fly, but second time, especially
> considering that the SAA716x code base from which i'm sure you took
> (not to use stole) those settings contains my name as copyright,
> because i actually added to that code base new code i developed from
> scratch like for example saa716x_input.[c|h], is another thing you
> cannot explain.
>
> so, i was waiting Manu to upstream his SAA716x driver code some day
> and then submit the improvements i made to it. yet again you're trying
> to take that from me and again, conveniently you included many people
> on CC, but not me.
>
> in my opinion what you're doing is not right, because that patch is
> not clean-room reverse-engineering, you just took those changes from
> another open-source base and if nothing else it's at least common
> courtesy in open-source community when you didn't make them to not
> submit them as "your" patches.
>
> i also think with your actions you're actually hurting the community,
> because people like me, that do actually have the technical
> understanding and can help and contribute further improvements are
> driven away from the community, because
> effectively the community accepting behavior like yours is encouraging
> code stealing!!
>
> --konstantin
>
> On Tue, Jan 7, 2014 at 6:33 PM, Luis Alves <ljalvs@gmail.com> wrote:
>> HI Andreas,
>>
>> My initial commit is based on:
>> http://powarman.dyndns.org/hgwebdir.cgi/v4l-dvb-saa716x/
>> (I think it's your repo with some commits from Soeren Moch)
>>
>> The difference to my working area is that I have the driver placed in
>> "drivers/media/pci/saa716x" (instead of
>> "drivers/media/common/saa716x") and everything is rebased on the
>> latest media_tree.
>> On top of that I just have 2 commits: one to be able to build FF cards
>> and another to fix some i2c issues.
>>
>> You can check my repo here:
>> https://github.com/ljalves/linux_media/commits/saa716x
>>
>> Regards,
>> Luis
>>
>>
>> On Tue, Jan 7, 2014 at 4:12 PM, Andreas Regel <andreas.regel@gmx.de> wrote:
>>> Hi Luis,
>>>
>>> Am 07.01.2014 12:58, schrieb Luis Alves:
>>>> Hi,
>>>>
>>>> I'm finishing a new frontend driver for one of my dvb cards, but the
>>>> pcie bridge uses the (cursed) saa716x.
>>>> As far as I know the progress to upstream Manu's driver to the
>>>> media_tree has stalled.
>>>>
>>>> In CC I've placed some of the people that I found working on it
>>>> lately, supporting a few dvb cards.
>>>>
>>>> It would be good if we could gather everything in one place and send a
>>>> few patchs to get this upstreamed for once...
>>>>
>>>> Manu, do you see any inconvenience in sending your driver to the
>>>> linux_media tree?
>>>> I'm available to place some effort on this task.
>>>
>>> which repository of the saa761x is your work based on?
>>>
>>> Regards,
>>> Andreas
>>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-media" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: Upstreaming SAA716x driver to the media_tree
  2014-01-07 17:31     ` Konstantin Dimitrov
  2014-01-07 20:23       ` Luis Alves
@ 2014-01-07 23:42       ` CrazyCat
  2014-01-08 20:52         ` Konstantin Dimitrov
  1 sibling, 1 reply; 17+ messages in thread
From: CrazyCat @ 2014-01-07 23:42 UTC (permalink / raw)
  To: Konstantin Dimitrov, Luis Alves
  Cc: Andreas Regel, linux-media, Chris Lee, Manu Abraham,
	Mauro Carvalho Chehab, Antti Palosaari

Konstantin Dimitrov пишет:
> so, i was waiting Manu to upstream his SAA716x driver code some day
> and then submit the improvements i made to it. yet again you're trying
> to take that from me and again, conveniently you included many people
> on CC, but not me.

it's ok :) and stop compile binary blobs for TBS :) like 'stupid' binaries for LNB power control and init for 6925/5925 :)


> in my opinion what you're doing is not right, because that patch is
> not clean-room reverse-engineering, you just took those changes from
> another open-source base and if nothing else it's at least common
> courtesy in open-source community when you didn't make them to not
> submit them as "your" patches.

this is open-source world :)

> i also think with your actions you're actually hurting the community,
> because people like me, that do actually have the technical
> understanding and can help and contribute further improvements are
> driven away from the community, because
> effectively the community accepting behavior like yours is encouraging
> code stealing!!

what stealed code ??? :) if you want write closed-source drivers for windose - make it ! :)


^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: Upstreaming SAA716x driver to the media_tree
  2014-01-07 23:42       ` CrazyCat
@ 2014-01-08 20:52         ` Konstantin Dimitrov
  0 siblings, 0 replies; 17+ messages in thread
From: Konstantin Dimitrov @ 2014-01-08 20:52 UTC (permalink / raw)
  To: CrazyCat
  Cc: Luis Alves, Andreas Regel, linux-media, Chris Lee, Manu Abraham,
	Mauro Carvalho Chehab, Antti Palosaari

On Wed, Jan 8, 2014 at 1:42 AM, CrazyCat <crazycat69@narod.ru> wrote:
> Konstantin Dimitrov пишет:
>
>> so, i was waiting Manu to upstream his SAA716x driver code some day
>> and then submit the improvements i made to it. yet again you're trying
>> to take that from me and again, conveniently you included many people
>> on CC, but not me.
>
>
> it's ok :) and stop compile binary blobs for TBS :) like 'stupid' binaries
> for LNB power control and init for 6925/5925 :)
>

do you really believe that's mine decision to direct that to me?! what
you're asking is not my decision to make.

>
>
>> in my opinion what you're doing is not right, because that patch is
>> not clean-room reverse-engineering, you just took those changes from
>> another open-source base and if nothing else it's at least common
>> courtesy in open-source community when you didn't make them to not
>> submit them as "your" patches.
>
>
> this is open-source world :)

no, it's not open-source world to take code that someone else made and
say it's yours

>
>
>> i also think with your actions you're actually hurting the community,
>> because people like me, that do actually have the technical
>> understanding and can help and contribute further improvements are
>> driven away from the community, because
>> effectively the community accepting behavior like yours is encouraging
>> code stealing!!
>
>
> what stealed code ??? :) if you want write closed-source drivers for windose
> - make it ! :)
>

i'm not discussing any closed source work here - i'm talking about
patches made to code that is licensed under GPL.

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: Upstreaming SAA716x driver to the media_tree
  2014-01-07 20:23       ` Luis Alves
@ 2014-01-08 21:21         ` Konstantin Dimitrov
  0 siblings, 0 replies; 17+ messages in thread
From: Konstantin Dimitrov @ 2014-01-08 21:21 UTC (permalink / raw)
  To: Luis Alves
  Cc: Andreas Regel, linux-media, Chris Lee, Manu Abraham, CrazyCat,
	Mauro Carvalho Chehab, Antti Palosaari

Luis, yes, here we're or more exactly you're again, because it's clear
now you have the exact same patterns - i told you already - one time
any excuse may fly, when it's a pattern it's a whole new story. so,
excuses like this one:

http://www.spinics.net/lists/linux-media/msg65889.html

and the points you're making now are more than hollow.

any reasonable man can see "your" I2C patch - there is many ways to do
it, but somehow magically and by coincidence you again do them exactly
like i did it - what i mean - when it's for specific board for example
you may choose to add another case in the switch statement instead do
it for all boards - of course, that's the case if you do it by
yourself and not copy.

so, it's public discussion, i have nothing to say to you in private,
in fact everything i want to say to you is in public. i see no
anything offensive in that someone that have full rights to do that
step in and point out when there is something that is not right. if
you take it as offensive that's your problem - in fact what you're
doing is offensive, not that i care so much to take it that way. after
all the discussion is public - everyone can form their own opinion.

as far as if my email is empty - i can say the exact same for your
work on SAA716x - that's work done mainly by Manu, Andreas and me -
actually after Manu left it - i and Andreas obviously span the Manu
code and went different ways from there. what i can claim full credit
is I2C patches (need for certain TBS boards only) and
"saa716x_input.[c|h]" - all of them GPL work - once again everyone can
check the copyright notice of "saa716x_input.[c|h]" and that same code
base contains the I2C patches and it's years old.

so, when you finally make some code that is really yours and actually
making code is easy comparing to understand how the silicone works and
fix problems, you're free to take credit for it. until now i see only
either you're taking open-source code that's not yours or do
reverse-engineering of blobs that otherwise it's not easy to make and
develop from scratch. at least the last action is totally legit,
contrary to the first one which is simply unethical, especially in
open-source.

--konstantin

On Tue, Jan 7, 2014 at 10:23 PM, Luis Alves <ljalvs@gmail.com> wrote:
> Hi Konstantin,
> (here we go again...)
>
> What the hell are you talking about? I didn't submit any I2C patch.
> This email is just about Manu sending his SAA716x driver to the media_tree.
>
> And where do you see any "saa716x_input.[c|h]" in my repo?
>
> Anyway, since you asked I took some pics:
> https://plus.google.com/photos/105602732859464871628/albums/5966247612074668305?authkey=CN3jl9mWhrDhQg
> (SDA_HOLD setting in the comment)
>
> For SDA_HOLD = 0x19 the next clock rising edge is really close to the
> data line release.
> Even 0x14 is too close, so I will use a smaller value so maybe 0x10 is
> fine for the 400kHZ clk speed.
>
> And I do have a TBS card that doesn't work with Manu default setting (0x19).
>
>
> Your email besides being offending, is empty.
> If someone's actions hurt the community are your own.
>
> Not going to enter in personal discussions in here nor going to waist
> time answering to anymore of your offending emails.
> If you want to exchange some thoughts, mail me personally.
>
> Regards,
> Luis
>
>
>
>
>
>
>
> On Tue, Jan 7, 2014 at 5:31 PM, Konstantin Dimitrov
> <kosio.dimitrov@gmail.com> wrote:
>> Luis,
>>
>> can you explain to us all here how exactly you came up to those
>> particular I2C fixes:
>>
>> https://github.com/ljalves/linux_media/commit/be7cd1ff82cc20578b805ad508d089f818ae726d
>>
>> because essentially they are the same as what i did years ago -
>> included as source code in drivers i made for some TBS cards (source
>> code is available all over online) or we just have the exact same case
>> with you as before:
>>
>> http://www.spinics.net/lists/linux-media/msg65888.html
>>
>> http://www.spinics.net/lists/linux-media/msg65889.html
>>
>> and you're continue to taking credit for patches i made or basically
>> stealing them.
>>
>> if they were some trivial patches i won't mind, but even they are
>> small, they are nothing like trivial.
>>
>> so, i believe you will have really hard time to explain "your" I2C
>> fixes, because for example the SDA hold time value of 0x14 is needed
>> for only one particular SAA716x-based card (other such cards can work
>> with wide range of SDA hold time settings) and i'm sure you don't know
>> that card and cannot cite its model or any technical details why
>> that's needed, because you don't have it, as well it takes quite an
>> effort and good knowledge of I2C signaling with oscilloscope to figure
>> out that value, as well that exactly that value needs changing.
>>
>> why you didn't use for example 0x16 or 0x13 for SDA hold time in
>> "your" I2C patch?!
>>
>> so, one time, like the previous time, excuse that you just didn't know
>> who the author of that work is may fly, but second time, especially
>> considering that the SAA716x code base from which i'm sure you took
>> (not to use stole) those settings contains my name as copyright,
>> because i actually added to that code base new code i developed from
>> scratch like for example saa716x_input.[c|h], is another thing you
>> cannot explain.
>>
>> so, i was waiting Manu to upstream his SAA716x driver code some day
>> and then submit the improvements i made to it. yet again you're trying
>> to take that from me and again, conveniently you included many people
>> on CC, but not me.
>>
>> in my opinion what you're doing is not right, because that patch is
>> not clean-room reverse-engineering, you just took those changes from
>> another open-source base and if nothing else it's at least common
>> courtesy in open-source community when you didn't make them to not
>> submit them as "your" patches.
>>
>> i also think with your actions you're actually hurting the community,
>> because people like me, that do actually have the technical
>> understanding and can help and contribute further improvements are
>> driven away from the community, because
>> effectively the community accepting behavior like yours is encouraging
>> code stealing!!
>>
>> --konstantin
>>
>> On Tue, Jan 7, 2014 at 6:33 PM, Luis Alves <ljalvs@gmail.com> wrote:
>>> HI Andreas,
>>>
>>> My initial commit is based on:
>>> http://powarman.dyndns.org/hgwebdir.cgi/v4l-dvb-saa716x/
>>> (I think it's your repo with some commits from Soeren Moch)
>>>
>>> The difference to my working area is that I have the driver placed in
>>> "drivers/media/pci/saa716x" (instead of
>>> "drivers/media/common/saa716x") and everything is rebased on the
>>> latest media_tree.
>>> On top of that I just have 2 commits: one to be able to build FF cards
>>> and another to fix some i2c issues.
>>>
>>> You can check my repo here:
>>> https://github.com/ljalves/linux_media/commits/saa716x
>>>
>>> Regards,
>>> Luis
>>>
>>>
>>> On Tue, Jan 7, 2014 at 4:12 PM, Andreas Regel <andreas.regel@gmx.de> wrote:
>>>> Hi Luis,
>>>>
>>>> Am 07.01.2014 12:58, schrieb Luis Alves:
>>>>> Hi,
>>>>>
>>>>> I'm finishing a new frontend driver for one of my dvb cards, but the
>>>>> pcie bridge uses the (cursed) saa716x.
>>>>> As far as I know the progress to upstream Manu's driver to the
>>>>> media_tree has stalled.
>>>>>
>>>>> In CC I've placed some of the people that I found working on it
>>>>> lately, supporting a few dvb cards.
>>>>>
>>>>> It would be good if we could gather everything in one place and send a
>>>>> few patchs to get this upstreamed for once...
>>>>>
>>>>> Manu, do you see any inconvenience in sending your driver to the
>>>>> linux_media tree?
>>>>> I'm available to place some effort on this task.
>>>>
>>>> which repository of the saa761x is your work based on?
>>>>
>>>> Regards,
>>>> Andreas
>>>>
>>> --
>>> To unsubscribe from this list: send the line "unsubscribe linux-media" in
>>> the body of a message to majordomo@vger.kernel.org
>>> More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: Upstreaming SAA716x driver to the media_tree
       [not found]   ` <CALzAhNW=uCLsu4sdrGyycxRoOdaRTFNM8yVj=Y+ahSZrWqrABg@mail.gmail.com>
@ 2014-01-14  3:35     ` Manu Abraham
  2014-01-21 21:59       ` Steven Toth
  0 siblings, 1 reply; 17+ messages in thread
From: Manu Abraham @ 2014-01-14  3:35 UTC (permalink / raw)
  To: Steven Toth
  Cc: Luis Alves, linux-media, Andreas Regel, Chris Lee, crazycat69,
	Mauro Carvalho Chehab, Antti Palosaari

On Tue, Jan 14, 2014 at 12:20 AM, Steven Toth <stoth@kernellabs.com> wrote:
>>> Manu, do you see any inconvenience in sending your driver to the
>>> linux_media tree?
>>> I'm available to place some effort on this task.
>>
>>
>> I can push the 716x driver and whatever additional changes that I have
>> later on this weekend, if that's okay with you.
>
> I never saw a push.

Spliiting and cleaning up the patches took up more time than expected.
Please wait a few days.

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: Upstreaming SAA716x driver to the media_tree
  2014-01-14  3:35     ` Manu Abraham
@ 2014-01-21 21:59       ` Steven Toth
  2014-01-22 15:26         ` Manu Abraham
  0 siblings, 1 reply; 17+ messages in thread
From: Steven Toth @ 2014-01-21 21:59 UTC (permalink / raw)
  To: Manu Abraham
  Cc: Luis Alves, linux-media, Andreas Regel, Chris Lee, crazycat69,
	Mauro Carvalho Chehab, Antti Palosaari

On Mon, Jan 13, 2014 at 10:35 PM, Manu Abraham <abraham.manu@gmail.com> wrote:
> On Tue, Jan 14, 2014 at 12:20 AM, Steven Toth <stoth@kernellabs.com> wrote:
>>>> Manu, do you see any inconvenience in sending your driver to the
>>>> linux_media tree?
>>>> I'm available to place some effort on this task.
>>>
>>>
>>> I can push the 716x driver and whatever additional changes that I have
>>> later on this weekend, if that's okay with you.
>>
>> I never saw a push.
>
> Spliiting and cleaning up the patches took up more time than expected.
> Please wait a few days.

Any news on this?

-- 
Steven Toth - Kernel Labs
http://www.kernellabs.com

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: Upstreaming SAA716x driver to the media_tree
  2014-01-21 21:59       ` Steven Toth
@ 2014-01-22 15:26         ` Manu Abraham
  2014-02-11 13:44           ` Luis Alves
  0 siblings, 1 reply; 17+ messages in thread
From: Manu Abraham @ 2014-01-22 15:26 UTC (permalink / raw)
  To: Steven Toth
  Cc: Luis Alves, linux-media, Andreas Regel, Chris Lee, crazycat69,
	Mauro Carvalho Chehab, Antti Palosaari

On Wed, Jan 22, 2014 at 3:29 AM, Steven Toth <stoth@kernellabs.com> wrote:
> On Mon, Jan 13, 2014 at 10:35 PM, Manu Abraham <abraham.manu@gmail.com> wrote:
>> On Tue, Jan 14, 2014 at 12:20 AM, Steven Toth <stoth@kernellabs.com> wrote:
>>>>> Manu, do you see any inconvenience in sending your driver to the
>>>>> linux_media tree?
>>>>> I'm available to place some effort on this task.
>>>>
>>>>
>>>> I can push the 716x driver and whatever additional changes that I have
>>>> later on this weekend, if that's okay with you.
>>>
>>> I never saw a push.
>>
>> Spliiting and cleaning up the patches took up more time than expected.
>> Please wait a few days.
>
> Any news on this?


I just pushed out a large chunk of the changes. There are a few
dependencies that need to be resolved with the rebased tree.

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: Upstreaming SAA716x driver to the media_tree
  2014-01-22 15:26         ` Manu Abraham
@ 2014-02-11 13:44           ` Luis Alves
  2014-02-11 19:47             ` Manu Abraham
  0 siblings, 1 reply; 17+ messages in thread
From: Luis Alves @ 2014-02-11 13:44 UTC (permalink / raw)
  To: Manu Abraham
  Cc: Steven Toth, linux-media, Andreas Regel, Chris Lee, crazycat69,
	Mauro Carvalho Chehab, Antti Palosaari

Hi,

Any update on this?

Thanks,
Luis

On Wed, Jan 22, 2014 at 3:26 PM, Manu Abraham <abraham.manu@gmail.com> wrote:
> On Wed, Jan 22, 2014 at 3:29 AM, Steven Toth <stoth@kernellabs.com> wrote:
>> On Mon, Jan 13, 2014 at 10:35 PM, Manu Abraham <abraham.manu@gmail.com> wrote:
>>> On Tue, Jan 14, 2014 at 12:20 AM, Steven Toth <stoth@kernellabs.com> wrote:
>>>>>> Manu, do you see any inconvenience in sending your driver to the
>>>>>> linux_media tree?
>>>>>> I'm available to place some effort on this task.
>>>>>
>>>>>
>>>>> I can push the 716x driver and whatever additional changes that I have
>>>>> later on this weekend, if that's okay with you.
>>>>
>>>> I never saw a push.
>>>
>>> Spliiting and cleaning up the patches took up more time than expected.
>>> Please wait a few days.
>>
>> Any news on this?
>
>
> I just pushed out a large chunk of the changes. There are a few
> dependencies that need to be resolved with the rebased tree.

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: Upstreaming SAA716x driver to the media_tree
  2014-02-11 13:44           ` Luis Alves
@ 2014-02-11 19:47             ` Manu Abraham
  2014-02-25 22:27               ` Luis Alves
  0 siblings, 1 reply; 17+ messages in thread
From: Manu Abraham @ 2014-02-11 19:47 UTC (permalink / raw)
  To: Luis Alves
  Cc: Steven Toth, linux-media, Andreas Regel, Chris Lee, crazycat69,
	Mauro Carvalho Chehab, Antti Palosaari

On Tue, Feb 11, 2014 at 7:14 PM, Luis Alves <ljalvs@gmail.com> wrote:
> Hi,
>
> Any update on this?

I need to address the issues Mauro pointed out, prior to the merge.
Will address the issues during the next week. Have been a bit busy
restoring the system at my end after a crash.

Regards,

Manu

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: Upstreaming SAA716x driver to the media_tree
  2014-02-11 19:47             ` Manu Abraham
@ 2014-02-25 22:27               ` Luis Alves
  2014-02-26 17:27                 ` Manu Abraham
  0 siblings, 1 reply; 17+ messages in thread
From: Luis Alves @ 2014-02-25 22:27 UTC (permalink / raw)
  To: Manu Abraham
  Cc: Steven Toth, linux-media, Andreas Regel, Chris Lee, crazycat69,
	Mauro Carvalho Chehab, Antti Palosaari

Hi Manu,

How's the progress going?
Looking forward to finally see this driver in the tree :D

Regards,
Luis

On Tue, Feb 11, 2014 at 7:47 PM, Manu Abraham <abraham.manu@gmail.com> wrote:
> On Tue, Feb 11, 2014 at 7:14 PM, Luis Alves <ljalvs@gmail.com> wrote:
>> Hi,
>>
>> Any update on this?
>
> I need to address the issues Mauro pointed out, prior to the merge.
> Will address the issues during the next week. Have been a bit busy
> restoring the system at my end after a crash.
>
> Regards,
>
> Manu

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: Upstreaming SAA716x driver to the media_tree
  2014-02-25 22:27               ` Luis Alves
@ 2014-02-26 17:27                 ` Manu Abraham
  0 siblings, 0 replies; 17+ messages in thread
From: Manu Abraham @ 2014-02-26 17:27 UTC (permalink / raw)
  To: Luis Alves
  Cc: Steven Toth, linux-media, Andreas Regel, Chris Lee, crazycat69,
	Mauro Carvalho Chehab, Antti Palosaari

Hi Luis,

On Wed, Feb 26, 2014 at 3:57 AM, Luis Alves <ljalvs@gmail.com> wrote:
> Hi Manu,
>
> How's the progress going?
> Looking forward to finally see this driver in the tree :D

Just got back setting things back to normalcy.
This weekend, I have plans to do more work for that.

Regards,

Manu

^ permalink raw reply	[flat|nested] 17+ messages in thread

end of thread, other threads:[~2014-02-26 17:27 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-01-07 11:58 Upstreaming SAA716x driver to the media_tree Luis Alves
2014-01-07 13:10 ` Manu Abraham
2014-01-07 15:59   ` Luis Alves
     [not found]   ` <CALzAhNW=uCLsu4sdrGyycxRoOdaRTFNM8yVj=Y+ahSZrWqrABg@mail.gmail.com>
2014-01-14  3:35     ` Manu Abraham
2014-01-21 21:59       ` Steven Toth
2014-01-22 15:26         ` Manu Abraham
2014-02-11 13:44           ` Luis Alves
2014-02-11 19:47             ` Manu Abraham
2014-02-25 22:27               ` Luis Alves
2014-02-26 17:27                 ` Manu Abraham
2014-01-07 16:12 ` Andreas Regel
2014-01-07 16:33   ` Luis Alves
2014-01-07 17:31     ` Konstantin Dimitrov
2014-01-07 20:23       ` Luis Alves
2014-01-08 21:21         ` Konstantin Dimitrov
2014-01-07 23:42       ` CrazyCat
2014-01-08 20:52         ` Konstantin Dimitrov

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.