Hi,

> -----Original Message-----
> From: Baolu Lu <[email protected]>
> Sent: Wednesday, May 25, 2022 3:27 PM
> To: iwamatsu nobuhiro(岩松 信洋 □SWC◯ACT)
> <[email protected]>; Joerg Roedel <[email protected]>; Will
> Deacon <[email protected]>; Rob Herring <[email protected]>; Jason
> Gunthorpe <[email protected]>
> Cc: [email protected]; [email protected];
> [email protected]; [email protected]; ishikawa
> yuji(石川 悠司 ○RDC□AITC○EA開)
> <[email protected]>; [email protected]
> Subject: Re: [PATCH 1/3] iommu: Add Visconti5 IOMMU driver
> 
> On 2022/5/25 09:31, Nobuhiro Iwamatsu wrote:
> > +static const struct iommu_ops visconti_atu_ops = {
> > +   .domain_alloc = visconti_atu_domain_alloc,
> > +   .probe_device = visconti_atu_probe_device,
> > +   .release_device = visconti_atu_release_device,
> > +   .device_group = generic_device_group,
> > +   .of_xlate = visconti_atu_of_xlate,
> > +   .pgsize_bitmap = ATU_IOMMU_PGSIZE_BITMAP,
> > +   .default_domain_ops = &(const struct iommu_domain_ops) {
> > +           .attach_dev = visconti_atu_attach_device,
> > +           .detach_dev = visconti_atu_detach_device,
> 
> The detach_dev callback is about to be deprecated. The new drivers should
> implement the default domain and blocking domain instead.

I see. I will update this with next version.

> 
> > +           .map = visconti_atu_map,
> > +           .unmap = visconti_atu_unmap,
> > +           .iova_to_phys = visconti_atu_iova_to_phys,
> > +           .free = visconti_atu_domain_free,
> > +   }
> > +};
> 
> Best regards,
> baolu

Best regards,
  Nobuhiro
_______________________________________________
iommu mailing list
[email protected]
https://lists.linuxfoundation.org/mailman/listinfo/iommu

Reply via email to