Re: [PATCH 2/2] media: venus: firmware: Use iommu_paging_domain_alloc()
From: Jason Gunthorpe
Date: Fri Sep 06 2024 - 09:21:08 EST
On Fri, Sep 06, 2024 at 12:51:08PM +0300, Stanimir Varbanov wrote:
> >> 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:
>
> 2/2 is in linux-next now.
And 1/2? Can someone pick it up too?
https://lore.kernel.org/all/2o6gpxknio4kvf5fmqgai5jaov65semmw3wvtmt4gdauwzhbti@ii3ydtlmp2sb/
Thanks,
Jason