ipmi: kcs_bmc: Miscellaneous cleanups

[PATCH 01/10] ipmi: kcs_bmc: Update module description

Posted by Andrew Jeffery 2 years, 1 month ago

KCS devices are often used for IPMI, but they're not constrained to it.
Update the subsystem module description to reflect its more general
capabilities.

Signed-off-by: Andrew Jeffery <andrew@codeconstruct.com.au>
---
 drivers/char/ipmi/kcs_bmc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/char/ipmi/kcs_bmc.c b/drivers/char/ipmi/kcs_bmc.c
index 8b1161d5194a..a429d9f8a7bf 100644
--- a/drivers/char/ipmi/kcs_bmc.c
+++ b/drivers/char/ipmi/kcs_bmc.c
@@ -187,4 +187,4 @@ EXPORT_SYMBOL(kcs_bmc_update_event_mask);
 MODULE_LICENSE("GPL v2");
 MODULE_AUTHOR("Haiyue Wang <haiyue.wang@linux.intel.com>");
 MODULE_AUTHOR("Andrew Jeffery <andrew@aj.id.au>");
-MODULE_DESCRIPTION("KCS BMC to handle the IPMI request from system software");
+MODULE_DESCRIPTION("Subsystem for BMCs to communicate via KCS devices");
-- 
2.39.2

Re: [PATCH 01/10] ipmi: kcs_bmc: Update module description

Posted by Jonathan Cameron 2 years, 1 month ago

On Fri,  3 Nov 2023 16:45:13 +1030
Andrew Jeffery <andrew@codeconstruct.com.au> wrote:

> KCS devices are often used for IPMI, but they're not constrained to it.
> Update the subsystem module description to reflect its more general
> capabilities.
> 
> Signed-off-by: Andrew Jeffery <andrew@codeconstruct.com.au>
LTGM, FWIW,

Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

> ---
>  drivers/char/ipmi/kcs_bmc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/char/ipmi/kcs_bmc.c b/drivers/char/ipmi/kcs_bmc.c
> index 8b1161d5194a..a429d9f8a7bf 100644
> --- a/drivers/char/ipmi/kcs_bmc.c
> +++ b/drivers/char/ipmi/kcs_bmc.c
> @@ -187,4 +187,4 @@ EXPORT_SYMBOL(kcs_bmc_update_event_mask);
>  MODULE_LICENSE("GPL v2");
>  MODULE_AUTHOR("Haiyue Wang <haiyue.wang@linux.intel.com>");
>  MODULE_AUTHOR("Andrew Jeffery <andrew@aj.id.au>");
> -MODULE_DESCRIPTION("KCS BMC to handle the IPMI request from system software");
> +MODULE_DESCRIPTION("Subsystem for BMCs to communicate via KCS devices");