[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20230103161550.4tui3ihl65olvkd7@houat>
Date: Tue, 3 Jan 2023 17:15:50 +0100
From: Maxime Ripard <maxime@...no.tech>
To: Robin Murphy <robin.murphy@....com>
Cc: Sean Anderson <sean.anderson@...o.com>,
David Airlie <airlied@...il.com>,
Daniel Vetter <daniel@...ll.ch>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Thomas Zimmermann <tzimmermann@...e.de>,
dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
Mark Brown <broonie@...nel.org>,
Jaroslav Kysela <perex@...ex.cz>,
Joerg Roedel <joro@...tes.org>,
Liam Girdwood <lgirdwood@...il.com>,
Matthias Brugger <matthias.bgg@...il.com>,
Takashi Iwai <tiwai@...e.com>, Will Deacon <will@...nel.org>,
Yong Wu <yong.wu@...iatek.com>, alsa-devel@...a-project.org,
iommu@...ts.linux.dev, linux-arm-kernel@...ts.infradead.org,
linux-mediatek@...ts.infradead.org
Subject: Re: [PATCH v2 2/3] iommu/sound: Use component_match_add_of helper
Hi Robin,
On Tue, Jan 03, 2023 at 01:01:07PM +0000, Robin Murphy wrote:
> Hi Sean,
>
> On 22/12/2022 11:37 pm, Sean Anderson wrote:
> > Convert users of component_match_add_release with component_release_of
> > and component_compare_of to component_match_add_of.
> >
> > Signed-off-by: Sean Anderson <sean.anderson@...o.com>
> > Acked-by: Mark Brown <broonie@...nel.org>
> > ---
> >
> > Changes in v2:
> > - Split off from helper addition
> >
> > drivers/iommu/mtk_iommu.c | 3 +--
> > drivers/iommu/mtk_iommu_v1.c | 3 +--
> > sound/soc/codecs/wcd938x.c | 6 ++----
> > 3 files changed, 4 insertions(+), 8 deletions(-)
> >
> > diff --git a/drivers/iommu/mtk_iommu.c b/drivers/iommu/mtk_iommu.c
> > index 2ab2ecfe01f8..483b7a9e4410 100644
> > --- a/drivers/iommu/mtk_iommu.c
> > +++ b/drivers/iommu/mtk_iommu.c
> > @@ -1079,8 +1079,7 @@ static int mtk_iommu_mm_dts_parse(struct device *dev, struct component_match **m
> > }
> > data->larb_imu[id].dev = &plarbdev->dev;
> > - component_match_add_release(dev, match, component_release_of,
> > - component_compare_of, larbnode);
> > + component_match_add_of(dev, match, larbnode);
>
> I've long since given up trying to make sense of how the DRM tree works, but
> the conflicting change is definitely already in mainline:
>
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit?id=b5765a1b44bea9dfcae69c53ffeb4c689d0922a7
As far as I can see, that patch doesn't affect DRM at all, and the
commit you pointed to doesn't either, nor has it been merged through the
DRM tree.
Can you expand a bit on how we're involved in this, what we should
clarify or help with?
Maxime
Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)
Powered by blists - more mailing lists