Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) client-ip=23.128.96.33;
Message-ID: <d77f35a3-5eec-d02f-459b-1968a58ff7d5@intel.com>
Date:   Tue, 17 Oct 2023 11:07:49 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.13.0
Subject: Re: [PATCH 0/4] Rename variable bUpdateBBVGA, byCurrentCh, byCurPwr
 and byBBPreEDRSSI
Content-Language: en-US
To:     Gilbert Adikankwu <gilbertadikankwu@gmail.com>
CC:     Philipp Hortmann <philipp.g.hortmann@gmail.com>,
        <outreachy@lists.linux.dev>, <forest@alittletooquiet.net>,
        <gregkh@linuxfoundation.org>, <linux-staging@lists.linux.dev>,
        <linux-kernel@vger.kernel.org>
References: <cover.1697495597.git.gilbertadikankwu@gmail.com>
 <0f6661a4-7281-4d26-8c26-ba03a567fcaa@gmail.com>
 <ZS41jFfwBJ672Rb7@gilbert-PC>
From:   Karolina Stolarek <karolina.stolarek@intel.com>
Organization: Intel Technology Poland sp. z o.o. - ul. Slowackiego 173, 80-298
 Gdansk - KRS 101882 - NIP 957-07-52-316
In-Reply-To: <ZS41jFfwBJ672Rb7@gilbert-PC>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?SERtc3lDREU2OXZkUnVva2QwNWpBanhKSnV4Qm1wZ3BMdUx2TUM2TUNPN0FK?=
 =?utf-8?B?Ni9XODJvZzJCNU1mV1htSjJBc2Y4VS9UU3R4Mkd4endpZ1UwNVFBbUFEa2NV?=
 =?utf-8?B?VlJoTk9ZamkyL01yVDcvemtCamFZSU9PYXZLUm5ETUZISWlqNFF6S3RPOW9Q?=
 =?utf-8?B?aFExZzhIVlpiSHdVZm5heUpoNU8reEk0YVZ1WWVBeVkzU1Rnby9BY3ZDZzAx?=
 =?utf-8?B?enBaRUdMUlU0TDgrU1NydVNxTkNvRFgyQTR4Q2JnR1MzeU1ZemVhKzg5YlpL?=
 =?utf-8?B?OE5NUzY5SGpFbjdsQVpiTDY2L2U5K3N4MkJHRGNQZkt0UTJueTRGL0pGQXhY?=
 =?utf-8?B?VXRpWVZNVGZtQ055Vm84WnUwdGhsRWhYSzRkc01tVlVkS3d6SmdIVGJGckFU?=
 =?utf-8?B?SStFVDkxeENFaGlWaU9vV0NuMU5SUkVHcVNSRDhLWlZZQlV5b1FhR3JITERW?=
 =?utf-8?B?MGN3MkdudjBQWDFBZHpjK0pWdmRzZlFLcmRxVXRINnFWNk9uYThvTm9BdkNQ?=
 =?utf-8?B?VDAxYVNxZzZML0Rjc2h6QVRzenh2TlRUNlBGenBGTWUwTEhTTm50ZU9FUmRw?=
 =?utf-8?B?aG9RN1NWT09nL0R2ditYZkV1Qm9tKytEVGRGQ3NWMVNEN2NtbWVpS3dzVWc4?=
 =?utf-8?B?WmpxV0lPbjFIZlZ5WUhnQUllaHYxSEpXTDllUzMzRXA4Q1JKekt6RGRmeWN2?=
 =?utf-8?B?UmNhNW1uOHFKU2NVeVM5dStVL2YrN2NrOGV3T09uaDJTTmhydFhhSzJRcEQ3?=
 =?utf-8?B?dEFsS3duK05EYTArVm5ubU5iYldIOVVCZHVvQVZjd0xmQitpbUlEL2hhaHoz?=
 =?utf-8?B?S09qbktXRHBsY2EwL2k2MGJsdjhqWnQ2UFJWcEljNGpqbFNsVmdTc2ppRzRj?=
 =?utf-8?B?T2dCM25Hd2tENmgvZEdhYzB2OVNjQ2lRaTV0aTk3S2NNdXZ5UzEraGZYOG1Z?=
 =?utf-8?B?bGxhS0xSYUxjdlUwN0pRcTFnZ3IwelVIMDh6a0NHOUEyeGs0V0Z2ZkJsdjlp?=
 =?utf-8?B?OXpPT25jUEFPeU0rczg1ekNCcGtRTFo2Zm5YK05uUFZneitzeXFkU0hidGZR?=
 =?utf-8?B?RnJNS1lnMEZmR3NZTnRjcm8rak5jay9wdTlmTTQ5VWRFbzRyTk4yclRlWEhl?=
 =?utf-8?B?NEpuclN0Y3FuSGxGOEFtVUpUaUhjTnA3bXRBNm5DelRYcVRZYitGTjNVQU5Y?=
 =?utf-8?B?T2hYNUZnSjRCeG9odmMyeW1rSmk2NUZHeWxodUg5cVp0MFJ3VlV5VjlqNlAz?=
 =?utf-8?B?dy9wYXFDM0hodHNKTkJ4WWlMMEFFd1lQTUQ2L2dNSGVFdGxoU1lFNUFPaWpY?=
 =?utf-8?B?NWhnUndTWEgzSTAzVk5YWHJTeGtVakVEZlBNa1hZc3J6U0NhR0VPNVh3b01N?=
 =?utf-8?B?Y0ZmMmRHTGl2bzNnZzYxTFZscWVuTXRFL0I0dklhZEVGRU9IRG9HN09DZ21v?=
 =?utf-8?B?R1FKRkdpNlJJZFJtdnVCcjhtZFRsQ0JDWUJRVTNjckY3UnIwL0ppVEpBWVJi?=
 =?utf-8?B?MnR0NDJySk5CWlR1SGo2TjNNNXl1TnlUMStwN3NFcFUycGxvV200WmR6em10?=
 =?utf-8?B?U0lmdGxlOVVjRldBNVc5V2pCemtsK1huWE43R1hqL1VBZi9jS2tyZlhFdTYv?=
 =?utf-8?B?UGdpZUZhK3hTWXNJZW9LK2ZJaHJsaHA5ek1PYjZXb2ovaEVGRWlXQktnZlNI?=
 =?utf-8?B?NWZrMFJINVVsVmQwS1crMFhKVW1VRVprVkMzai9BTW43ekk3VjlkeklmOHRj?=
 =?utf-8?B?N25oMzRNZ0krdGZqUTdFYll3MTB2bCtVT2FoN3ZoS2ZvenFobVFJVndxZ280?=
 =?utf-8?B?MUEycWFmaXNURFhsWEtTdjJTWDkvU3VHQXpDUEVtZ0w2VEdubTBqNFEvTEFr?=
 =?utf-8?B?N3UzNHM4WTdMWHNscmZyM2ltWFBYVlNQRGxDRCtoYm9EWUhsZ0VUM0xBeUNS?=
 =?utf-8?B?YzgrOG5OT2JqblpsdzVDQURSNWFXaXk2cWxPUWNpTUI5U2tnVHFxYW56bk1r?=
 =?utf-8?B?U1NmWUVIZU55OXhEUzlRZFdkbDRpSGN1MElUMU1pZm9kc0hmNkw1SUI4ZlZx?=
 =?utf-8?B?TTQ1bTBwS0tJMjJESEJSdlovdWhsMFlTektCQ2V2MEtCMXppRWo5T1NpOEU2?=
 =?utf-8?B?WnhQd2R5a3dqNlBqOE02bFYzZGoyeUdFN3ZvdU9jYjZNWWpwZmdha2dDRXdj?=
 =?utf-8?B?S2c9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 8768e11c-2730-4231-f4ff-08dbcef0871a
