On Sat Jun 14, 2025 at 2:32 AM CEST, Stefano Stabellini wrote: > On Fri, 13 Jun 2025, Alejandro Vallejo wrote: >> There's the unwritten convention in x86 of splitting type names using >> underscores. Add such convention to the CODINNG_STYLE to make it > ^ CODING_STYLE > >> common and less unwritten. >> >> Signed-off-by: Alejandro Vallejo <agarc...@amd.com> >> --- >> CODING_STYLE | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/CODING_STYLE b/CODING_STYLE >> index 5644f1697f..e91ef7b3ca 100644 >> --- a/CODING_STYLE >> +++ b/CODING_STYLE >> @@ -126,6 +126,9 @@ register). >> Especially with pointer types, whenever the pointed to object is not >> (supposed to be) modified, qualify the pointed to type with "const". >> >> +When defining types, split its words using underscores (e.g: prefer my_foo >> to >> +myfoo). > > Minor NIT: > When defining types, separate words using underscores (for example, use > "my_foo" instead of "myfoo").
Fine by me. That's clearer. > > > Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Thanks! > > >> Comments >> -------- >> >> -- >> 2.43.0 >> Cheers, Alejandro