[PATCH linux dev-4.13 09/16] hwmon (p9_sbe): Rename lock member of struct p9_sbe_occ
Eddie James
eajames at linux.vnet.ibm.com
Fri Feb 23 07:31:15 AEDT 2018
On 02/19/2018 10:18 PM, Andrew Jeffery wrote:
> Improve code clarity (and searchability) by renaming the lock to client_lock.
> The end result is it is easier to search for uses in the source file.
Acked-by: Eddie James <eajames at linux.vnet.ibm.com>
>
> Signed-off-by: Andrew Jeffery <andrew at aj.id.au>
> ---
> drivers/hwmon/occ/p9_sbe.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/hwmon/occ/p9_sbe.c b/drivers/hwmon/occ/p9_sbe.c
> index 9b8f2f650895..51bdfa89f1a6 100644
> --- a/drivers/hwmon/occ/p9_sbe.c
> +++ b/drivers/hwmon/occ/p9_sbe.c
> @@ -34,7 +34,7 @@ struct p9_sbe_occ {
> * open, close and NULL assignment. This prevents simultaneous opening
> * and closing of the client, or closing multiple times.
> */
> - spinlock_t lock;
> + spinlock_t client_lock;
> };
>
> #define to_p9_sbe_occ(x) container_of((x), struct p9_sbe_occ, occ)
> @@ -44,11 +44,11 @@ static void p9_sbe_occ_close_client(struct p9_sbe_occ *ctx)
> unsigned long flags;
> struct occ_client *tmp_client;
>
> - spin_lock_irqsave(&ctx->lock, flags);
> + spin_lock_irqsave(&ctx->client_lock, flags);
> tmp_client = ctx->client;
> ctx->client = NULL;
> occ_drv_release(tmp_client);
> - spin_unlock_irqrestore(&ctx->lock, flags);
> + spin_unlock_irqrestore(&ctx->client_lock, flags);
> }
>
> static int p9_sbe_occ_send_cmd(struct occ *occ, u8 *cmd)
> @@ -58,10 +58,10 @@ static int p9_sbe_occ_send_cmd(struct occ *occ, u8 *cmd)
> struct occ_response *resp = &occ->resp;
> struct p9_sbe_occ *ctx = to_p9_sbe_occ(occ);
>
> - spin_lock_irqsave(&ctx->lock, flags);
> + spin_lock_irqsave(&ctx->client_lock, flags);
> if (ctx->sbe)
> ctx->client = occ_drv_open(ctx->sbe, 0);
> - spin_unlock_irqrestore(&ctx->lock, flags);
> + spin_unlock_irqrestore(&ctx->client_lock, flags);
>
> if (!ctx->client)
> return -ENODEV;
More information about the openbmc
mailing list