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 X-Spam-Level: X-Spam-Status: No, score=-12.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 40214C433ED for ; Wed, 7 Apr 2021 08:47:08 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 0625761154 for ; Wed, 7 Apr 2021 08:47:07 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230376AbhDGIrN (ORCPT ); Wed, 7 Apr 2021 04:47:13 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56514 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1349651AbhDGIqZ (ORCPT ); Wed, 7 Apr 2021 04:46:25 -0400 Received: from mail-pj1-x102f.google.com (mail-pj1-x102f.google.com [IPv6:2607:f8b0:4864:20::102f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BD55EC061762; Wed, 7 Apr 2021 01:45:34 -0700 (PDT) Received: by mail-pj1-x102f.google.com with SMTP id mj7-20020a17090b3687b029014d162a65b6so895316pjb.2; Wed, 07 Apr 2021 01:45:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=V6KnqIpI1G1nZtIsittyDe7aXh9sZirwgC5BBRWs6BI=; b=nofHWIoczBDzr2I2/6p4FLM7vs8zuNxFMAduuI62uwcX0hnSa23wJkLbMMBSL1bwCM s/ewRAtGh2PkYL9gF9HVKu69PQtgFwPdLeCMv3iStY5L1pgQA0eTuqEq0arvnr49WkLZ gBSSqO+fkb7OOUdMnAVf0yFXtCe3dON8fAOZpC5/ObEg5Dcf5tNBTvLbIkMSCuCnefQM wxU3Eeg0LmHK3JYtU7ZC5UjOY69cL4F3hcbNXvcT3I2Bp0xawFV+UqytvFbPTm+eQgBL jiZdTM0AdOm7JC5JLOWFnozVL9vxljiVcd/HJ/Gkotw0l45hRDiHEwXFu6+dHPnnzCBI IpoA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=V6KnqIpI1G1nZtIsittyDe7aXh9sZirwgC5BBRWs6BI=; b=KgUz2jwXi3fk/GUMezWxR6ZTr1qFMkwH73Kj96kfaXrBrsqwn0skA5QOdHW7A/ICMC GeyaRwczLnrIKDzUIuH2jNRfmMepnzm18rpLOuHhbrtJxliPzyNIBKLa5woYe3vuRx1x QcjnlBKX0NPIwkZdoKjHcS/sPyIx+Yd0/WLBH+TtRFYLXygHpX68t/fRntknbj8PeQZ7 8VcbZPtS3hqYUrRAyvd/Gbhp04VRWpZl6ooS1LxUMVM+fPfahaAFfQ+mREOM+VpAARSB AEo1coDEVwbMo5UhNFBxVgkMPp+Yb2VvA8asbA531ya6ESI3r/Aigfc93ZhCCaKU/iHw vRcg== X-Gm-Message-State: AOAM531WvSEJNFyAv3Th7d7v8tsAwML1CR7bcpvDHLlrYYX+N/eT/0He f4gz56UDF2mg6MT16dFK/pw= X-Google-Smtp-Source: ABdhPJyGOYtOzCI4lGrq08zi0hBR743xVLhdTGGXcJkG5P4gsRpAJ7nDkj/ZXYX0bchExUrub+OFNQ== X-Received: by 2002:a17:90a:6089:: with SMTP id z9mr2266360pji.211.1617785134171; Wed, 07 Apr 2021 01:45:34 -0700 (PDT) Received: from localhost.localdomain ([134.173.248.5]) by smtp.gmail.com with ESMTPSA id y3sm3943562pfg.145.2021.04.07.01.45.33 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 07 Apr 2021 01:45:33 -0700 (PDT) Date: Wed, 7 Apr 2021 01:45:31 -0700 From: Pavle Rohalj To: Greg KH Cc: sudipm.mukherjee@gmail.com, teddy.wang@siliconmotion.com, linux-fbdev@vger.kernel.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 01/49] staging: sm750fb: Update dvi_ctrl_device to snake case Message-ID: References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-fbdev@vger.kernel.org On Wed, Apr 07, 2021 at 10:31:21AM +0200, Greg KH wrote: > On Tue, Apr 06, 2021 at 11:35:56PM -0700, Pavle Rohalj wrote: > > Fix "Avoid CamelCase" checkpatch.pl checks for dvi_ctrl_device structure and > > its usages. > > > > Signed-off-by: Pavle Rohalj > > --- > > drivers/staging/sm750fb/ddk750_dvi.c | 30 ++++++++-------- > > drivers/staging/sm750fb/ddk750_dvi.h | 20 +++++------ > > drivers/staging/sm750fb/ddk750_sii164.c | 48 ++++++++++++------------- > > drivers/staging/sm750fb/ddk750_sii164.h | 20 +++++------ > > 4 files changed, 59 insertions(+), 59 deletions(-) > > > > diff --git a/drivers/staging/sm750fb/ddk750_dvi.c b/drivers/staging/sm750fb/ddk750_dvi.c > > index cd564ea40779..db19bf732482 100644 > > --- a/drivers/staging/sm750fb/ddk750_dvi.c > > +++ b/drivers/staging/sm750fb/ddk750_dvi.c > > @@ -11,20 +11,20 @@ > > * function API. Please set the function pointer to NULL whenever the function > > * is not supported. > > */ > > -static struct dvi_ctrl_device g_dcftSupportedDviController[] = { > > +static struct dvi_ctrl_device dcft_supported_dvi_controller[] = { > > Why the "dcft_" prefix? We know this is a "dvi control device" by the > fact that the type says it is :) > Didn't catch that--makes sense. > > #ifdef DVI_CTRL_SII164 > > { > > - .pfnInit = sii164InitChip, > > - .pfnGetVendorId = sii164GetVendorID, > > - .pfnGetDeviceId = sii164GetDeviceID, > > + .pfn_init = sii164_init_chip, > > + .pfn_get_vendor_id = sii164_get_vendor_id, > > + .pfn_get_device_id = sii164_get_device_id, > > #ifdef SII164_FULL_FUNCTIONS > > - .pfnResetChip = sii164ResetChip, > > - .pfnGetChipString = sii164GetChipString, > > - .pfnSetPower = sii164SetPower, > > - .pfnEnableHotPlugDetection = sii164EnableHotPlugDetection, > > - .pfnIsConnected = sii164IsConnected, > > - .pfnCheckInterrupt = sii164CheckInterrupt, > > - .pfnClearInterrupt = sii164ClearInterrupt, > > + .pfn_reset_chip = sii164_reset_chip, > > + .pfn_get_chip_string = sii164_get_chip_string, > > + .pfn_set_power = sii164_set_power, > > + .pfn_enable_hot_plug_detection = sii164_enable_hot_plug_detection, > > + .pfn_is_connected = sii164_is_connected, > > + .pfn_check_interrupt = sii164_check_interrupt, > > + .pfn_clear_interrupt = sii164_clear_interrupt, > > #endif > > }, > > #endif > > @@ -41,11 +41,11 @@ int dviInit(unsigned char edge_select, > > unsigned char pll_filter_enable, > > unsigned char pll_filter_value) > > { > > - struct dvi_ctrl_device *pCurrentDviCtrl; > > + struct dvi_ctrl_device *current_dvi_ctrl; > > > > - pCurrentDviCtrl = g_dcftSupportedDviController; > > - if (pCurrentDviCtrl->pfnInit) { > > - return pCurrentDviCtrl->pfnInit(edge_select, > > + current_dvi_ctrl = dcft_supported_dvi_controller; > > + if (current_dvi_ctrl->pfn_init) { > > + return current_dvi_ctrl->pfn_init(edge_select, > > bus_select, > > dual_edge_clk_select, > > hsync_enable, > > diff --git a/drivers/staging/sm750fb/ddk750_dvi.h b/drivers/staging/sm750fb/ddk750_dvi.h > > index 1c7a565b617a..4ca2591ea94b 100644 > > --- a/drivers/staging/sm750fb/ddk750_dvi.h > > +++ b/drivers/staging/sm750fb/ddk750_dvi.h > > @@ -27,16 +27,16 @@ typedef void (*PFN_DVICTRL_CLEARINTERRUPT)(void); > > > > /* Structure to hold all the function pointer to the DVI Controller. */ > > struct dvi_ctrl_device { > > - PFN_DVICTRL_INIT pfnInit; > > - PFN_DVICTRL_RESETCHIP pfnResetChip; > > - PFN_DVICTRL_GETCHIPSTRING pfnGetChipString; > > - PFN_DVICTRL_GETVENDORID pfnGetVendorId; > > - PFN_DVICTRL_GETDEVICEID pfnGetDeviceId; > > - PFN_DVICTRL_SETPOWER pfnSetPower; > > - PFN_DVICTRL_HOTPLUGDETECTION pfnEnableHotPlugDetection; > > - PFN_DVICTRL_ISCONNECTED pfnIsConnected; > > - PFN_DVICTRL_CHECKINTERRUPT pfnCheckInterrupt; > > - PFN_DVICTRL_CLEARINTERRUPT pfnClearInterrupt; > > + PFN_DVICTRL_INIT pfn_init; > > "pfn_" means "pointer to a function" which is not needed at all. Just > make this be "init". > I changed that in one of the next few patches, but now I realize I should have combined the two changes. > And the whole crazy "PFN_DVICTRL_INIT" also is not needed, just put the > real function prototype in here so that we don't have to unwind the mess > to look it up. > > So, this line would look like: > void (*init)(void); > > Much smaller, more obvious, matches the kernel coding style, and is way > easier to understand exactly what is happening here. > > Typedefs can be used to hide complexity, but here they are just adding > it, for no good reason at all. > > I appreciate long patch series being sent out, but maybe make them > smaller so you do not have to redo 49 patches because you are asked to > make a change on the very first patch like here. Perhaps stick to 20 > max for a bit until you get the process down and understand more about > what the kernel programming style is? > > thanks, > > greg k-h Sounds good. Thank you for the feedback. I will work more on this. -Pavle