| Message ID | 20240130221808.104878-1-andrew.cooper3@citrix.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | xen: Drop superfluous semi-colons | expand |
On 30.01.2024 23:18, Andrew Cooper wrote: > All these cases happen to be benign, but drop them anyway. This is one step > towards making -Wextra-semi work. > > Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> Reviewed-by: Jan Beulich <jbeulich@suse.com>
diff --git a/xen/common/sched/private.h b/xen/common/sched/private.h index 79e3f56c5a08..26a196f4283b 100644 --- a/xen/common/sched/private.h +++ b/xen/common/sched/private.h @@ -518,7 +518,7 @@ static inline void sched_unit_unpause(const struct sched_unit *unit) } #define REGISTER_SCHEDULER(x) static const struct scheduler *x##_entry \ - __used_section(".data.schedulers") = &x; + __used_section(".data.schedulers") = &x struct cpupool { diff --git a/xen/drivers/acpi/apei/hest.c b/xen/drivers/acpi/apei/hest.c index 4ec28c3c11ba..34b376bc2d74 100644 --- a/xen/drivers/acpi/apei/hest.c +++ b/xen/drivers/acpi/apei/hest.c @@ -86,7 +86,7 @@ static int hest_esrc_len(const struct acpi_hest_header *hest_hdr) BUG_ON(len == -1); return len; -}; +} int apei_hest_parse(apei_hest_func_t func, void *data) { diff --git a/xen/include/xen/livepatch.h b/xen/include/xen/livepatch.h index df339a134e40..45df4bba4f45 100644 --- a/xen/include/xen/livepatch.h +++ b/xen/include/xen/livepatch.h @@ -182,7 +182,7 @@ static inline int livepatch_op(struct xen_sysctl_livepatch_op *op) return -ENOSYS; } -static inline void check_for_livepatch_work(void) { }; +static inline void check_for_livepatch_work(void) {} static inline bool is_patch(const void *addr) { return 0; diff --git a/xen/include/xen/serial.h b/xen/include/xen/serial.h index 870ca2f6eb03..12ef24351d0f 100644 --- a/xen/include/xen/serial.h +++ b/xen/include/xen/serial.h @@ -175,7 +175,7 @@ void ehci_dbgp_init(void); #ifdef CONFIG_XHCI void xhci_dbc_uart_init(void); #else -static void inline xhci_dbc_uart_init(void) {}; +static void inline xhci_dbc_uart_init(void) {} #endif void arm_uart_init(void);
All these cases happen to be benign, but drop them anyway. This is one step towards making -Wextra-semi work. Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> --- CC: George Dunlap <George.Dunlap@citrix.com> CC: Jan Beulich <JBeulich@suse.com> CC: Stefano Stabellini <sstabellini@kernel.org> CC: Wei Liu <wl@xen.org> CC: Julien Grall <julien@xen.org> --- xen/common/sched/private.h | 2 +- xen/drivers/acpi/apei/hest.c | 2 +- xen/include/xen/livepatch.h | 2 +- xen/include/xen/serial.h | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-)