On 2026-10-21 04:46, Andy Shevchenko wrote:
ctype.h provides tolower() macro specifically for the cases
that might be architecture-optimised. Using __tolower(),
which is internal implementation, needs a good justification,
as this driver is the only user of that and there is no evidence
it's required.
Note, currently it's no difference as the macro always defined
to the internal implementation, but it might change in the future.
Signed-off-by: Andy Shevchenko <[email protected]>
Thanks. I'm applying the patch to amd-staging-drm-next.
Reviewed-by: Felix Kuehling <[email protected]>
---
drivers/gpu/drm/amd/amdkfd/kfd_topology.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_topology.c
b/drivers/gpu/drm/amd/amdkfd/kfd_topology.c
index 35b3abe57b80..1ef4d26a0ac6 100644
--- a/drivers/gpu/drm/amd/amdkfd/kfd_topology.c
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_topology.c
@@ -2091,7 +2091,7 @@ int kfd_topology_add_device(struct kfd_node *gpu)
* needed for the topology
*/
for (i = 0; i < KFD_TOPOLOGY_PUBLIC_NAME_SIZE-1; i++) {
- dev->node_props.name[i] = __tolower(asic_name[i]);
+ dev->node_props.name[i] = tolower(asic_name[i]);
if (asic_name[i] == '\0')
break;
}
