Re: [PATCH 2/2] media: venus: firmware: Use iommu_paging_domain_alloc()
From: Jason Gunthorpe
Date: Wed Sep 04 2024 - 08:16:36 EST
On Tue, Aug 20, 2024 at 11:43:50PM +0300, Stanimir Varbanov wrote:
> Hi,
>
> Thank you for the patch!
>
> On 12.08.24 г. 10:21 ч., Lu Baolu wrote:
> > An iommu domain is allocated in venus_firmware_init() and is attached to
> > core->fw.dev in the same function. Use iommu_paging_domain_alloc() to
> > make it explicit.
> >
> > Signed-off-by: Lu Baolu <baolu.lu@xxxxxxxxxxxxxxx>
> > Reviewed-by: Jason Gunthorpe <jgg@xxxxxxxxxx>
> > Link: https://lore.kernel.org/r/20240610085555.88197-10-baolu.lu@xxxxxxxxxxxxxxx
> > ---
> > drivers/media/platform/qcom/venus/firmware.c | 6 +++---
> > 1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/media/platform/qcom/venus/firmware.c b/drivers/media/platform/qcom/venus/firmware.c
> > index fe7da2b30482..66a18830e66d 100644
> > --- a/drivers/media/platform/qcom/venus/firmware.c
> > +++ b/drivers/media/platform/qcom/venus/firmware.ced
>
> Acked-by: Stanimir Varbanov <stanimir.k.varbanov@xxxxxxxxx>
>
> I'll take the patch through linux-media tree.
Did this patch and the 1/2:
https://lore.kernel.org/linux-iommu/20240812072105.9578-1-baolu.lu@xxxxxxxxxxxxxxx/
Get picked up for media? I don't see it in linux-next?
Thanks,
Jason