linux-sh.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>
To: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
	linux-media@vger.kernel.org,
	Sylwester Nawrocki <s.nawrocki@samsung.com>,
	Hans Verkuil <hverkuil@xs4all.nl>,
	linux-sh@vger.kernel.org, Magnus Damm <magnus.damm@gmail.com>,
	Sakari Ailus <sakari.ailus@iki.fi>,
	Prabhakar Lad <prabhakar.lad@ti.com>,
	Sascha Hauer <s.hauer@pengutronix.de>
Subject: Re: [PATCH v11 00/21] V4L2 clock and asynchronous probing
Date: Fri, 14 Jun 2013 22:43:20 +0000	[thread overview]
Message-ID: <51BB9C88.1040606@gmail.com> (raw)
In-Reply-To: <Pine.LNX.4.64.1306142244310.11221@axis700.grange>

Hi Guennadi,

On 06/14/2013 10:45 PM, Guennadi Liakhovetski wrote:
> On Fri, 14 Jun 2013, Laurent Pinchart wrote:
>> >  On Friday 14 June 2013 21:08:10 Guennadi Liakhovetski wrote:
>>> >  >  v11 of the V4L2 clock helper and asynchronous probing patch set.
>>> >  >  Functionally identical to v10, only differences are a couple of comment
>>> >  >  lines and one renamed struct field - as requested by respectable
>>> >  >  reviewers:)
>>> >  >
>>> >  >  Only patches #15, 16 and 18 changed.
>> >
>> >  [snip]
>> >
>>> >  >    .../devicetree/bindings/media/sh_mobile_ceu.txt    |   18 +
>> >
>> >      Documentation/video4linux/v4l2-framework.txt is missing:-)
>
> I know. I will add it as soon as these patches are in.

Actually it would be nice to see some documentation, before this patch set
is merged. I don't think the whole concept is going to be changed or 
rejected
by Mauro at this point :) And we always have been pushing back on 
merging any
code without some corresponding documentation. The v4l2 API seems 
relatively
well documented, and I would prefer to keep it that way. :-)

Regards,
Sylwester


  reply	other threads:[~2013-06-14 22:43 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-14 19:08 [PATCH v11 00/21] V4L2 clock and asynchronous probing Guennadi Liakhovetski
2013-06-14 19:08 ` [PATCH v11 01/21] soc-camera: move common code to soc_camera.c Guennadi Liakhovetski
2013-06-14 19:08 ` [PATCH v11 02/21] soc-camera: add host clock callbacks to start and stop the master clock Guennadi Liakhovetski
2013-06-14 19:08 ` [PATCH v11 03/21] pxa-camera: move interface activation and deactivation to clock callbacks Guennadi Liakhovetski
2013-06-14 19:08 ` [PATCH v11 04/21] omap1-camera: " Guennadi Liakhovetski
2013-06-14 19:08 ` [PATCH v11 05/21] atmel-isi: " Guennadi Liakhovetski
2013-06-14 19:08 ` [PATCH v11 06/21] mx3-camera: " Guennadi Liakhovetski
2013-06-14 19:08 ` [PATCH v11 07/21] mx2-camera: " Guennadi Liakhovetski
2013-06-14 19:08 ` [PATCH v11 08/21] mx1-camera: " Guennadi Liakhovetski
2013-06-14 19:08 ` [PATCH v11 09/21] sh-mobile-ceu-camera: move interface activation and deactivation to clock callback Guennadi Liakhovetski
2013-06-14 19:08 ` [PATCH v11 10/21] soc-camera: make .clock_{start,stop} compulsory, .add / .remove optional Guennadi Liakhovetski
2013-06-14 19:08 ` [PATCH v11 11/21] soc-camera: don't attach the client to the host during probing Guennadi Liakhovetski
2013-06-14 19:08 ` [PATCH v11 12/21] sh-mobile-ceu-camera: add primitive OF support Guennadi Liakhovetski
2013-06-14 19:08 ` [PATCH v11 13/21] sh-mobile-ceu-driver: support max width and height in DT Guennadi Liakhovetski
2013-06-14 19:08 ` [PATCH v11 14/21] V4L2: add temporary clock helpers Guennadi Liakhovetski
2013-06-17 15:39   ` Prabhakar Lad
2013-06-14 19:08 ` [PATCH v11 15/21] V4L2: add a device pointer to struct v4l2_subdev Guennadi Liakhovetski
2013-06-16 15:30   ` Prabhakar Lad
2013-06-14 19:08 ` [PATCH v11 16/21] V4L2: support asynchronous subdevice registration Guennadi Liakhovetski
2013-06-16 15:29   ` Prabhakar Lad
2013-06-21 16:22   ` Laurent Pinchart
2013-06-14 19:08 ` [PATCH v11 17/21] soc-camera: switch I2C subdevice drivers to use v4l2-clk Guennadi Liakhovetski
2013-06-14 19:08 ` [PATCH v11 18/21] soc-camera: add V4L2-async support Guennadi Liakhovetski
2013-06-14 19:08 ` [PATCH v11 19/21] sh_mobile_ceu_camera: add asynchronous subdevice probing support Guennadi Liakhovetski
2013-06-14 19:08 ` [PATCH v11 20/21] imx074: support asynchronous probing Guennadi Liakhovetski
2013-06-14 19:08 ` [PATCH v11 21/21] ARM: shmobile: convert ap4evb to asynchronously register camera subdevices Guennadi Liakhovetski
2013-06-14 20:27 ` [PATCH v11 00/21] V4L2 clock and asynchronous probing Laurent Pinchart
2013-06-14 20:45   ` Guennadi Liakhovetski
2013-06-14 22:43     ` Sylwester Nawrocki [this message]
2013-06-15  8:51     ` Hans Verkuil
2013-06-17 11:13 ` 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=51BB9C88.1040606@gmail.com \
    --to=sylvester.nawrocki@gmail.com \
    --cc=g.liakhovetski@gmx.de \
    --cc=hverkuil@xs4all.nl \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-sh@vger.kernel.org \
    --cc=magnus.damm@gmail.com \
    --cc=prabhakar.lad@ti.com \
    --cc=s.hauer@pengutronix.de \
    --cc=s.nawrocki@samsung.com \
    --cc=sakari.ailus@iki.fi \
    /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;
as well as URLs for NNTP newsgroup(s).