linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Tycho Lürsen" <tycholursen@gmail.com>
To: Jemma Denson <jdenson@gmail.com>, Soeren Moch <smoch@web.de>,
	Mauro Carvalho Chehab <mchehab@s-opensource.com>,
	Luis Alves <ljalvs@gmail.com>,
	Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: SAA716x DVB driver
Date: Sat, 20 Jan 2018 16:49:42 +0100	[thread overview]
Message-ID: <d76143d2-5c7d-87ca-3f2f-1a73778b400f@gmail.com> (raw)
In-Reply-To: <CAObVMRvxT_=LmO-mJNPRewQq05PqMHpD83m1UBoK+QiBSwqUNw@mail.gmail.com>

Hi Jemma,


Op 19-01-18 om 16:11 schreef Jemma Denson:
> Hi Tycho,
>
> On 19/01/18 13:59, Tycho Lürsen wrote:
>> Hi Jemma,
>>
>> I'm with you: let's get merged at least something!
>>
>> Did you find  a maintainer for this driver?
>> I can do simple stuff like in my fork of Soeren Moch's repo, but thats
>> where it ends. I dont have the knowledge needed to maintain a driver.
> Not yet, but I can't really say I've been looking - unfortunately real
> life got in the way of anything over christmas. I'm not sure I do
> either, but it really depends on what's required. From what I can see
> from maintaining another driver then as long as the driver is working
> there's not a whole lot to do.
Right, but we still need a maintainer. Are you capable/willing to 
volunteer for the job?
>
>> I think that your proposal to use a stripped version of Luis Alves
>> repo is a no go, since it contains a couple of demod/tuner drivers
>> that are not upstreamed yet. That complicates the upstreaming process
>> too much, I think.
> Oh, I would have stripped it *right* down and removed every card except
> my TBS6280. The end result would probably be pretty close to Soeren's at
> that point anyway, so I was starting to think like what you've done and
> base it on that instead.
If you want, I can strip the driver down a lot more and ad back the 
drivers you need. Just tell me what it is you need.
>
>> I used a stripped version of Soeren Moch's repo to prove its stability
>> instead, adding the drivers I need so I can test it. You can see what
>> I did at :
>> https://github.com/bas-t/linux-saa716x/commits/for-media-stripped
>>
>> This has been tested with linux 4.9.77, 4.14.14 and 4.15-rc8.
>> Works like a charm for me.
>>
> Looks like a good start, I'd be tempted to remove all the other cards
> though unless you have them available to test with. Keeps the submission
> simpler and less to worry about, they can be added back in later if
> someone has an itch to scratch (and hardware to test with!).
Isn't that a bit drastic?
I mean: those few drivers have been there for ages, and I can't recall 
anyone complaining about them in a serious way.
>
> I do have a few other tbs 716x cards available here at work so might be
> able to test some others out, but we're a bit busy at the moment so
> would have to be on my own time and there's not much of that available
> at the moment either :(
As I said: give me the numbers of your tbs cards and I will add support 
for them (if they are supported by Luis Alves repo).
That way you are able to test the stability of the saa716x driver in 
it's present state.
>
>
> Jemma.
Tycho.

  reply	other threads:[~2018-01-20 15:49 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-16 18:34 [GIT PULL] SAA716x DVB driver Soeren Moch
2017-07-20 10:49 ` Mauro Carvalho Chehab
2017-07-21 20:44 ` Soeren Moch
2017-08-27 10:30   ` Mauro Carvalho Chehab
2017-09-09 12:52     ` Soeren Moch
2017-09-09 21:20       ` Mauro Carvalho Chehab
2017-09-16 12:54         ` Soeren Moch
2017-09-16 17:49           ` Mauro Carvalho Chehab
2017-09-24 22:17             ` Soeren Moch
2017-11-24 16:28               ` Tycho Lürsen
2017-11-27 11:24                 ` Mauro Carvalho Chehab
2017-12-02 18:51                   ` Jemma Denson
2017-12-02 19:49                     ` Mauro Carvalho Chehab
2017-12-02 23:59                       ` Soeren Moch
2017-12-03 10:57                         ` Jemma Denson
2017-12-03 14:11                           ` Soeren Moch
2017-12-03 15:04                             ` Jemma Denson
2017-12-04 10:07                               ` Mauro Carvalho Chehab
2018-01-19 13:59                           ` Tycho Lürsen
2018-01-19 15:11                             ` Jemma Denson
2018-01-20 15:49                               ` Tycho Lürsen [this message]
2018-01-25 17:08                                 ` Jemma Denson
2017-11-28 18:35               ` [GIT PULL] " Mauro Carvalho Chehab
2017-12-02 23:58                 ` Soeren Moch
2017-08-23  9:56 ` [GIT PULL RESEND] " Soeren Moch
2018-03-07 15:14   ` Mauro Carvalho Chehab
2018-01-25 19:56 Soeren Moch
2018-01-26  7:59 ` Tycho Lürsen

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=d76143d2-5c7d-87ca-3f2f-1a73778b400f@gmail.com \
    --to=tycholursen@gmail.com \
    --cc=jdenson@gmail.com \
    --cc=linux-media@vger.kernel.org \
    --cc=ljalvs@gmail.com \
    --cc=mchehab@s-opensource.com \
    --cc=smoch@web.de \
    /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).