From: khalasa@piap.pl (Krzysztof Hałasa)
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>,
Ezequiel Garcia <ezequiel@vanguardiasur.com.ar>
Subject: Re: tw686x driver
Date: Thu, 03 Mar 2016 07:51:07 +0100 [thread overview]
Message-ID: <m3povcnjfo.fsf@t19.piap.pl> (raw)
In-Reply-To: <56D6A50F.4060404@xs4all.nl> (Hans Verkuil's message of "Wed, 2 Mar 2016 09:32:15 +0100")
Hi Hans,
Hans Verkuil <hverkuil@xs4all.nl> writes:
> So lessons learned:
>
> Krzysztof, next time don't wait many months before posting a new version fixing
> requested changes.
Actually, this is not how it happened.
On July 3, 2015 I posted the original driver:
http://www.spinics.net/lists/linux-media/msg91474.html
Ezequiel reviewed the code on 6 Jul 2015:
http://www.spinics.net/lists/linux-media/msg91516.html
On 27 Jul 2015, as you can easily find in your own mail archive, he
informed me that he's working on the driver and that he's going to post
updated patches himself. This was holidays time for me and I stated
that I have to suspend my work till the end of August.
I asked him to add his changes on top of my code several times (and this
is really easy with git). This never happened, and on 14 Aug 2015 he
posted:
> Problem is I've re-written the driver, taking yours as a starting point
> and reference.
> In other words, I don't have a proper git branch with a history, starting
> from the patch you submitted. Instead, I would be submitting a new
> patch for Hans and Mauro to review.
Maybe I got the meaning of this wrong, and he wasn't writing about
rewriting the driver "from scratch". Yes, after receiving this mail
I stopped my development, waiting for his driver to show up. That's
where the "many months" are. Yes, Ezequiel waited for "many months" with
his version - not me.
Only after he has posted "his" driver, I could find out what the
"rewriting" meant.
Don't get me wrong, I was never opposed to him improving my driver.
I only requested that his contributions are clearly shown, in a form
of a patch or a patch set (or a git tree etc.), and so are my own.
I really can't understand why his code can't be transparently applied
over my original patch (or the updated one, which compiles and works
fine with recent media tree).
Is it too much to ask?
The lesson I learned is thus this instead:
- don't publish code because it can be hijacked, twisted and you'll
have to fight for even getting your authorship back. Forget about proper
attribution and history.
--
Krzysztof Halasa
Industrial Research Institute for Automation and Measurements PIAP
Al. Jerozolimskie 202, 02-486 Warsaw, Poland
next prev parent reply other threads:[~2016-03-03 6:51 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-02 8:32 tw686x driver Hans Verkuil
2016-03-03 6:51 ` Krzysztof Hałasa [this message]
2016-03-03 7:32 ` Hans Verkuil
2016-03-03 12:41 ` Krzysztof Hałasa
2016-03-03 13:37 ` Hans Verkuil
2016-03-03 14:22 ` Krzysztof Hałasa
2016-03-03 14:39 ` Hans Verkuil
2016-03-04 6:11 ` Krzysztof Hałasa
2016-03-04 11:11 ` Hans Verkuil
2016-03-04 12:37 ` Krzysztof Hałasa
2016-03-04 13:40 ` Hans Verkuil
2016-03-07 6:41 ` Krzysztof Hałasa
2016-03-10 7:16 ` Krzysztof Hałasa
2016-03-10 7:24 ` Hans Verkuil
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=m3povcnjfo.fsf@t19.piap.pl \
--to=khalasa@piap.pl \
--cc=ezequiel@vanguardiasur.com.ar \
--cc=hverkuil@xs4all.nl \
--cc=linux-media@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox