From: Hans Verkuil <hverkuil@xs4all.nl>
To: Mauro Carvalho Chehab <m.chehab@samsung.com>
Cc: linux-media@vger.kernel.org, Hans Verkuil <hans.verkuil@cisco.com>
Subject: Re: [RFC PATCH 3/6] DocBook media: partial rewrite of "Opening and Closing Devices"
Date: Tue, 04 Feb 2014 13:28:25 +0100 [thread overview]
Message-ID: <52F0DCE9.5020109@xs4all.nl> (raw)
In-Reply-To: <20140204102053.7faaa317@samsung.com>
On 02/04/14 13:20, Mauro Carvalho Chehab wrote:
> Em Fri, 17 Jan 2014 10:24:11 +0100
> Hans Verkuil <hverkuil@xs4all.nl> escreveu:
>
>>
>> I'll post the revised version of this patch next.
>
> Weird, I'm not seeing the revised version of this patch posted.
https://patchwork.linuxtv.org/patch/21620/
>
> Anyway, from your original patch:
>
>> + <para>Today each device node supports just one function, with the
>> +exception of overlay support.</para>
>
> It is still mixing overlay with "function" (where "function" means
> VBI, radio, video).
Yes, I want to tackle that separately.
>
> I'll drop this one from the series I'm applying. Please review and submit
> latter a new version of this one to the ML for easier review.
Please just take this as is. The 'function' terminology was there in the
original, so it is not something this patch has introduced. Yes, it should be
changed, but it's something I need to think about some more. This patch may
not solve everything, but at least it greatly improves it.
Regards,
Hans
next prev parent reply other threads:[~2014-02-04 12:31 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-07 13:06 [RFC PATCH 0/6] DocBook media: fixes Hans Verkuil
2014-01-07 13:06 ` [RFC PATCH 1/6] DocBook media: fix email addresses Hans Verkuil
2014-01-07 13:06 ` [RFC PATCH 2/6] DocBook media: update copyright years and Introduction Hans Verkuil
2014-01-07 13:06 ` [RFC PATCH 3/6] DocBook media: partial rewrite of "Opening and Closing Devices" Hans Verkuil
2014-01-13 15:20 ` Mauro Carvalho Chehab
2014-01-13 16:15 ` Hans Verkuil
2014-01-13 17:23 ` Mauro Carvalho Chehab
2014-01-17 9:24 ` Hans Verkuil
2014-02-04 12:20 ` Mauro Carvalho Chehab
2014-02-04 12:28 ` Hans Verkuil [this message]
2014-01-07 13:06 ` [RFC PATCH 4/6] DocBook media: update four more sections Hans Verkuil
2014-01-07 13:06 ` [RFC PATCH 5/6] DocBook media: update three sections Hans Verkuil
2014-01-07 13:06 ` [RFC PATCH 6/6] DocBook media: drop the old incorrect packed RGB table 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=52F0DCE9.5020109@xs4all.nl \
--to=hverkuil@xs4all.nl \
--cc=hans.verkuil@cisco.com \
--cc=linux-media@vger.kernel.org \
--cc=m.chehab@samsung.com \
/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