From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mail.kernel.org (mail.kernel.org [198.145.29.99])
	(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
	(No client certificate requested)
	by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9B852173
	for <linux-staging@lists.linux.dev>; Tue,  1 Jun 2021 08:28:35 +0000 (UTC)
Received: by mail.kernel.org (Postfix) with ESMTPSA id 818CA61003;
	Tue,  1 Jun 2021 08:28:32 +0000 (UTC)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org;
	s=k20201202; t=1622536115;
	bh=bedRNk8wKv9NNSArocmrL5fhZW3QrCccbA4cX/dEOdc=;
	h=Date:From:To:Cc:Subject:In-Reply-To:References:From;
	b=tgWxKFb46eJYPD/PnkAXyI8LDCApIWd3GHG/fyhfFfyFiSbjUSNuWjDj5ShGwFONM
	 RZgzn7tu17i4WOx4ZtuDZxDbbROuULhzo/S8oktQHERFhrx+rkytFudN72++fK98Nc
	 894NSiCoXzQDYXcNdyZAKV+P/4gsK7DsE2zF15SL5tcqZ3EDhKHb9gMbUst9Kkbhon
	 vIFo4DZsRrI3qh68zkhvkPx6t0BzavOufxg3wxtXJoQfOOrBxu1VjI/M1EPoxw6Yg9
	 izZigHWQxDKLWPC0UKUMNHSudlbdvYUjaptZPC0J8y0fPHkx7R4brhcwIukny8knac
	 voSA9ZWr/Mbkg==
Date: Tue, 1 Jun 2021 10:28:29 +0200
From: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: linuxarm@huawei.com, mauro.chehab@huawei.com, Alaa Emad
 <alaaemadhossney.ae@gmail.com>, Brad Love <brad@nextdimension.cc>, Hans
 Verkuil <hverkuil-cisco@xs4all.nl>, Mauro Carvalho Chehab
 <mchehab@kernel.org>, Maxim Plotnikov <wgh@torlan.ru>, Sean Young
 <sean@mess.org>, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org,
 linux-staging@lists.linux.dev
Subject: Re: [PATCH] media: sp8870: move it to staging
Message-ID: <20210601102829.37d9447a@coco.lan>
In-Reply-To: <YLW8rkDRNp1TdWqY@kroah.com>
References: <5e694bc1b0670b58b93a9fea5712ed0f306e6050.1622493461.git.mchehab+huawei@kernel.org>
	<YLW8rkDRNp1TdWqY@kroah.com>
X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.33; x86_64-redhat-linux-gnu)
X-Mailing-List: linux-staging@lists.linux.dev
List-Id: <linux-staging.lists.linux.dev>
List-Subscribe: <mailto:linux-staging+subscribe@lists.linux.dev>
List-Unsubscribe: <mailto:linux-staging+unsubscribe@lists.linux.dev>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

Em Tue, 1 Jun 2021 06:50:54 +0200
Greg Kroah-Hartman <gregkh@linuxfoundation.org> escreveu:

> On Mon, May 31, 2021 at 10:37:54PM +0200, Mauro Carvalho Chehab wrote:
> > This driver is used only ba av7110, which is preparing for
> > its retirement. So, move this ancillary driver to stay together
> > with av7110.
> > 
> > Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
> > ---
> >  drivers/media/dvb-frontends/Kconfig           | 12 -----------
> >  drivers/media/dvb-frontends/Makefile          |  1 -
> >  drivers/staging/media/av7110/Kconfig          | 20 +++++++++++++++++++
> >  drivers/staging/media/av7110/Makefile         |  2 ++
> >  .../media/av7110}/sp8870.c                    |  0
> >  .../media/av7110}/sp8870.h                    |  0
> >  6 files changed, 22 insertions(+), 13 deletions(-)
> >  rename drivers/{media/dvb-frontends => staging/media/av7110}/sp8870.c (100%)
> >  rename drivers/{media/dvb-frontends => staging/media/av7110}/sp8870.h (100%)  
> 
> Don't you need a TODO file for these moves that says what is wrong with
> the driver and when it will be removed and to not send cleanups for it?

Yes, but it was added on the previous patch that moved the master driver
(av7110):

	https://lore.kernel.org/linux-media/4e7136434c68eabcdb66d25f44946ab794f4af0b.1622476959.git.mchehab+huawei@kernel.org/T/#u

The TODO is just saying:

	This driver is too old and relies on a different API.
	Drop it from Kernel on a couple of versions.

(there was a typo there, I'll fix at the next version)

Thanks,
Mauro