Date:   Sun, 12 Mar 2023 17:38:17 -0700
From:   Ira Weiny <ira.weiny@intel.com>
To:     kernel test robot <lkp@intel.com>,
        Kloudifold <cloudifold.3125@gmail.com>,
        <teddy.wang@siliconmotion.com>, <sudipm.mukherjee@gmail.com>,
        <gregkh@linuxfoundation.org>, <linux-staging@lists.linux.dev>,
        <outreachy@lists.linux.dev>
CC:     <oe-kbuild-all@lists.linux.dev>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] staging: sm750: Rename camel case functions in
 sm750_cursor.*
Message-ID: <640e7079b2723_229a8929448@iweiny-mobl.notmuch>
References: <ZAuiDTDT8dmBcRoH@CloudiRingWorld>
 <202303110849.X24WnHnM-lkp@intel.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <202303110849.X24WnHnM-lkp@intel.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?737wgA5EsEEw32YpF0fOOV0dtvh1q8uZZoord99i4k/OcmIqr3D7nQMgws3u?=
 =?us-ascii?Q?z5a1YzD5RqNrVUIEp9FU7Z9EjRCwFN6EiWZD1qYWxOJ1Z1tBCJ6TzpwBiuj5?=
 =?us-ascii?Q?JMx8HN20WcXCsi1hQehjfn+lzAB3tPuGaQirpoXk9SQmjzR82XOnmDLNzsn1?=
 =?us-ascii?Q?f6ysEqMaaztviiVX8lOHIBpy9IdK0i88juWZRTpTNczOwgZ4VYSJuSPyVLRS?=
 =?us-ascii?Q?S5J//dXezFGF8M9Co3pD/u7cJjm216eAW0of6Ftzje2VTb+wT0VYzELIXj5v?=
 =?us-ascii?Q?D301uGBiqTxHZ2Xb4UQ79wd5V1z8AckzEXIEZMfkdRolaHlVE6IlEPSMHGzM?=
 =?us-ascii?Q?f5fnMufVAsM8VZQ6RVwh6QRN/vA7aCnJIwNBfxUM2w8eNClLxyxBU54v0vUn?=
 =?us-ascii?Q?CxVPah8OEqpxV41bWvuGe5lBMg/1xIqHfaTJlR3cnUU3sRzb0Q0MaL0xtut3?=
 =?us-ascii?Q?l2CGI5y2w08Nen1rV/Rt6wyKfq8acRZRtmMh0WTLDr8Bzt8v0/66x+Y0/04M?=
 =?us-ascii?Q?NJ20E2rwg9KScKXdK7JoeuHKPc8pccsXKoVm7U+Nim4jWlxUdhSxWU+BAbfH?=
 =?us-ascii?Q?QoYKtH6FAP1SK4iNzhJS7gLBUgbqhddHNW/ReFTrUd/gLAKYPuo5QrH5pOZv?=
 =?us-ascii?Q?lnvfjdRVCTQxLN7/VrJgZPwEsCIH6Mvt3MsFqB7B8o1z00MIeAI+NI/wUInh?=
 =?us-ascii?Q?gJwwGRN1rKGEoP8AwNUEFckrFUn4rVA2dzwWtScZ06DPcQ80++YIzVyVRjlN?=
 =?us-ascii?Q?5NRoHA+eRFMkvT68M9AdHKImwscqSV1VpPRxh5LFf4/+RZ6UokXOd8zdv/5h?=
 =?us-ascii?Q?VHobjfY5kQBWvKBX++abafuDwZkaW/Wircsa2VPQH+N2xCxVOk81cbhJAJOv?=
 =?us-ascii?Q?JW7ad11NQ9ZA2XsjVVUCfWo5SYySQKevv0QBz7Du5fduO6DAnUvHKaqzwxYD?=
 =?us-ascii?Q?vYQncu7bHzoUnwqy8+eeLiq2wfsl4e505o+O5x3GH05USyQUWo11YsVPmEqG?=
 =?us-ascii?Q?oq9sPCdNcVJyd0QlQtrwSfH971z+4Y68mghfdqswRHfjhNmYt4HsVaIgLjYz?=
 =?us-ascii?Q?la7TuSpaiMdgQVZR+zD2aHZPyZnozNy5krznXmU70G022c+BBtxMtHNumJVd?=
 =?us-ascii?Q?vXv/sgoS5Nnh3xIiUrHc2EaKSl4MKF6+gq5u0bXoBPr6zxIzAe9DmxZ+2J95?=
 =?us-ascii?Q?3/zmlAvyppITees9Hq0mmlM9di4O04yU7OG4XW3A5WNVk11Nlos6bfvdWAFI?=
 =?us-ascii?Q?wVMUbBl/JbjsHB2lfIY5egJ3XjAtJ/K7oNOEycOWJg3+5Jojb1+HFR+yMLqg?=
 =?us-ascii?Q?/en0oBEv9Z7IbU0TlTLTZl2UHOfgUUgOkcERvAAj8eRk5y4YxVV3YpvE+DpM?=
 =?us-ascii?Q?hiEvx6yjfoBPZ7IqU/0rtJDp4Pmlc8eWlBtDGRUR5aKyAf8i2CmM8Dlpc9X2?=
 =?us-ascii?Q?MxYt08qcv4Q7Y0kUM+RLtECJyZ2XNBkZC6hqWHyVw50PsoZjt4/gjndZDx4I?=
 =?us-ascii?Q?PjTDi1M4NW71F0Vur5iDqokinlrEewMu8BsdrkAe2zzQFoIdKWiQOS0VnHSl?=
 =?us-ascii?Q?YTt3M+EQUJJ2RBeBXvuflD2gyw7CE12pa44p1H4Q?=
