On 27.06.2024 02:57, Stefano Stabellini wrote: > On Wed, 26 Jun 2024, Nicola Vetrini wrote: >> From: Alessandro Zucchelli <[email protected]> >> >> This addresses violations of MISRA C:2012 Rule 5.3 which states as >> following: An identifier declared in an inner scope shall not hide an >> identifier declared in an outer scope. >> >> In this case the gloabl variable being shadowed is the global static struct >> mctctl in this file, therefore the local variables are renamed to avoid this.
"global" and "static" contradict one another; I think you mean "file scope". While there (nit) also s/gloabl/global/ or perhaps even s/gloabl// to deal with the conflict with "static" also there (without writing "file scope" twice). >> No functional change. >> >> Signed-off-by: Alessandro Zucchelli <[email protected]> >> Signed-off-by: Nicola Vetrini <[email protected]> > > Nice one! > > Reviewed-by: Stefano Stabellini <[email protected]> With the adjustment: Acked-by: Jan Beulich <[email protected]> Jan
