From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id F3B27C433EF for ; Sat, 4 Dec 2021 08:38:26 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1384331AbhLDIlu (ORCPT ); Sat, 4 Dec 2021 03:41:50 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41462 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1354761AbhLDIlt (ORCPT ); Sat, 4 Dec 2021 03:41:49 -0500 Received: from mail-lf1-x12c.google.com (mail-lf1-x12c.google.com [IPv6:2a00:1450:4864:20::12c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id CC492C061751 for ; Sat, 4 Dec 2021 00:38:23 -0800 (PST) Received: by mail-lf1-x12c.google.com with SMTP id bu18so12455924lfb.0 for ; Sat, 04 Dec 2021 00:38:23 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=message-id:date:mime-version:user-agent:subject:content-language:to :cc:references:from:in-reply-to:content-transfer-encoding; bh=Iq91punDd7cKLt5NHGxJuQc18kZP0dgQtqZmR6jVBZM=; b=B8pVjFT5w2dL+65gFXHPa5Qu8kWqrX2ZEFd4TrDk8lPm4Lx6Q/PI6hHNZ1e+O0a0Xa u7s7GcySQEE1q2XApr3kPJyhz9cpg7JtmCbcWqU2HGYR5icFGLIkzICWChvIrRML8ktK qEX/INs2y5pC6g9BFJ9BLgEL3xNIjHQTYRrcZCmdf4Bk5FlrkkI88VRt3TPdY01SuTcp 1PevbpDttW12Pkx+g+1HqzHQku8nQ+I8hDDpoW43a0BLIMTzjGocM6amOFtwHt72261f /G/S6Qk2el/piag++QoQULoqx0wq2OLZwhhQHxQWe/O3YuSHx8yJtNdASbY39CiUCMmO zr1g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=Iq91punDd7cKLt5NHGxJuQc18kZP0dgQtqZmR6jVBZM=; b=Su2TJdHPcGlllaJT1ZYnNcoKY8J7mEUr04aMsXG2Mk02xeg+76M8OGQBmmnuuPPQ5p hOE4XqvSS+hEcMthQHq0vlfNpERaK30CTLy7fst7kxGLQe/475aGWp4Eqt2xLX2nBrxg aQDa5lCt8zr9V1jq/JSi7yk+oR87WQAuPlduYRXZGDMEI5VAlVFpyUyCdU1dlu5WWJhw bPFnKJbMcVVY5f0/19UAF5O/nDks65oX2af70ZmCQDDURpy/5cFG+KpXMUC+Zpcv9Zbh WJ9wisIxc/N3wJXLmNIkbZKHruJKV6ZUcnp4Gi6UbBuLFUzbc3OK+FHZOhlLvaePp0je lruA== X-Gm-Message-State: AOAM5300NaIvshN4HzO0XhYm2p1Pp9B1wN8BBWYgtWoVzUhZ9tzN2VHL 0bl65sZhJukAtlwtdHCe1+E= X-Google-Smtp-Source: ABdhPJwyVgZ536/k6VjtKdFHBUUYhqOvFzG5ibN+LIc3/B3tmdsyDtGMtZBrYlNTHEMDnSD1LmwSQw== X-Received: by 2002:a05:6512:519:: with SMTP id o25mr22179619lfb.422.1638607101886; Sat, 04 Dec 2021 00:38:21 -0800 (PST) Received: from [192.168.1.11] ([94.103.229.236]) by smtp.gmail.com with ESMTPSA id s4sm745615ljp.73.2021.12.04.00.38.20 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 04 Dec 2021 00:38:21 -0800 (PST) Message-ID: <19041795-e692-dd19-c14f-cd951bc7a3de@gmail.com> Date: Sat, 4 Dec 2021 11:38:16 +0300 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.3.2 Subject: Re: [PATCH v3] staging: vt6655: refactor byRadioCtl to radio_ctl Content-Language: en-US To: Alberto Merciai Cc: Forest Bond , Greg Kroah-Hartman , Tommaso Merciai , =?UTF-8?B?QWxkYXMgVGFyYcWha2V2acSNaXVz?= , Dan Carpenter , Karolina Drobnik , Lucas Henneman , linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org References: <20211203202410.3807147-1-alb3rt0.m3rciai@gmail.com> From: Pavel Skripkin In-Reply-To: <20211203202410.3807147-1-alb3rt0.m3rciai@gmail.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 12/3/21 23:24, Alberto Merciai wrote: > Replace camelcase hungarian notated variable "byRadioCtl" > into linux kernel coding style equivalent variable "radio_ctl". > > Signed-off-by: Alberto Merciai > --- > > v2 -> v3 > - replace typo inside mail message body > v1 -> v2 > - Correct mail subject > > drivers/staging/vt6655/device.h | 2 +- > drivers/staging/vt6655/device_main.c | 8 ++++---- > 2 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/staging/vt6655/device.h b/drivers/staging/vt6655/device.h > index c51eba94230d..245f992e5a23 100644 > --- a/drivers/staging/vt6655/device.h > +++ b/drivers/staging/vt6655/device.h > @@ -227,7 +227,7 @@ struct vnt_private { > bool bPWBitOn; > > /* GPIO Radio Control */ > - unsigned char byRadioCtl; > + unsigned char radio_ctl; > unsigned char byGPIO; > bool hw_radio_off; > bool bPrvActive4RadioOFF; > diff --git a/drivers/staging/vt6655/device_main.c b/drivers/staging/vt6655/device_main.c > index 897d70cf32b8..37a82550ca82 100644 > --- a/drivers/staging/vt6655/device_main.c > +++ b/drivers/staging/vt6655/device_main.c > @@ -371,18 +371,18 @@ static void device_init_registers(struct vnt_private *priv) > > priv->radio_off = false; > > - priv->byRadioCtl = SROMbyReadEmbedded(priv->port_offset, > + priv->radio_ctl = SROMbyReadEmbedded(priv->port_offset, > EEP_OFS_RADIOCTL); > priv->hw_radio_off = false; > > - if (priv->byRadioCtl & EEP_RADIOCTL_ENABLE) { > + if (priv->radio_ctl & EEP_RADIOCTL_ENABLE) { > /* Get GPIO */ > MACvGPIOIn(priv->port_offset, &priv->byGPIO); > > if (((priv->byGPIO & GPIO0_DATA) && > - !(priv->byRadioCtl & EEP_RADIOCTL_INV)) || > + !(priv->radio_ctl & EEP_RADIOCTL_INV)) || > (!(priv->byGPIO & GPIO0_DATA) && > - (priv->byRadioCtl & EEP_RADIOCTL_INV))) > + (priv->radio_ctl & EEP_RADIOCTL_INV))) > priv->hw_radio_off = true; > } > > This one is odd. Code uses byRadioCtl as local storage to check register value within one function... I might be missing something, but I think, it can be removed as well. With regards, Pavel Skripkin