[PATCH V2 00/10] AMD XDNA driver
Lizhi Hou
lizhi.hou at amd.com
Tue Aug 6 17:18:31 UTC 2024
On 8/6/24 01:05, Markus Elfring wrote:
> …
>> Changes since v1:
>> - Remove some inline defines
>> - Minor changes based code review comments
> …
>
> How “good” does such a version description fit to previous
> patch review feedback (like the following)?
>
> https://lkml.org/lkml/2024/7/19/803
> https://lore.kernel.org/linux-kernel/010a46ba-9dc4-e3e3-7894-b28b312c6ab1@amd.com/
> [01/10] accel/amdxdna: Add a new driver for AMD AI Engine
> “guard looks cleaner. We will use it.”
We reconsidered this request and searched accel and drm subsystem. I did
not see it was used. This does not look like a required change for
upstream at this moment. We would keep the current code for this patch
series.
>
>
> Can further adjustment suggestions be taken better into account?
I went through all your comments. I made the required changes which are
defined in coding style document. And thanks for pointing those out.
Please understand we would only make required change at this moment. For
the nice to have changes, we may consider to change in the future patches.
Thanks,
Lizhi
>
> Regards,
> Markus
More information about the dri-devel
mailing list