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@xxxxxxx/
[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
