On Tue, Aug 26, 2025 at 04:20:54PM -0700, Randy Dunlap wrote:
> > +
> > +/**
> > + * map_range() - Install translation for an IOVA range
> 
> Maybe I don't understand the macros (haven't looked lately).
> 
> Function name appears to be map_pages(), not map_range().

There is a later series that changes this around and it didn't get
split up right

> > +    * @iommu_table: Table to operate on
> > +    * @top_paddr: New CPU physical address of the top pointer
> > +    * @top_level: IOMMU PT level of the new top
> 
> We don't really have a way to document parameters of a callback function
> inside a struct, but for now kernel-doc.py won't complain about it.
> (Somehow kernel-doc.pl did once upon a time and then that became dead code.)

Seems like we should :)

Fixed the rest, thanks

Jason

Reply via email to