On 26.06.2024 11:27, Federico Serafini wrote: > This patch series fixes a missing escape in a deviation and addresses some > violations. > > Federico Serafini (12): > automation/eclair: fix deviation of MISRA C Rule 16.3 > x86/cpuid: use fallthrough pseudo keyword > x86/domctl: address a violation of MISRA C Rule 16.3 > x86/vpmu: address violations of MISRA C Rule 16.3 > x86/traps: address violations of MISRA C Rule 16.3 > x86/mce: address violations of MISRA C Rule 16.3 > x86/hvm: address violations of MISRA C Rule 16.3
Just a remark, no strict request to make further re-arrangements: Looks like what was patch 11 in v2 was now folded into this patch. Yet then why were other HVM parts left separate: > x86/vpt: address a violation of MISRA C Rule 16.3 This and ... > x86/mm: add defensive return > x86/mpparse: address a violation of MISRA C Rule 16.3 > x86/vPIC: address a violation of MISRA C Rule 16.3 > x86/vlapic: address a violation of MISRA C Rule 16.3 ... these two. In general I'd expect splitting / keeping together to be done consistently within a series, unless of course there's something that wants keeping separate for other than purely mechanical reasons. Jan
