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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 AD8E5C433FE for ; Fri, 4 Nov 2022 09:21:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:Subject:Cc:To:From:Date:References: In-Reply-To:Message-Id:Mime-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=Tm/KhjYdNjpSB1IddjTMeLV/Ec38sskHodD98WH8Vzg=; b=QAyYgqDR2Hdtpv 2BxqYU3vhrI7IHOUy66u9u+k4HGqr1ISlMK4Sy8cUDpHGXgBfhI5EyNDdhz46r9WGtjnOtCtKnovA aR4i1nbvV9lJ8yMQPUV+Hg2tyygxYdO0Op31OfJOAyMptWE6GfYR72RYypKq+Ae2eXHooOQ3AIj5H fH7hsfHuYumrZnjmr2Wta/sQ1LaFcKczdfC5MFd84k79xHrsNvC5NymEKs5OWyIMnGb7ZXKbA4C0Q xS9iTHtmg5z7tOyPuIFpuDZDtdmAApvJBBL8r0FsgnBRjMycsYvXheUT8vub3oxOQg4XYYIZ+WA0T yj8SbkuzTQiac4E5HvzA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oqssc-0034au-Q5; Fri, 04 Nov 2022 09:20:50 +0000 Received: from new2-smtp.messagingengine.com ([66.111.4.224]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oqssY-0034Yi-4W; Fri, 04 Nov 2022 09:20:48 +0000 Received: from compute3.internal (compute3.nyi.internal [10.202.2.43]) by mailnew.nyi.internal (Postfix) with ESMTP id 8B17F580511; Fri, 4 Nov 2022 05:20:41 -0400 (EDT) Received: from imap51 ([10.202.2.101]) by compute3.internal (MEProxy); Fri, 04 Nov 2022 05:20:41 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arndb.de; h=cc :cc:content-type:date:date:from:from:in-reply-to:in-reply-to :message-id:mime-version:references:reply-to:sender:subject :subject:to:to; s=fm3; t=1667553641; x=1667560841; bh=SJoBz4HEVE RiFUnQzb3ZIub9dT0AONhBbUZgzTI8xuQ=; b=jOAioEcqTCQaB6i99azVot5cAk gre11Ve+vBmc+Fs/vkDIrNjCSzfHPOUS4wzGCXwfIKJy9biOGQnW4iqxv75LAqbt RdYq92O8WI8weSFAZzxtnrW2XuUukmSBN8KcaYYLkhR8/jLD6oh6MJQ9cwWd7yl8 mH21fZUTzBP3gaoYiV4qH0o52fq8RuYFTCOdHa7Soal92hUHyLnB5uIHP9R7ZXTY yrk8ucJVboxa6GlUfb8qItPWrs5N4B3xbNfDaXknJzA5ZsbLT0XMXiOZj0LDZKk8 6WHRFTqWjdbWEHnXA5Wxg4OS34kfyJ+czE32JS893SSJ8vebcjylW97lvLOw== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type:date:date:feedback-id :feedback-id:from:from:in-reply-to:in-reply-to:message-id :mime-version:references:reply-to:sender:subject:subject:to:to :x-me-proxy:x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s= fm3; t=1667553641; x=1667560841; bh=SJoBz4HEVERiFUnQzb3ZIub9dT0A ONhBbUZgzTI8xuQ=; b=BmHXv8dWgMDB3A30MW7r0ER+x2KPnCs67D5Tm4znOHms KOvE21G5Qhp1IrhrNpstSMDDbjbRSKKI+wikCsrQUA1D/bc3nhsP6Pfph0Q2sq/L w5xrS9CVegwYB6hNxnfcyAYSDOlMngVd/4QaY9exxV3aeJMS+7bUlXh8ZFR6fpSK gPa2Xr8LydLZo5vJzfOV1L/TUvE6+ISJhz2GoZweOm3SeuEKaa51K7Ium5W5K020 eA1TNoxWdJRVK08gDXfqjSbR39bjBE0aJEu+l1hoD12ykpSVSlQ8ZDa9eXuaf1Sv VIUPNFOv24w/xRDpe00prXWHte2pkStlJDT70gy7rw== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvgedrvddugddtudcutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpefofgggkfgjfhffhffvvefutgesthdtredtreertdenucfhrhhomhepfdetrhhn ugcuuegvrhhgmhgrnhhnfdcuoegrrhhnugesrghrnhgusgdruggvqeenucggtffrrghtth gvrhhnpeevhfffledtgeehfeffhfdtgedvheejtdfgkeeuvefgudffteettdekkeeufeeh udenucffohhmrghinhepkhgvrhhnvghlrdhorhhgnecuvehluhhsthgvrhfuihiivgeptd enucfrrghrrghmpehmrghilhhfrhhomheprghrnhgusegrrhhnuggsrdguvg X-ME-Proxy: Feedback-ID: i56a14606:Fastmail Received: by mailuser.nyi.internal (Postfix, from userid 501) id 17FD1B603ED; Fri, 4 Nov 2022 05:20:38 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.7.0-alpha0-1087-g968661d8e1-fm-20221021.001-g968661d8 Mime-Version: 1.0 Message-Id: <6b35d496-2313-4fd9-851b-2ba469bc9261@app.fastmail.com> In-Reply-To: <803778517.2279639.1667493436959.JavaMail.open-xchange@opme11oxm02aub.pom.fr.intraorange> References: <20221019161831.3864786-1-arnd@kernel.org> <7d9eebc8-39b6-4dc1-9ffc-f17ec584bee2@app.fastmail.com> <803778517.2279639.1667493436959.JavaMail.open-xchange@opme11oxm02aub.pom.fr.intraorange> Date: Fri, 04 Nov 2022 10:20:20 +0100 From: "Arnd Bergmann" To: "Robert Jarzmik" , "Arnd Bergmann" , "Robert Jarzmik" Cc: jingoohan1@gmail.com, "Linus Walleij" , "Sudip Mukherjee" , Linux-Renesas , marek.vasut@gmail.com, "open list:GPIO SUBSYSTEM" , "Ulf Hansson" , linux-pm@vger.kernel.org, "Sergey Shtylyov" , alsa-devel@alsa-project.org, philipp.zabel@gmail.com, linux-usb@vger.kernel.org, linux-leds@vger.kernel.org, slapin@ossfans.org, "Bartosz Golaszewski" , "Miquel Raynal" , "Mark Brown" , linux-mtd@lists.infradead.org, "Lee Jones" , "Russell King" , "Damien Le Moal" , linux-input@vger.kernel.org, mkpetch@internode.on.net, lgirdwood@gmail.com, "Vignesh Raghavendra" , "Daniel Mack" , kernel@wantstofly.org, "Dmitry Torokhov" , sre@kernel.org, linux-arm-kernel@lists.infradead.org, patches@opensource.cirrus.com, linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org, "Felipe Balbi" , "Helge Deller" , "Alan Stern" , "Wolfram Sang" , "Greg Kroah-Hartman" , perex@perex.cz, linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org, linux-rtc@vger.kernel.org, "linux-mmc @ vger . kernel . org" , lost.distance@yahoo.com, "Haojian Zhuang" , "Viresh Kumar" , tiwai@suse.com, "Dominik Brodowski" , "Alexandre Belloni" Subject: Re: [PATCH 00/30] ARM: pxa: remove all unused boards&drivers X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221104_022046_652879_5C6CCF4E X-CRM114-Status: GOOD ( 36.37 ) X-BeenThere: linux-mtd@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-mtd" Errors-To: linux-mtd-bounces+linux-mtd=archiver.kernel.org@lists.infradead.org On Thu, Nov 3, 2022, at 17:37, Jarzmik Robert wrote: > I'm sorry Arnd, my mailer messed up again, and instead of using my > normal robert.jarzmik@free.fr, it used my ISP mail ... >> > > The reason you're not seeing the AC97_BUS_NEW used is because this > becomes visible only in device-tree files, which were posted (for > mioa701 for example) but never properly reviewed nor merged. > > As from memory, at least mioa701, zylonite, em_x270 are DT ported, ie. > there is a DT file which makes them boot. For the mioa701, the DT file > offers the same functionnality, ie. all drivers in mioa701.c legacy > file are working as well in a DT variant (not using mioa701.c > obviously). Ok, I see. I need a little clarification here, so I can adapt my patch series to keep the necessary files in place: - Zylonite has both PXA300 and PXA320 variants. I removed the PXA320 code because nothing selected it any more and it is not hooked up to the MACH_PXA3XX_DT Kconfig symbol. Should I undo this and keep all three PXA3xx variant, removing only PXA930 but selecting CPU_PXA310/320 from MACH_PXA3XX_DT? - The em_x270 board file was already removed as part of 9d3239147d6d ("ARM: pxa: remove Compulab pxa2xx boards"), which already removed the associated drivers. Is it correct to assume that the DT support for it has also become unusable at that point, or would you expect it to still have some use? I now remove the related CM-X300 machine as well, but it sounds like you did not have DT support for that, right? - I'm not sure how the probing of the ASoC drivers works in the DT case. Do I only need to make SND_PXA2XX_SOC_AC97 user-visible to let everything get probed automatically from DT, or do we need to also keep SND_PXA2XX_SOC_MIOA701 and SND_SOC_ZYLONITE? >> Any idea where I went wrong here? Did I make a mistake in following the Kconfig dependencies, or are some parts of this > incorrectly annotated? > > I don't think you did a mistake, I think I did. When I saw the patch of > "deprecation" of mioa701 and all the other pxa files, I took it that > the platform-device was deprecated, and was to be removed. This is the > right thing to do in my opinion. I wouldn't mind if all board file go > actually (expect the QEMU one), as this would leave only the boards > with proper DT support, and would remove some clutter from the kernel. Right, this is the plan: the only board files I left in place are 'gumstix' family that is supported by qemu and has the largest amount of RAM in there, and the 'spitz' family that is also supported by qemu and apparently had at least one user that was interested in working on DT (I can't find a record of who that was now). > I also thought the drivers won't get touched by the purge, exception > made of "platform similar ones", such as in the sound tree > (sound/soc/pxa) where we have some of them, and maybe mfd tree. I made two lists of drivers here: the first list is for those that become invisible in Kconfig and can no longer even be compile tested. My reasoning here was that these are likely already dead and will only get worse without compile testing. If anyone ends up doing a new DT conversion of a removed board later on, these can obviously get resurrected. The current state of the patches is in [1], I'll go through them again based on your feedback, but let me know if you see anything else that I'm removing that you think should be kept. The second list of drivers is for those that have no DT support and are impossible to get used without anyone declaring a (platform, spi, i2c, ...) device in source code somewhere. The list is still incomplete because this is hard to check automatically. I have included a few patches to remove drivers that have been obviously unused for a long time, or never had an in-tree user at all, but I left the majority of these drivers for a later series. Arnd [1] https://git.kernel.org/pub/scm/linux/kernel/git/soc/soc.git/log/?h=boardfile-remove&id=73c4b7cfbc2b2ef0 ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/