X-MS-Exchange-CrossTenant-Network-Message-Id: bf346700-951b-4960-a320-08db235b3f36
X-MS-Exchange-CrossTenant-AuthSource: SA1PR11MB6733.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Mar 2023 00:38:21.5879
 (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: mHvhipZ0NCh/l/Nr+IzyYtK6fE45kTW284InBJUbX5wbK3dVFClYL0uTqBZNHHl8WeQVMtDoYyhAYJOF62uvgw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR11MB8065
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

kernel test robot wrote:
> Hi Kloudifold,
>
> 
> Thank you for the patch! Yet something to improve:
> 
> [auto build test ERROR on staging/staging-testing]
> 
> url:    https://github.com/intel-lab-lkp/linux/commits/Kloudifold/staging-sm750-Rename-camel-case-functions-in-sm750_cursor/20230311-053447
> patch link:    https://lore.kernel.org/r/ZAuiDTDT8dmBcRoH%40CloudiRingWorld
> patch subject: [PATCH] staging: sm750: Rename camel case functions in sm750_cursor.*
> config: sparc-allyesconfig (https://download.01.org/0day-ci/archive/20230311/202303110849.X24WnHnM-lkp@intel.com/config)
> compiler: sparc64-linux-gcc (GCC) 12.1.0
> reproduce (this is a W=1 build):
>         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         # https://github.com/intel-lab-lkp/linux/commit/c82f66fbd5902ae486c4e2227e5071a59c49a05a
>         git remote add linux-review https://github.com/intel-lab-lkp/linux
>         git fetch --no-tags linux-review Kloudifold/staging-sm750-Rename-camel-case-functions-in-sm750_cursor/20230311-053447
>         git checkout c82f66fbd5902ae486c4e2227e5071a59c49a05a
>         # save the config file
>         mkdir build_dir && cp config build_dir/.config
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=sparc olddefconfig
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=sparc SHELL=/bin/bash drivers/staging/
> 
> If you fix the issue, kindly add following tag where applicable
> | Reported-by: kernel test robot <lkp@intel.com>
> | Link: https://lore.kernel.org/oe-kbuild-all/202303110849.X24WnHnM-lkp@intel.com/

Kloudifold,

As this report shows it looks like you did not compile your patch before
sending it.  Please do so at a minimum.

Also, when making substitutions such as this 'git grep' is your friend:

$ git grep sm750_hw_cursor_setSize
drivers/staging/sm750fb/sm750.c: sm750_hw_cursor_setSize(cursor,
drivers/staging/sm750fb/sm750_cursor.c:void sm750_hw_cursor_setSize(struct lynx_cursor *cursor, int w, int h)
drivers/staging/sm750fb/sm750_cursor.h:void sm750_hw_cursor_setSize(struct lynx_cursor *cursor, int w, int h);

Ira