From: Yupeng Yan <yyan@codeaurora.org>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Markus Rechberger <mrechberger@gmail.com>,
Mauro Carvalho Chehab <mchehab@redhat.com>,
Hans Verkuil <hverkuil@xs4all.nl>,
Haibo Zhong <hzhong@codeaurora.org>,
Shuzhen Wang <shuzhenw@codeaurora.org>,
"linux-media@vger.kernel.org" <linux-media@vger.kernel.org>
Subject: Re: RFC: V4L2 driver for Qualcomm MSM camera.
Date: Thu, 06 Jan 2011 16:03:58 -0800 [thread overview]
Message-ID: <4D26586E.3080500@codeaurora.org> (raw)
In-Reply-To: <201101050115.57699.laurent.pinchart@ideasonboard.com>
Thanks for the comments - certainly good arguments to our lawyers :-)...
actually the information of how to config the ISP HW is requested to be
protected for now, we are working on certain degree of openness.
The HW config code is just a small part of the user space (daemon)
tasks, the user space code also processes ISP states, carries out 3A
algorithm and perform post processing features, etc. which will have to
be protected, this is the reason why the daemon is used in QC solution.
On 1/4/2011 4:15 PM, Laurent Pinchart wrote:
> Hi,
>
> On Tuesday 04 January 2011 20:37:31 Yan, Yupeng wrote:
>> We will exploring the usage of libv4l2...however we still have the
>> difficulties to open-source hardware: our ISP and sensors, will need help
>> on how to address such issues.
> I suppose you mean open-sourcing hardware driver. There's no requirement to
> open-source the hardware at this point, although I would love to see that
> happening :-)
>
> Let's start with sensors, it's the easiest. Sensor drivers must not depend on
> the ISP driver, and the ISP driver must not depend on sensor drivers. They
> communicate together through the media controller and the V4L2 subdev APIs.
>
> You will obviously need to test your ISP driver with at least one sensor, but
> you're not required to submit the sensor driver along with the ISP driver
> (although it would be nice if you did so).
>
> Correct me if I'm wrong, but I don't think Qualcomm produces video sensors, so
> no Qualcomm confidential information would be disclosed in a sensor driver. If
> the sensor manufacturer hasn't released a sensor driver, or sensor
> specifications covered by a license that allows a GPL driver to be written,
> you will need to sort it out with the manufacturer. Note that public
> documentation is not strictly required to release an open-source driver (see
> the SMIA++ driver written by Nokia for instance), but that would be
> appreciated.
>
> For the ISP the problem is not that complex either. From what I understand
> Qualcomm is scared that publishing an open-source driver will backfire. I can
> see several reasons (I should probably say myths instead of reasons) for that
> fear:
>
> - "Competitors will steal my code". They can't. The code is highly hardware
> specific and doesn't contain much added value. It can't be reused as such.
> Even if code could be reused, it couldn't be "stolen" as it will be covered by
> the GPLv2.
>
> - "Competitors will get key information about my hardware". You need to ask
> yourself what you consider as key information. The fact that the output width
> is stored in register 0x12345678 is hardly key information (if it is, there's
> a more basic issue). The way statistics are computed might be. Based on my
> experience with ISPs key information isn't disclosed by the driver. The
> hardware algorithms (such as color conversion, faulty pixels correction,
> statistics gathering, ...) implementation are usually not disclosed by the
> code.
>
> - "Competitors will sue me for patent infringement". This goes along with the
> previous point. Patented ISP features are usually the ones you consider as key
> information, and details are thus not disclosed by the driver. Note that I
> don't care here about trivial patents with broad claims such as "a system to
> capture an image to memory". Even without any open-source driver infringement
> can still easily be proven (and the patent can easily be invalidated, although
> that costs money - Qualcomm should join the Open Invention Network :-)).
>
> None of those reasons are valid compared to the benefits you get from an open-
> source Linux driver.
>
next prev parent reply other threads:[~2011-01-07 0:03 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-23 19:38 RFC: V4L2 driver for Qualcomm MSM camera Shuzhen Wang
2010-12-24 11:19 ` Hans Verkuil
2010-12-27 6:45 ` Shuzhen Wang
2010-12-27 12:11 ` Mauro Carvalho Chehab
2010-12-28 18:35 ` Shuzhen Wang
2010-12-28 20:23 ` Laurent Pinchart
2011-01-04 2:37 ` Shuzhen Wang
2011-01-04 6:14 ` Haibo Zhong
2011-01-04 8:46 ` Laurent Pinchart
2011-01-04 10:40 ` Hans Verkuil
2011-01-04 14:29 ` Mauro Carvalho Chehab
2011-01-04 18:08 ` Markus Rechberger
2011-01-04 19:37 ` Yan, Yupeng
2011-01-05 0:15 ` Laurent Pinchart
2011-01-07 0:03 ` Yupeng Yan [this message]
2011-01-07 14:37 ` Laurent Pinchart
2011-01-04 8:35 ` Laurent Pinchart
2011-01-04 11:30 ` Sakari Ailus
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=4D26586E.3080500@codeaurora.org \
--to=yyan@codeaurora.org \
--cc=hverkuil@xs4all.nl \
--cc=hzhong@codeaurora.org \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@redhat.com \
--cc=mrechberger@gmail.com \
--cc=shuzhenw@codeaurora.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