From: Hans Verkuil <hverkuil@xs4all.nl>
To: Ira Snyder <ira.snyder@gmail.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
iws@ovro.caltech.edu, linux-kernel@vger.kernel.org,
linux-media@vger.kernel.org, dwh@ovro.caltech.edu
Subject: Re: [PATCH 0/3] carma-fpga: remove videobuf dependency
Date: Fri, 12 Dec 2014 15:02:11 +0100 [thread overview]
Message-ID: <548AF563.3050204@xs4all.nl> (raw)
In-Reply-To: <CAH1ssNCnfdzCx5DhVW1=Kd8_J0wGaQ2yhbmUoZLG29g+12qV+g@mail.gmail.com>
Hi Dave,
On 11/24/2014 06:45 PM, Ira Snyder wrote:
>
> On Nov 23, 2014 6:31 AM, "Hans Verkuil" <hverkuil@xs4all.nl <mailto:hverkuil@xs4all.nl>> wrote:
>>
>> While checking which drivers were still abusing internal videobuf API
>> functions I came across these carma fpga misc drivers. These drivers
>> have absolutely nothing to do with videobuf or the media subsystem.
>>
>> Drivers shouldn't use those low-level functions in the first place,
>> and in fact in the long run the videobuf API will be removed altogether.
>>
>> So remove the videobuf dependency from these two drivers.
>>
>> This has been compile tested (and that clearly hasn't been done for
>> carma-fpga-program.c recently).
>>
>> Greg, is this something you want to take as misc driver maintainer?
>> That makes more sense than going through the media tree.
>>
>> The first patch should probably go to 3.18.
>>
>> I have no idea if anyone can test this with actual hardware. Ira, is
>> that something you can do?
>>
>
> Hi Hans. My colleague Dave Hawkins (CC'd) is in charge of this hardware now. He can help to get this patch tested.
Any updates on this?
Regards,
Hans
next prev parent reply other threads:[~2014-12-12 14:02 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-23 14:23 [PATCH 0/3] carma-fpga: remove videobuf dependency Hans Verkuil
2014-11-23 14:23 ` [PATCH 1/3] carma-fpga-program.c: fix compile errors Hans Verkuil
2014-11-23 14:23 ` [PATCH 2/3] carma-fpga: drop videobuf dependency Hans Verkuil
2014-11-23 14:23 ` [PATCH 3/3] carma-fpga-program: " Hans Verkuil
[not found] ` <CAH1ssNCnfdzCx5DhVW1=Kd8_J0wGaQ2yhbmUoZLG29g+12qV+g@mail.gmail.com>
2014-12-12 14:02 ` Hans Verkuil [this message]
2014-12-12 17:00 ` [PATCH 0/3] carma-fpga: remove " David Hawkins
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=548AF563.3050204@xs4all.nl \
--to=hverkuil@xs4all.nl \
--cc=dwh@ovro.caltech.edu \
--cc=gregkh@linuxfoundation.org \
--cc=ira.snyder@gmail.com \
--cc=iws@ovro.caltech.edu \
--cc=linux-kernel@vger.kernel.org \
--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