Message ID | 96027a74236514f38f54051d24a93b6d692644ac.1695972930.git.federico.serafini@bugseng.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | Address violations of MISRA C:2012 Rule 8.3 | expand |
On 29/09/23 10:56, Federico Serafini wrote: > Drop bool_t to make function declaration and definition consistent and address > a violation of MISRA C:2012 Rule 8.3. > No functional change. > > Signed-off-by: Federico Serafini <federico.serafini@bugseng.com> > --- > xen/drivers/video/vesa.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/xen/drivers/video/vesa.c b/xen/drivers/video/vesa.c > index b007ff5678..70feca21ac 100644 > --- a/xen/drivers/video/vesa.c > +++ b/xen/drivers/video/vesa.c > @@ -145,7 +145,7 @@ static void cf_check lfb_flush(void) > __asm__ __volatile__ ("sfence" : : : "memory"); > } > > -void __init vesa_endboot(bool_t keep) > +void __init vesa_endboot(bool keep) > { > if ( keep ) > { I am seeing that there is an acked patch that solves this, so this patch should be dropped.
diff --git a/xen/drivers/video/vesa.c b/xen/drivers/video/vesa.c index b007ff5678..70feca21ac 100644 --- a/xen/drivers/video/vesa.c +++ b/xen/drivers/video/vesa.c @@ -145,7 +145,7 @@ static void cf_check lfb_flush(void) __asm__ __volatile__ ("sfence" : : : "memory"); } -void __init vesa_endboot(bool_t keep) +void __init vesa_endboot(bool keep) { if ( keep ) {
Drop bool_t to make function declaration and definition consistent and address a violation of MISRA C:2012 Rule 8.3. No functional change. Signed-off-by: Federico Serafini <federico.serafini@bugseng.com> --- xen/drivers/video/vesa.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)