Messages in this thread |  | | From | Alex Deucher <> | Date | Fri, 27 Mar 2020 10:16:01 -0400 | Subject | Re: [PATCH] drm/prime: fix extracting of the DMA addresses from a scatterlist |
| |
On Fri, Mar 27, 2020 at 6:25 AM Shane Francis <bigbeeshane@gmail.com> wrote: > > Hello Marek, > > On Fri, Mar 27, 2020 at 9:00 AM Marek Szyprowski > <m.szyprowski@samsung.com> wrote: > > > I have tested the above patch against my original issues with amdgpu > > > and radeon drivers and everything is still working as expected. > > > > > > Sorry I missed this in my original patches. > > > > No problem. Thanks for testing! > > > > Best regards > > -- > > Marek Szyprowski, PhD > > Samsung R&D Institute Poland > > > Just a thought. > > Would it be worth adding some comments to the code to explain why this > is needed, reading > the thread around my original patches and the DMA-API documentation it > is not instantly > clear why you would be mapping the pages in this way. > > Would probably prevent someone in the future making the same mistake I > did while updating > this code.
With a comment similar to the commit messaged added to this function, this patch is: Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Thanks!
Alex
|  |