On Wed, 2021-08-25 at 10:23 +0300, Dan Carpenter wrote: > The code works the same either way, but it's better to use semi- > colons > to separate statements. > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Acked-by: Srinivas Pandruvada <srinivas.pandruvada@xxxxxxxxxxxxxxx> > --- > drivers/platform/x86/intel_speed_select_if/isst_if_common.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git > a/drivers/platform/x86/intel_speed_select_if/isst_if_common.c > b/drivers/platform/x86/intel_speed_select_if/isst_if_common.c > index 8a4d52a9028d..c9a85eb2e860 100644 > --- a/drivers/platform/x86/intel_speed_select_if/isst_if_common.c > +++ b/drivers/platform/x86/intel_speed_select_if/isst_if_common.c > @@ -265,9 +265,9 @@ static int isst_if_get_platform_info(void __user > *argp) > { > struct isst_if_platform_info info; > > - info.api_version = ISST_IF_API_VERSION, > - info.driver_version = ISST_IF_DRIVER_VERSION, > - info.max_cmds_per_ioctl = ISST_IF_CMD_LIMIT, > + info.api_version = ISST_IF_API_VERSION; > + info.driver_version = ISST_IF_DRIVER_VERSION; > + info.max_cmds_per_ioctl = ISST_IF_CMD_LIMIT; > info.mbox_supported = > punit_callbacks[ISST_IF_DEV_MBOX].registered; > info.mmio_supported = > punit_callbacks[ISST_IF_DEV_MMIO].registered; >