Patchwork [em28xx] add config for em28xx-based board by MAGIX

Em Fri, 18 Aug 2017 12:11:00 +0000
panic <lists@xandea.de> escreveu:

> Hi,
> 
> the patch below adds the entries to the config arrays for a capture-only
> board distributed by MAGIX [0]. The hardware itself (EM2860, SAA7113,
> EMP202) is already supported.

> This patch lacks the configuration for the GPIO pin, because I had/have
> no time yet to figure out how it works. Video and audio work fine for me
> in mplayer/mencoder.

You may won't need to touch it. Several capture-only boards just don't
use GPIO at all. That's easy to test: just connect the hardware on your
machine with Linux booted with the driver. If it works as-is, you
won't need to touch GPIOs. If, on the other hand, you need to first
boot it with the original driver and reboot linux, then you'll need
to sniff what the original driver is doing with regards to GPIO.

> The patch works against Linux 4.9.0 from Debian stretch/stable.
> 
> This is my first kernel submission, so tell me if you need more info or
> if something should be changed. Thanks!
> 
> Cheers,
> panic
> 
> [0] contains not much info, but for the record:
>     http://www.magix.com/gb/rescue-your-videotapes/

The patch itself is OK, except that it should be based on upstream
Kernel (although I can easily rebase it, as the enclosed version).

The main issue with it is that you need to follow the
submission rules of the Kernel. In particular, all patches
should contain your real name and a Signed-off-by with it, as
stated at:

	https://www.kernel.org/doc/html/latest/process/submitting-patches.html#sign-your-work-the-developer-s-certificate-of-origin

As reference, I enclosed how the patch should like, rebased to
the upstream Kernel version. You'll need to put your real name
on it.

Thanks,
Mauro


-

[em28xx] add config for em28xx-based board by MAGIX

From: your real name <lists@xandea.de>

Adds the entries to the config arrays for a capture-only board 
distributed by MAGIX [0]. The hardware itself (EM2860, SAA7113,
EMP202) is already supported.

[0] contains not much info, but for the record:
    http://www.magix.com/gb/rescue-your-videotapes/

Signed-off-by: your real name <lists@xandea.de>


---
 drivers/media/usb/em28xx/em28xx-cards.c |   20 ++++++++++++++++++++
 drivers/media/usb/em28xx/em28xx.h       |    1 +
 2 files changed, 21 insertions(+)