Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757137AbdLWXwr (ORCPT ); Sat, 23 Dec 2017 18:52:47 -0500 Received: from vps-vb.mhejs.net ([37.28.154.113]:49420 "EHLO vps-vb.mhejs.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751325AbdLWXwp (ORCPT ); Sat, 23 Dec 2017 18:52:45 -0500 Subject: Re: [PATCH v5 3/6] cx25840: add pin to pad mapping and output format configuration To: Philippe Ombredanne Cc: Michael Krufky , Mauro Carvalho Chehab , Andy Walls , linux-kernel , Linux Media Mailing List , Hans Verkuil References: <07ce3ca4-b1db-331d-9218-83a2e7cc2017@maciej.szmigiero.name> From: "Maciej S. Szmigiero" Message-ID: <430a3221-20a6-639a-600f-04dada61c5bd@maciej.szmigiero.name> Date: Sun, 24 Dec 2017 00:52:41 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2440 Lines: 57 On 23.12.2017 15:08, Philippe Ombredanne wrote: > On Sat, Dec 23, 2017 at 12:18 AM, Maciej S. Szmigiero > wrote: >> This commit adds pin to pad mapping and output format configuration support >> in CX2584x-series chips to cx25840 driver. >> >> This functionality is then used to allow disabling ivtv-specific hacks >> (called a "generic mode"), so cx25840 driver can be used for other devices >> not needing them without risking compatibility problems. >> >> Signed-off-by: Maciej S. Szmigiero >> --- >> drivers/media/i2c/cx25840/cx25840-core.c | 396 ++++++++++++++++++++++++++++++- >> drivers/media/i2c/cx25840/cx25840-core.h | 13 + >> drivers/media/i2c/cx25840/cx25840-vbi.c | 3 + >> include/media/drv-intf/cx25840.h | 74 +++++- >> 4 files changed, 484 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/media/i2c/cx25840/cx25840-core.c b/drivers/media/i2c/cx25840/cx25840-core.c >> index 2189980a0f29..e2fd33a64550 100644 >> --- a/drivers/media/i2c/cx25840/cx25840-core.c >> +++ b/drivers/media/i2c/cx25840/cx25840-core.c >> @@ -21,6 +21,9 @@ >> * CX23888 DIF support for the HVR1850 >> * Copyright (C) 2011 Steven Toth >> * >> + * CX2584x pin to pad mapping and output format configuration support are >> + * Copyright (C) 2011 Maciej S. Szmigiero >> + * >> * This program is free software; you can redistribute it and/or >> * modify it under the terms of the GNU General Public License >> * as published by the Free Software Foundation; either version 2 > > Since you are touching the copyright here, I wonder if you could reach > out to other copyright holders and switch to using an SPDX tag > instead? > Well, I'm really just adding two functionalities to the cx25840 driver, which consist of 6 additional files besides "cx25840-core.c". All this code is mostly written by other people, in addition to this there is also a note at the top of this file that the driver is "[b]ased on the saa7115 driver and on the first version of Chris Kennedy's cx25840 driver". Since of 119 *.c files in drivers/media/i2c/ directory only 3 already have SPDX tags the rest will (probably) be amended with this tag anyway, and that's when the cx25840 driver can also be tagged, using an appropriate process for this (IANAL). > -- > Cordially > Philippe Ombredanne Best regards, Maciej Szmigiero