From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 7FDD2C54EBD for ; Fri, 13 Jan 2023 07:56:04 +0000 (UTC) Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id 02081A49B; Fri, 13 Jan 2023 08:55:12 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz 02081A49B DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1673596562; bh=vRdyiqF+PpWa7HcHfhCtuslDgXVXt8V4AOIdljF1BQo=; h=Date:From:To:Subject:References:In-Reply-To:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: Cc:From; b=Dj9cs6Lhy+Kw5JHmhFbHAjbUAUqm44Hna52mrRcmpH0eoDPxeyYBR1PGEXIHY7Z04 T+IRhrvwegrAYeMfS9DR6gotzAIAGrg5LvnhOQDHtnGyCy9wfMpHTpvrlj1f+8ijUC OgAzhkWevv0XL0/9PJP0mJdt9ZF5cHqQtKO0YJkc= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id AE2E3F8030F; Fri, 13 Jan 2023 08:55:11 +0100 (CET) Received: by alsa1.perex.cz (Postfix, from userid 50401) id B171AF8030F; Fri, 13 Jan 2023 08:55:09 +0100 (CET) Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [213.167.242.64]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id B75DFF8016D for ; Fri, 13 Jan 2023 08:55:06 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz B75DFF8016D Authentication-Results: alsa1.perex.cz; dkim=pass (1024-bit key, unprotected) header.d=ideasonboard.com header.i=@ideasonboard.com header.a=rsa-sha256 header.s=mail header.b=MplxUnHw Received: from pendragon.ideasonboard.com (85-76-5-15-nat.elisa-mobile.fi [85.76.5.15]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 2DBD14D4; Fri, 13 Jan 2023 08:55:05 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1673596505; bh=vRdyiqF+PpWa7HcHfhCtuslDgXVXt8V4AOIdljF1BQo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=MplxUnHw8ErIF7LyxmuN/+OcdYTL+KHEXJob0ft2ezOhqAVYE3m8P/x5TGmDxunAX hNGSu2Rfv8w4h3WiaTzkK6SyygY+7vdn0jtGh9flpoxnn8uBnWmrnoDsTJWb5wUuXu ajDTUWUaMexyxBvmwaL7WwO3/s2maPmjOdQvt9qo= Date: Fri, 13 Jan 2023 09:55:04 +0200 From: Laurent Pinchart To: Christoph Hellwig Subject: Re: [PATCH 01/22] gpu/drm: remove the shmobile drm driver Message-ID: References: <20230113062339.1909087-1-hch@lst.de> <20230113062339.1909087-2-hch@lst.de> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-fbdev@vger.kernel.org, Rich Felker , Geert Uytterhoeven , linux-sh@vger.kernel.org, alsa-devel@alsa-project.org, dri-devel@lists.freedesktop.org, linux-mtd@lists.infradead.org, linux-i2c@vger.kernel.org, linux-arch@vger.kernel.org, Yoshinori Sato , linux-serial@vger.kernel.org, linux-input@vger.kernel.org, linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-watchdog@vger.kernel.org, Arnd Bergmann , linux-gpio@vger.kernel.org, Greg Kroah-Hartman , linux-usb@vger.kernel.org, linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-spi@vger.kernel.org, linux-renesas-soc@vger.kernel.org, Kieran Bingham , netdev@vger.kernel.org, dmaengine@vger.kernel.org, linux-rtc@vger.kernel.org Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" On Fri, Jan 13, 2023 at 09:46:49AM +0200, Laurent Pinchart wrote: > Hi Christoph, > > Thank you for the patch. > > On Fri, Jan 13, 2023 at 07:23:18AM +0100, Christoph Hellwig wrote: > > This driver depends on ARM && ARCH_SHMOBILE, but ARCH_SHMOBILE can only be > > set for each/sh, making the driver dead code except for the COMPILE_TEST > > case. > > > > Signed-off-by: Christoph Hellwig > > No objection from me. > > Reviewed-by: Laurent Pinchart To expand a little bit on this, ARCH_SHMOBILE used to be set for the ARM-based shmobile SoCs too, until commit 08e735233ea29b17bfec8e4cb302e799d9f920b8 Author: Geert Uytterhoeven Date: Tue Aug 28 17:10:10 2018 +0200 ARM: shmobile: Remove the ARCH_SHMOBILE Kconfig symbol All drivers for Renesas ARM SoCs have gained proper ARCH_RENESAS platform dependencies. Hence finish the conversion from ARCH_SHMOBILE to ARCH_RENESAS for Renesas 32-bit ARM SoCs, as started by commit 9b5ba0df4ea4f940 ("ARM: shmobile: Introduce ARCH_RENESAS"). Signed-off-by: Geert Uytterhoeven Acked-by: Arnd Bergmann Signed-off-by: Simon Horman merged in v4.20. The DRM shmobile driver's Kconfig entry wasn't updated, making it dead code indeed. I haven't tested this driver in ages, hence my lack of objection, but someone may want to keep it for the pre-R-Car ARM SoCs. Also, I just noticed that you forgot to update the MAINTAINERS file in this patch to drop the drivers/gpu/drm/shmobile/ entry. > > --- > > drivers/gpu/drm/Kconfig | 2 - > > drivers/gpu/drm/Makefile | 1 - > > drivers/gpu/drm/shmobile/Kconfig | 12 - > > drivers/gpu/drm/shmobile/Makefile | 8 - > > .../gpu/drm/shmobile/shmob_drm_backlight.c | 82 --- > > .../gpu/drm/shmobile/shmob_drm_backlight.h | 19 - > > drivers/gpu/drm/shmobile/shmob_drm_crtc.c | 683 ------------------ > > drivers/gpu/drm/shmobile/shmob_drm_crtc.h | 55 -- > > drivers/gpu/drm/shmobile/shmob_drm_drv.c | 303 -------- > > drivers/gpu/drm/shmobile/shmob_drm_drv.h | 42 -- > > drivers/gpu/drm/shmobile/shmob_drm_kms.c | 150 ---- > > drivers/gpu/drm/shmobile/shmob_drm_kms.h | 29 - > > drivers/gpu/drm/shmobile/shmob_drm_plane.c | 261 ------- > > drivers/gpu/drm/shmobile/shmob_drm_plane.h | 19 - > > drivers/gpu/drm/shmobile/shmob_drm_regs.h | 310 -------- > > 15 files changed, 1976 deletions(-) > > delete mode 100644 drivers/gpu/drm/shmobile/Kconfig > > delete mode 100644 drivers/gpu/drm/shmobile/Makefile > > delete mode 100644 drivers/gpu/drm/shmobile/shmob_drm_backlight.c > > delete mode 100644 drivers/gpu/drm/shmobile/shmob_drm_backlight.h > > delete mode 100644 drivers/gpu/drm/shmobile/shmob_drm_crtc.c > > delete mode 100644 drivers/gpu/drm/shmobile/shmob_drm_crtc.h > > delete mode 100644 drivers/gpu/drm/shmobile/shmob_drm_drv.c > > delete mode 100644 drivers/gpu/drm/shmobile/shmob_drm_drv.h > > delete mode 100644 drivers/gpu/drm/shmobile/shmob_drm_kms.c > > delete mode 100644 drivers/gpu/drm/shmobile/shmob_drm_kms.h > > delete mode 100644 drivers/gpu/drm/shmobile/shmob_drm_plane.c > > delete mode 100644 drivers/gpu/drm/shmobile/shmob_drm_plane.h > > delete mode 100644 drivers/gpu/drm/shmobile/shmob_drm_regs.h -- Regards, Laurent Pinchart