From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D07E53FCC for ; Mon, 27 Sep 2021 15:26:16 +0000 (UTC) Received: by mail.kernel.org (Postfix) with ESMTPSA id 126BE60F91; Mon, 27 Sep 2021 15:26:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1632756376; bh=zOTYtImCLca9S+yictRsMbXs3evr7x+36sm1zHHoQCE=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=hyUKchGE9/bcB887Zwbjn09ge/SwiLmbQZ3VneTPLZBU24HUm7+UmW6L55NuWHrZQ 8VnVBf/Z5rqyOS8ccsmVkkXyYlTZaaRLc95qqZnOiNrErwf7RqfUwUcwbj1784KfJl FLiSqS0s9PVmKvmt89sN3yGlNmkzeWUj9uzjwnvg= Date: Mon, 27 Sep 2021 17:26:14 +0200 From: Greg Kroah-Hartman To: Tommaso Merciai Cc: Forest Bond , Lucas Henneman , Yujia Qiao , Madhumitha Prabakaran , Marcos Antonio de Jesus Filho , Aldas =?utf-8?B?VGFyYcWha2V2acSNaXVz?= , "Gustavo A. R. Silva" , Deepak R Varma , linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH v5 1/3] staging: vt6655: fix camelcase in bShortSlotTime Message-ID: References: <20210926162527.21462-1-tomm.merciai@gmail.com> <20210926162527.21462-2-tomm.merciai@gmail.com> Precedence: bulk X-Mailing-List: linux-staging@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210926162527.21462-2-tomm.merciai@gmail.com> On Sun, Sep 26, 2021 at 06:25:18PM +0200, Tommaso Merciai wrote: > Replace camel case variable bShortSlotTime with snake case > variable b_short_slot_time. That is a very odd variable name. Why did you pick that one? What deos it mean? You do understand where the original naming format came from here, and what it was trying to represent, right? If not, please go read up on it: https://en.wikipedia.org/wiki/Hungarian_notation That style is not used in Linux, and so, when converting from it, do not attempt to keep the same style present here, that is kind of pointless, don't you agree? thanks, greg k-h