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 gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (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 4C99BC4332F for ; Fri, 16 Dec 2022 17:42:06 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id AA32310E14D; Fri, 16 Dec 2022 17:41:59 +0000 (UTC) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by gabe.freedesktop.org (Postfix) with ESMTP id 214C910E0F1; Fri, 16 Dec 2022 17:41:55 +0000 (UTC) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id A7FD41042; Fri, 16 Dec 2022 09:42:33 -0800 (PST) Received: from [10.57.88.234] (unknown [10.57.88.234]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id DA5E33F5A1; Fri, 16 Dec 2022 09:41:45 -0800 (PST) Message-ID: <2db73405-464f-6980-a7c1-7fe232611331@arm.com> Date: Fri, 16 Dec 2022 17:41:40 +0000 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; rv:102.0) Gecko/20100101 Thunderbird/102.5.1 Subject: Re: [PATCH 0/2] drm: Add component_match_add_of and convert users of drm_of_component_match_add Content-Language: en-GB To: Sean Anderson , David Airlie , Daniel Vetter , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , dri-devel@lists.freedesktop.org References: <20221103182222.2247724-1-sean.anderson@seco.com> <68562aca-5256-9e4b-bcd5-983e43408a7d@seco.com> From: Robin Murphy In-Reply-To: <68562aca-5256-9e4b-bcd5-983e43408a7d@seco.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: alsa-devel@alsa-project.org, Xinliang Liu , Liviu Dudau , linux-kernel@vger.kernel.org, Paul Cercueil , John Stultz , Mihail Atanassov , Will Deacon , Samuel Holland , Joerg Roedel , Takashi Iwai , Russell King , Jernej Skrabec , Chen-Yu Tsai , Mali DP Maintainers , linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org, Alain Volmat , Xinwei Kong , linux-sunxi@lists.linux.dev, Chun-Kuang Hu , Chen Feng , Abhinav Kumar , etnaviv@lists.freedesktop.org, Mark Brown , linux-mediatek@lists.infradead.org, Matthias Brugger , Jaroslav Kysela , Sean Paul , Yong Wu , linux-mips@vger.kernel.org, Tomi Valkeinen , iommu@lists.linux.dev, freedreno@lists.freedesktop.org, Liam Girdwood , Dmitry Baryshkov , Tian Tao , Jyri Sarha Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" On 2022-12-16 17:08, Sean Anderson wrote: > On 11/3/22 14:22, Sean Anderson wrote: >> This series adds a new function component_match_add_of to simplify the >> common case of calling component_match_add_release with >> component_release_of and component_compare_of. There is already >> drm_of_component_match_add, which allows for a custom compare function. >> However, all existing users just use component_compare_of (or an >> equivalent). >> >> I can split the second commit up if that is easier to review. >> >> >> Sean Anderson (2): >> component: Add helper for device nodes >> drm: Convert users of drm_of_component_match_add to >> component_match_add_of >> >> .../gpu/drm/arm/display/komeda/komeda_drv.c | 6 ++-- >> drivers/gpu/drm/arm/hdlcd_drv.c | 9 +----- >> drivers/gpu/drm/arm/malidp_drv.c | 11 +------ >> drivers/gpu/drm/armada/armada_drv.c | 10 ++++--- >> drivers/gpu/drm/drm_of.c | 29 +++---------------- >> drivers/gpu/drm/etnaviv/etnaviv_drv.c | 4 +-- >> .../gpu/drm/hisilicon/kirin/kirin_drm_drv.c | 3 +- >> drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 3 +- >> drivers/gpu/drm/mediatek/mtk_drm_drv.c | 4 +-- >> drivers/gpu/drm/msm/msm_drv.c | 14 ++++----- >> drivers/gpu/drm/sti/sti_drv.c | 3 +- >> drivers/gpu/drm/sun4i/sun4i_drv.c | 3 +- >> drivers/gpu/drm/tilcdc/tilcdc_external.c | 10 ++----- >> drivers/iommu/mtk_iommu.c | 3 +- >> drivers/iommu/mtk_iommu_v1.c | 3 +- >> include/drm/drm_of.h | 12 -------- >> include/linux/component.h | 9 ++++++ >> sound/soc/codecs/wcd938x.c | 6 ++-- >> 18 files changed, 46 insertions(+), 96 deletions(-) >> > > ping? > > Should I send a v2 broken up like Mark suggested? FWIW you'll need to rebase the IOMMU changes on 6.2-rc1 anyway - mtk_iommu stops using component_match_add_release() at all. Thanks, Robin.