linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@kernel.org>
To: Soeren Moch <smoch@web.de>
Cc: Andreas Regel <andreas.regel@gmx.de>,
	Manu Abraham <manu@linuxtv.org>,
	Oliver Endriss <o.endriss@gmx.de>,
	linux-media@vger.kernel.org
Subject: Re: [GIT PULL RESEND] SAA716x DVB driver
Date: Wed, 7 Mar 2018 12:14:38 -0300	[thread overview]
Message-ID: <20180307121438.389f411c@vento.lan> (raw)
In-Reply-To: <76416aa4-0955-b10b-42ba-f24ea0bea678@web.de>

Hi Sören,

Em Wed, 23 Aug 2017 11:56:50 +0200
Soeren Moch <smoch@web.de> escreveu:

> Resend this pull request. Apparently my explanation one month ago,
> why we need the userspace API of this driver in the current form [1],
> got lost.

As discussed in priv, I'm ok to merge this at staging, provided that
you add this to its TODO[1] with something like:

	* The osd/video/audio APIs should be redesigned in order to fit
	  the needs for modern embedded hardware.

In other words, once we have an API for FF devices that work for modern
embedded hardware, this driver should either be changed to work with it
or may be removed on future Kernel versions, if it becomes incompatible
with whatever DVB core changes required for it.

[1] https://github.com/s-moch/linux-saa716x/blob/for-media/drivers/staging/media/saa716x/TODO

I'm marking the current pull request with "Changes Requested".

Thanks,
Mauro

      reply	other threads:[~2018-03-07 15:14 UTC|newest]

Thread overview: 26+ 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
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 [this message]

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=20180307121438.389f411c@vento.lan \
    --to=mchehab@kernel.org \
    --cc=andreas.regel@gmx.de \
    --cc=linux-media@vger.kernel.org \
    --cc=manu@linuxtv.org \
    --cc=o.endriss@gmx.de \
    --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).