X-MS-Exchange-CrossTenant-AuthSource: BY5PR11MB4243.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Oct 2023 09:07:46.1004
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 1arWOu1m/QyLm1VhQWsV0l2FKuuftyaWDegIAasJjeGf3HlbiLetxLl+HayJ6waBMW/UxC5rVrJUUhic9nGHkTTpBo13Lyjwu597QUz1mac=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR11MB6237
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.2 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS
	autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lipwig.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (lipwig.vger.email [0.0.0.0]); Tue, 17 Oct 2023 02:08:01 -0700 (PDT)

On 17.10.2023 09:19, Gilbert Adikankwu wrote:
> On Tue, Oct 17, 2023 at 08:19:49AM +0200, Philipp Hortmann wrote:
>> On 10/17/23 00:58, Gilbert Adikankwu wrote:
>>> Make the variable names adhere to Linux kernel coding style and mute
>>> checkpatch.pl errors.

A small nit from my side -- "mute" suggests that you suppressed the 
warning/error, but your changes fix them, so why now say "fix 
checkpatch.pl errors"?

>>>
>>> Gilbert Adikankwu (4):
>>>     staging: vt6655: Rename variable bUpdateBBVGA
>>>     staging: vt6655: Rename variable byCurrentCh
>>>     staging: vt6655: Rename variable byCurPwr
>>>     staging: vt6655: Rename variable byBBPreEDRSSI
>>>
>>>    drivers/staging/vt6655/baseband.c    |  2 +-
>>>    drivers/staging/vt6655/channel.c     | 18 +++++++++---------
>>>    drivers/staging/vt6655/device.h      |  8 ++++----
>>>    drivers/staging/vt6655/device_main.c |  8 ++++----
>>>    drivers/staging/vt6655/dpc.c         |  2 +-
>>>    drivers/staging/vt6655/rf.c          |  4 ++--
>>>    6 files changed, 21 insertions(+), 21 deletions(-)
>>>
>>
>> Hi,
>>
>> -       if (priv->byCurrentCh == ch->hw_value)
>> +       if (priv->current_ch == ch->hw_value)^M
>>                  return ret;
>>
>> ERROR: DOS line endings
>>
>> What does the "^M" at the end of the new line?
>> Did you run checkpatch on your patches?
>>
>> Thanks
>>
>> Bye
>>
>> Philipp
> 
> Hello,
> 
> I do not know what this error means. If anything, I'm surprised there is
> an error. checkpatch is hooked to my git commit and all the patches
> built cleanly on my machine. I will resend now.

I'm not seeing that error when the patch is applied. You could 
double-check the patch files before sending by calling checkpatch.pl on 
them directly.

This could be also because of how some email clients change the endings 
on download, like Thunderbird, and the patch itself might be fine.

All the best,
Karolina

> 
> Thanks
> Gilbert
>