[PATCH] swiotlb: Update the comment of swiotlb

[PATCH] swiotlb: Update the comment of swiotlb

[Yisheng Xie]

swiotlb use physical address of bounce buffer when do map and unmap,
therefore, related comment should be updated.

Signed-off-by: Yisheng Xie <xieyisheng1@huawei.com>
---
 lib/swiotlb.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/lib/swiotlb.c b/lib/swiotlb.c
index fece575..e3d9445 100644
--- a/lib/swiotlb.c
+++ b/lib/swiotlb.c
@@ -593,9 +593,8 @@ phys_addr_t swiotlb_tbl_map_single(struct device *hwdev,
 }
 
 /*
- * Allocates bounce buffer and returns its kernel virtual address.
+ * Allocates bounce buffer and returns its physical address.
  */
-
 static phys_addr_t
 map_single(struct device *hwdev, phys_addr_t phys, size_t size,
 	   enum dma_data_direction dir, unsigned long attrs)
@@ -614,7 +613,7 @@ phys_addr_t swiotlb_tbl_map_single(struct device *hwdev,
 }
 
 /*
- * dma_addr is the kernel virtual address of the bounce buffer to unmap.
+ * tlb_addr is the physical address of the bounce buffer to unmap.
  */
 void swiotlb_tbl_unmap_single(struct device *hwdev, phys_addr_t tlb_addr,
 			      size_t size, enum dma_data_direction dir,
-- 
1.7.12.4

Re: [PATCH] swiotlb: Update the comment of swiotlb

[Christoph Hellwig]

Thanks,

applied to the dma-mapping tree.

Re: [PATCH] swiotlb: Update the comment of swiotlb

[Yisheng Xie]

Hi Christoph,

On 2018/5/9 15:38, Christoph Hellwig wrote:
> Thanks,
> 
> applied to the dma-mapping tree.

Thanks

BTW, should lib/swiotlb.c also add to DMA MAPPING HELPERS, or
add yourself as a maintainer of SWIOTLB SUBSYSTEM ? It will make
get_maintainer.pl get you :)

Thanks
Yisheng
> 
> .
> 

Re: [PATCH] swiotlb: Update the comment of swiotlb

[Christoph Hellwig]

On Wed, May 09, 2018 at 05:08:30PM +0800, Yisheng Xie wrote:
> Hi Christoph,
> 
> On 2018/5/9 15:38, Christoph Hellwig wrote:
> > Thanks,
> > 
> > applied to the dma-mapping tree.
> 
> Thanks
> 
> BTW, should lib/swiotlb.c also add to DMA MAPPING HELPERS, or
> add yourself as a maintainer of SWIOTLB SUBSYSTEM ? It will make
> get_maintainer.pl get you :)

Konrad is the maintainer, but due to me touching the file a lot
lately I keep merging the patches for now.  But anything non-trivial
absolutely requires an Ack from Konrad.

Re: [PATCH] swiotlb: Update the comment of swiotlb

[Konrad Rzeszutek Wilk]

On Wed, May 09, 2018 at 01:39:57PM +0200, Christoph Hellwig wrote:
> On Wed, May 09, 2018 at 05:08:30PM +0800, Yisheng Xie wrote:
> > Hi Christoph,
> > 
> > On 2018/5/9 15:38, Christoph Hellwig wrote:
> > > Thanks,
> > > 
> > > applied to the dma-mapping tree.
> > 
> > Thanks
> > 
> > BTW, should lib/swiotlb.c also add to DMA MAPPING HELPERS, or
> > add yourself as a maintainer of SWIOTLB SUBSYSTEM ? It will make
> > get_maintainer.pl get you :)
> 
> Konrad is the maintainer, but due to me touching the file a lot
> lately I keep merging the patches for now.  But anything non-trivial
> absolutely requires an Ack from Konrad.

<nods>

Thank you for pulling this in your tree.