Message ID | 20230307163041.3815-6-wsa+renesas@sang-engineering.com (mailing list archive) |
---|---|
State | Accepted |
Delegated to: | Sakari Ailus |
Headers |
Received: from vger.kernel.org ([23.128.96.18]) by www.linuxtv.org with esmtp (Exim 4.92) (envelope-from <linux-media-owner@vger.kernel.org>) id 1pZaE0-00D0ea-7T; Tue, 07 Mar 2023 16:31:40 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230379AbjCGQbh (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Tue, 7 Mar 2023 11:31:37 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39622 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229879AbjCGQbc (ORCPT <rfc822;linux-media@vger.kernel.org>); Tue, 7 Mar 2023 11:31:32 -0500 Received: from mail.zeus03.de (www.zeus03.de [194.117.254.33]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id DE1AD85379 for <linux-media@vger.kernel.org>; Tue, 7 Mar 2023 08:31:10 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=simple; d=sang-engineering.com; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; s=k1; bh=BL47/9gshlsQiA D5V7ioQV+yS8e/Scob5w8583YxF3o=; b=JDSYoj8VlQmdXygO7ag7zGmEizmUR4 rmYi5kQbOz4dp6qm+SOZ12S6B5zg5R1o8Kg9yxr6EmDyWyo4SJUD5Ia8qNI39dSv rvXDSYTdsSwBBj/B8rb1CLBD2QULzLezh88j6DLbeUFrATr7zs1eJiy2ajVsPO9y U02JKNT4atRw8= Received: (qmail 752025 invoked from network); 7 Mar 2023 17:31:05 +0100 Received: by mail.zeus03.de with ESMTPSA (TLS_AES_256_GCM_SHA384 encrypted, authenticated); 7 Mar 2023 17:31:05 +0100 X-UD-Smtp-Session: l3s3148p1@ROgh8FH2bI0gAQnoAFQ+AGEn9EY5VOxJ From: Wolfram Sang <wsa+renesas@sang-engineering.com> To: linux-renesas-soc@vger.kernel.org Cc: Wolfram Sang <wsa+renesas@sang-engineering.com>, Geert Uytterhoeven <geert+renesas@glider.be>, Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>, Mauro Carvalho Chehab <mchehab@kernel.org>, linux-media@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 05/11] media: renesas: fdp1: remove R-Car H3 ES1.* handling Date: Tue, 7 Mar 2023 17:30:33 +0100 Message-Id: <20230307163041.3815-6-wsa+renesas@sang-engineering.com> X-Mailer: git-send-email 2.35.1 In-Reply-To: <20230307163041.3815-1-wsa+renesas@sang-engineering.com> References: <20230307163041.3815-1-wsa+renesas@sang-engineering.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO, RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_PASS,SPF_NONE, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-LSpam-Score: -2.3 (--) X-LSpam-Report: No, score=-2.3 required=5.0 tests=BAYES_00=-1.9,DKIM_SIGNED=0.1,HEADER_FROM_DIFFERENT_DOMAINS=0.5,MAILING_LIST_MULTI=-1,T_DKIM_INVALID=0.01 autolearn=ham autolearn_force=no |
Series |
tree-wide: remove support for Renesas R-Car H3 ES1
|
|
Commit Message
Wolfram Sang
March 7, 2023, 4:30 p.m. UTC
R-Car H3 ES1.* was only available to an internal development group and needed a lot of quirks and workarounds. These become a maintenance burden now, so our development group decided to remove upstream support and disable booting for this SoC. Public users only have ES2 onwards. Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com> --- Please apply individually per subsystem. There are no dependencies and the SoC doesn't boot anymore since v6.3-rc1. drivers/media/platform/renesas/rcar_fdp1.c | 4 ---- 1 file changed, 4 deletions(-)
Comments
Quoting Wolfram Sang (2023-03-07 16:30:33) > R-Car H3 ES1.* was only available to an internal development group and > needed a lot of quirks and workarounds. These become a maintenance > burden now, so our development group decided to remove upstream support > and disable booting for this SoC. Public users only have ES2 onwards. > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com> Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> > --- > Please apply individually per subsystem. There are no dependencies and the SoC > doesn't boot anymore since v6.3-rc1. > > drivers/media/platform/renesas/rcar_fdp1.c | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/drivers/media/platform/renesas/rcar_fdp1.c b/drivers/media/platform/renesas/rcar_fdp1.c > index 37ecf489d112..f4a1fc8ce595 100644 > --- a/drivers/media/platform/renesas/rcar_fdp1.c > +++ b/drivers/media/platform/renesas/rcar_fdp1.c > @@ -254,7 +254,6 @@ MODULE_PARM_DESC(debug, "activate debug info"); > > /* Internal Data (HW Version) */ > #define FD1_IP_INTDATA 0x0800 > -#define FD1_IP_H3_ES1 0x02010101 > #define FD1_IP_M3W 0x02010202 > #define FD1_IP_H3 0x02010203 > #define FD1_IP_M3N 0x02010204 > @@ -2359,9 +2358,6 @@ static int fdp1_probe(struct platform_device *pdev) > > hw_version = fdp1_read(fdp1, FD1_IP_INTDATA); > switch (hw_version) { > - case FD1_IP_H3_ES1: > - dprintk(fdp1, "FDP1 Version R-Car H3 ES1\n"); > - break; > case FD1_IP_M3W: > dprintk(fdp1, "FDP1 Version R-Car M3-W\n"); > break; > -- > 2.35.1 >
Hi Wolfram, On Tue, Mar 7, 2023 at 5:31 PM Wolfram Sang <wsa+renesas@sang-engineering.com> wrote: > R-Car H3 ES1.* was only available to an internal development group and > needed a lot of quirks and workarounds. These become a maintenance > burden now, so our development group decided to remove upstream support > and disable booting for this SoC. Public users only have ES2 onwards. > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com> Thanks for your patch, which is now commit af4273b43f2bd9ee ("media: renesas: fdp1: remove R-Car H3 ES1.* handling") in media/master (next-20230417 and later). > --- a/drivers/media/platform/renesas/rcar_fdp1.c > +++ b/drivers/media/platform/renesas/rcar_fdp1.c > @@ -254,7 +254,6 @@ MODULE_PARM_DESC(debug, "activate debug info"); > > /* Internal Data (HW Version) */ > #define FD1_IP_INTDATA 0x0800 > -#define FD1_IP_H3_ES1 0x02010101 > #define FD1_IP_M3W 0x02010202 > #define FD1_IP_H3 0x02010203 > #define FD1_IP_M3N 0x02010204 > @@ -2359,9 +2358,6 @@ static int fdp1_probe(struct platform_device *pdev) > > hw_version = fdp1_read(fdp1, FD1_IP_INTDATA); > switch (hw_version) { > - case FD1_IP_H3_ES1: > - dprintk(fdp1, "FDP1 Version R-Car H3 ES1\n"); > - break; > case FD1_IP_M3W: > dprintk(fdp1, "FDP1 Version R-Car M3-W\n"); > break; Apparently 0x02010101 is also used on (at least) R-Car M2-W ES1.0, causing the following annoying (but further harmless?) messages during boot: rcar_fdp1 fe940000.fdp1: FDP1 Unidentifiable (0x02010101) rcar_fdp1 fe944000.fdp1: FDP1 Unidentifiable (0x02010101) Note that the R-Car Gen2 documentation states the register's contents are all zeroes. But that value would trigger the error message, too. Sorry for not noticing before. Apparently I never booted a kernel with this patch on koelsch... Gr{oetje,eeting}s, Geert
Hi Geert, thanks for reporting! > > -#define FD1_IP_H3_ES1 0x02010101 ... > Apparently 0x02010101 is also used on (at least) R-Car M2-W ES1.0, > causing the following annoying (but further harmless?) messages > during boot: > > rcar_fdp1 fe940000.fdp1: FDP1 Unidentifiable (0x02010101) > rcar_fdp1 fe944000.fdp1: FDP1 Unidentifiable (0x02010101) Hmm, that means before my removal patch, Gen2 has been incorrectly defined as H3 ES1? > Note that the R-Car Gen2 documentation states the register's contents > are all zeroes. But that value would trigger the error message, too. Bad, but well... > Sorry for not noticing before. Apparently I never booted a kernel > with this patch on koelsch... We could re-add this version and just let it print "FDP1 Initial Version" or something? I could test this on my Lager board. All the best, Wolfram
Hi Wolfram, On Mon, Apr 24, 2023 at 9:17 PM Wolfram Sang <wsa+renesas@sang-engineering.com> wrote: > > > -#define FD1_IP_H3_ES1 0x02010101 > > ... > > > Apparently 0x02010101 is also used on (at least) R-Car M2-W ES1.0, > > causing the following annoying (but further harmless?) messages > > during boot: > > > > rcar_fdp1 fe940000.fdp1: FDP1 Unidentifiable (0x02010101) > > rcar_fdp1 fe944000.fdp1: FDP1 Unidentifiable (0x02010101) > > Hmm, that means before my removal patch, Gen2 has been incorrectly > defined as H3 ES1? Indeed, but the driver doesn't seem to do anything with the detected version, except for printing a debug or error message. > > Note that the R-Car Gen2 documentation states the register's contents > > are all zeroes. But that value would trigger the error message, too. > > Bad, but well... > > > Sorry for not noticing before. Apparently I never booted a kernel > > with this patch on koelsch... > > We could re-add this version and just let it print "FDP1 Initial > Version" or something? I could test this on my Lager board. I plan to test it on a few other boards, too... Just wanted to let you know ASAP... Gr{oetje,eeting}s, Geert
On Mon, Apr 24, 2023 at 09:37:00PM +0200, Geert Uytterhoeven wrote: > On Mon, Apr 24, 2023 at 9:17 PM Wolfram Sang wrote: > > > > -#define FD1_IP_H3_ES1 0x02010101 > > > > ... > > > > > Apparently 0x02010101 is also used on (at least) R-Car M2-W ES1.0, > > > causing the following annoying (but further harmless?) messages > > > during boot: > > > > > > rcar_fdp1 fe940000.fdp1: FDP1 Unidentifiable (0x02010101) > > > rcar_fdp1 fe944000.fdp1: FDP1 Unidentifiable (0x02010101) > > > > Hmm, that means before my removal patch, Gen2 has been incorrectly > > defined as H3 ES1? > > Indeed, but the driver doesn't seem to do anything with the detected > version, except for printing a debug or error message. It may not now, but I'd rather get this fixes nonetheless to avoid forgetting about the issue and wondering later once we take the version into account why M2-W doesn't work. > > > Note that the R-Car Gen2 documentation states the register's contents > > > are all zeroes. But that value would trigger the error message, too. > > > > Bad, but well... > > > > > Sorry for not noticing before. Apparently I never booted a kernel > > > with this patch on koelsch... > > > > We could re-add this version and just let it print "FDP1 Initial > > Version" or something? I could test this on my Lager board. > > I plan to test it on a few other boards, too... > Just wanted to let you know ASAP...
diff --git a/drivers/media/platform/renesas/rcar_fdp1.c b/drivers/media/platform/renesas/rcar_fdp1.c index 37ecf489d112..f4a1fc8ce595 100644 --- a/drivers/media/platform/renesas/rcar_fdp1.c +++ b/drivers/media/platform/renesas/rcar_fdp1.c @@ -254,7 +254,6 @@ MODULE_PARM_DESC(debug, "activate debug info"); /* Internal Data (HW Version) */ #define FD1_IP_INTDATA 0x0800 -#define FD1_IP_H3_ES1 0x02010101 #define FD1_IP_M3W 0x02010202 #define FD1_IP_H3 0x02010203 #define FD1_IP_M3N 0x02010204 @@ -2359,9 +2358,6 @@ static int fdp1_probe(struct platform_device *pdev) hw_version = fdp1_read(fdp1, FD1_IP_INTDATA); switch (hw_version) { - case FD1_IP_H3_ES1: - dprintk(fdp1, "FDP1 Version R-Car H3 ES1\n"); - break; case FD1_IP_M3W: dprintk(fdp1, "FDP1 Version R-Car M3-W\n"); break;