Re: [PATCH v7 4/6] pinctrl: Add RTL8231 pin control and GPIO support

[Sander Vanheule]

On Wed, 2025-11-19 at 12:19 +0800, kernel test robot wrote:
> Hi Sander,
> 
> kernel test robot noticed the following build warnings:
> 
> [auto build test WARNING on lee-mfd/for-mfd-next]
> [also build test WARNING on lee-mfd/for-mfd-fixes lee-leds/for-leds-next linusw-
> pinctrl/devel linusw-pinctrl/for-next linus/master v6.18-rc6 next-20251118]
> [If your patch is applied to the wrong git tree, kindly drop us a note.
> And when submitting patch, we suggest to use '--base' as documented in
> https://git-scm.com/docs/git-format-patch#_base_tree_information]
> 
> url:   
> https://github.com/intel-lab-lkp/linux/commits/Sander-Vanheule/dt-bindings-leds-Binding-for-RTL8231-scan-matrix/20251118-055707
> base:   https://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git for-mfd-next
> patch link:    https://lore.kernel.org/r/20251117215138.4353-5-sander%40svanheule.net
> patch subject: [PATCH v7 4/6] pinctrl: Add RTL8231 pin control and GPIO support
> config: i386-randconfig-063-20251119
> (https://download.01.org/0day-ci/archive/20251119/202511191158.bVKUDrKa-lkp@xxxxxxxxx/co
> nfig)
> compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
> reproduce (this is a W=1 build):
> (https://download.01.org/0day-ci/archive/20251119/202511191158.bVKUDrKa-lkp@xxxxxxxxx/re
> produce)
> 
> If you fix the issue in a separate patch/commit (i.e. not just a new version of
> the same patch/commit), kindly add following tags
> > Reported-by: kernel test robot <lkp@xxxxxxxxx>
> > Closes: https://lore.kernel.org/oe-kbuild-all/202511191158.bVKUDrKa-lkp@xxxxxxxxx/
> 
> sparse warnings: (new ones prefixed by >>)
> > > drivers/pinctrl/pinctrl-rtl8231.c:28:27: sparse: sparse: missing identifier in
> > > declaration
>    drivers/pinctrl/pinctrl-rtl8231.c:28:27: sparse: sparse: Expected ; at the end of
> type declaration
>    drivers/pinctrl/pinctrl-rtl8231.c:28:27: sparse: sparse: got :
> > > drivers/pinctrl/pinctrl-rtl8231.c:52:44: sparse: sparse: invalid bitfield specifier
> > > for type incomplete type enum rtl8231_pin_function.


sparse doesn't seem to understand the enum type specifier either. Fixed by instead casting
rtl8231_pin_function where a uintptr_t is required.

Best,
Sander