linux-doc.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jonathan Denose <jdenose@google.com>
To: Oliver Sang <oliver.sang@intel.com>
Cc: Philip Li <philip.li@intel.com>,
	kernel test robot <lkp@intel.com>, Jiri Kosina <jikos@kernel.org>,
	 Benjamin Tissoires <bentiss@kernel.org>,
	Dmitry Torokhov <dmitry.torokhov@gmail.com>,
	 Jonathan Corbet <corbet@lwn.net>,
	Henrik Rydberg <rydberg@bitmath.org>,
	oe-kbuild-all@lists.linux.dev,  linux-input@vger.kernel.org,
	linux-kernel@vger.kernel.org,  linux-doc@vger.kernel.org,
	Angela Czubak <aczubak@google.com>,
	 "Sean O'Brien" <seobrien@google.com>
Subject: Re: [PATCH 11/11] HID: multitouch: add haptic multitouch support
Date: Fri, 1 Aug 2025 16:30:58 -0500	[thread overview]
Message-ID: <CAMCVhVM4fbtAAVReSBJ7noGOjp+MO94kbrwQXs-zWvcHWCmwFw@mail.gmail.com> (raw)
In-Reply-To: <aIrbJy7Rb6aVNOk9@xsang-OptiPlex-9020>

On Wed, Jul 30, 2025 at 9:55 PM Oliver Sang <oliver.sang@intel.com> wrote:
>
> hi, Jonathan,
>
> On Mon, Jul 21, 2025 at 11:06:33AM -0500, Jonathan Denose wrote:
> > On Thu, Jul 17, 2025 at 6:59 PM Philip Li <philip.li@intel.com> wrote:
> > >
> > > On Thu, Jul 17, 2025 at 01:43:28PM -0500, Jonathan Denose wrote:
> > > > On Tue, Jul 15, 2025 at 6:36 AM kernel test robot <lkp@intel.com> wrote:
> > > > > kernel test robot noticed the following build errors:
> > > > >
> > > > > [auto build test ERROR on 86731a2a651e58953fc949573895f2fa6d456841]
> > > > >
> > > > > url:    https://github.com/intel-lab-lkp/linux/commits/Jonathan-Denose/HID-add-haptics-page-defines/20250714-231444
> > > > > base:   86731a2a651e58953fc949573895f2fa6d456841
> > > > > patch link:    https://lore.kernel.org/r/20250714-support-forcepads-v1-11-71c7c05748c9%40google.com
> > > > > patch subject: [PATCH 11/11] HID: multitouch: add haptic multitouch support
> > > > > config: hexagon-randconfig-r112-20250715 (https://download.01.org/0day-ci/archive/20250715/202507151942.94dhYylY-lkp@intel.com/config)
> > > > > compiler: clang version 17.0.6 (https://github.com/llvm/llvm-project 6009708b4367171ccdbf4b5905cb6a803753fe18)
> > > > > reproduce: (https://download.01.org/0day-ci/archive/20250715/202507151942.94dhYylY-lkp@intel.com/reproduce)
> > > >
> > > > I'm having trouble reproducing this build error. I tried following the
> > >
> > > Sorry Jonathan, the reproduce step we provide is wrong, would you mind to give
> > > a try similar to the steps in [1]? We will resolve the bug as early as possible.
> > >
> > > [1] https://download.01.org/0day-ci/archive/20250717/202507170506.Wzz1lR5I-lkp@intel.com/reproduce
> > >
> > > > steps in the linked reproduce file, but when running:
> > > > COMPILER_INSTALL_PATH=$HOME/0day ~/lkp-tests/kbuild/make.cross C=1
> > > > CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__ -fmax-errors=unlimited
> > > > -fmax-warnings=unlimited' O=build_dir ARCH=hexagon olddefconfig
> > > >
> > > > I get the errors:
> > > > 0day/gcc-4.6.1-nolibc/hexagon-linux/bin/hexagon-linux-gcc: unknown C compiler
> > > > scripts/Kconfig.include:45: Sorry, this C compiler is not supported.
> > > >
> > > > It looks to me like the hexagon-linux-gcc compiler is correctly
> > > > installed at $HOME/0day so I'm not sure what to do from here. Can
> > > > someone please assist me with this?
> > > >
> > > > --
> > > > Jonathan
> > > >
> > Great! Thanks for providing the correct reproduce steps Phillip.
> >
> > I tried them and both of the make.cross steps completed successfully.
> > I am not getting the build errors that the test bot is reporting.
>
> sorry for this. just want to confirm one thing, did you follow below steps?
> (the link [1] above is just for example, we need do small modification to
> reproduce the issue in original report, there are 4 diff in below with [1],
> (1) use your commit, (2) 'wget' command to get correct config (3) change to
> use clang-17, btw, clang-20 can also reproduce the issue (4) change build
> source to 'drivers/hid')
>
> reproduce:
>         git clone https://github.com/intel/lkp-tests.git ~/lkp-tests
>         # https://github.com/intel-lab-lkp/linux/commit/4ccef2fdc95970f67857113edb4103d53205ac9c
>         git remote add linux-review https://github.com/intel-lab-lkp/linux
>         git fetch --no-tags linux-review Jonathan-Denose/HID-add-haptics-page-defines/20250714-231444
>         git checkout 4ccef2fdc95970f67857113edb4103d53205ac9c
>         # save the config file
>         wget https://download.01.org/0day-ci/archive/20250715/202507151942.94dhYylY-lkp@intel.com/config
>         mkdir build_dir && cp config build_dir/.config
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang-17 ~/lkp-tests/kbuild/make.cross W=1 O=build_dir ARCH=hexagon olddefconfig
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang-17 ~/lkp-tests/kbuild/make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash drivers/hid/
>
>
> I can reproduce the issue with above steps, if you still cannot reproduce,
> could you give me your full log? below is mine just FYI (this is for clang-20,
> and I use some different folders but not important)
>
> xsang@xsang-OptiPlex-9020:~/linux$ COMPILER_INSTALL_PATH=/home/xsang/cross-compiler/ COMPILER=clang-20 /home/xsang/lkp-tests/kbuild/make.cross W=1 O=build_dir ARCH=hexagon olddefconfig
> Compiler will be installed in /home/xsang/cross-compiler/
> lftpget -c https://cdn.kernel.org/pub/tools/llvm/files/./llvm-20.1.8-x86_64.tar.xz
> /home/xsang/linux
> tar Jxf /home/xsang/cross-compiler//./llvm-20.1.8-x86_64.tar.xz -C /home/xsang/cross-compiler/
> PATH=/home/xsang/cross-compiler//llvm-20.1.8-x86_64/bin:/home/xsang/.local/bin:/home/xsang/bin:/home/xsang/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/home/xsang/.local/bin
> make --keep-going LLVM=1 CROSS_COMPILE=hexagon-linux- --jobs=72 KCFLAGS= -Wno-error=return-type -Wreturn-type -funsigned-char -Wundef W=1 O=build_dir ARCH=hexagon olddefconfig
> make[1]: Entering directory '/home/xsang/linux/build_dir'
>   GEN     Makefile
>   HOSTCC  scripts/basic/fixdep
>   HOSTCC  scripts/kconfig/conf.o
>   HOSTCC  scripts/kconfig/confdata.o
>   HOSTCC  scripts/kconfig/expr.o
>   LEX     scripts/kconfig/lexer.lex.c
>   YACC    scripts/kconfig/parser.tab.[ch]
>   HOSTCC  scripts/kconfig/menu.o
>   HOSTCC  scripts/kconfig/preprocess.o
>   HOSTCC  scripts/kconfig/symbol.o
>   HOSTCC  scripts/kconfig/util.o
>   HOSTCC  scripts/kconfig/lexer.lex.o
>   HOSTCC  scripts/kconfig/parser.tab.o
>   HOSTLD  scripts/kconfig/conf
> #
> # configuration written to .config
> #
> make[1]: Leaving directory '/home/xsang/linux/build_dir'
>
>
> xsang@xsang-OptiPlex-9020:~/linux$ COMPILER_INSTALL_PATH=/home/xsang/cross-compiler/ COMPILER=clang-20 /home/xsang/lkp-tests/kbuild/make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash drivers/hid/
> Compiler will be installed in /home/xsang/cross-compiler/
> PATH=/home/xsang/cross-compiler//llvm-20.1.8-x86_64/bin:/home/xsang/.local/bin:/home/xsang/bin:/home/xsang/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/home/xsang/.local/bin
> make --keep-going LLVM=1 CROSS_COMPILE=hexagon-linux- --jobs=72 KCFLAGS= -Wno-error=return-type -Wreturn-type -funsigned-char -Wundef W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash drivers/hid/
> make[1]: Entering directory '/home/xsang/linux/build_dir'
>   GEN     Makefile
>   GENSEED scripts/basic/randstruct.seed
>   WRAP    arch/hexagon/include/generated/uapi/asm/ucontext.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/auxvec.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/bitsperlong.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/bpf_perf_event.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/errno.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/fcntl.h
>   UPD     include/generated/uapi/linux/version.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/ioctl.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/ipcbuf.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/ioctls.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/mman.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/msgbuf.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/poll.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/posix_types.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/resource.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/sembuf.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/shmbuf.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/siginfo.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/sockios.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/socket.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/stat.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/statfs.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/termbits.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/types.h
>   WRAP    arch/hexagon/include/generated/uapi/asm/termios.h
>   SYSHDR  arch/hexagon/include/generated/uapi/asm/unistd_32.h
>   UPD     include/config/kernel.release
>   UPD     include/generated/compile.h
>   HOSTCC  scripts/dtc/dtc.o
>   HOSTCC  scripts/dtc/flattree.o
>   HOSTCC  scripts/dtc/fstree.o
>   HOSTCC  scripts/dtc/data.o
>   HOSTCC  scripts/dtc/livetree.o
>   HOSTCC  scripts/dtc/treesource.o
>   HOSTCC  scripts/dtc/srcpos.o
>   HOSTCC  scripts/dtc/checks.o
>   HOSTCC  scripts/dtc/util.o
>   LEX     scripts/dtc/dtc-lexer.lex.c
>   YACC    scripts/dtc/dtc-parser.tab.[ch]
>   HOSTCC  scripts/dtc/libfdt/fdt.o
>   HOSTCC  scripts/dtc/libfdt/fdt_ro.o
>   UPD     include/generated/utsrelease.h
>   HOSTCC  scripts/dtc/libfdt/fdt_wip.o
>   HOSTCC  scripts/dtc/libfdt/fdt_sw.o
>   HOSTCC  scripts/dtc/libfdt/fdt_rw.o
>   HOSTCC  scripts/dtc/libfdt/fdt_strerror.o
>   HOSTCC  scripts/dtc/libfdt/fdt_empty_tree.o
>   HOSTCC  scripts/dtc/libfdt/fdt_addresses.o
>   HOSTCC  scripts/dtc/libfdt/fdt_overlay.o
>   HOSTCC  scripts/dtc/fdtoverlay.o
>   WRAP    arch/hexagon/include/generated/asm/extable.h
>   WRAP    arch/hexagon/include/generated/asm/iomap.h
>   WRAP    arch/hexagon/include/generated/asm/kvm_para.h
>   WRAP    arch/hexagon/include/generated/asm/mcs_spinlock.h
>   WRAP    arch/hexagon/include/generated/asm/text-patching.h
>   WRAP    arch/hexagon/include/generated/asm/archrandom.h
>   WRAP    arch/hexagon/include/generated/asm/barrier.h
>   WRAP    arch/hexagon/include/generated/asm/bug.h
>   WRAP    arch/hexagon/include/generated/asm/cfi.h
>   WRAP    arch/hexagon/include/generated/asm/compat.h
>   WRAP    arch/hexagon/include/generated/asm/current.h
>   WRAP    arch/hexagon/include/generated/asm/device.h
>   WRAP    arch/hexagon/include/generated/asm/div64.h
>   WRAP    arch/hexagon/include/generated/asm/dma-mapping.h
>   WRAP    arch/hexagon/include/generated/asm/emergency-restart.h
>   WRAP    arch/hexagon/include/generated/asm/ftrace.h
>   WRAP    arch/hexagon/include/generated/asm/hardirq.h
>   WRAP    arch/hexagon/include/generated/asm/hw_irq.h
>   WRAP    arch/hexagon/include/generated/asm/irq_regs.h
>   WRAP    arch/hexagon/include/generated/asm/irq_work.h
>   WRAP    arch/hexagon/include/generated/asm/kdebug.h
>   WRAP    arch/hexagon/include/generated/asm/kmap_size.h
>   WRAP    arch/hexagon/include/generated/asm/kprobes.h
>   WRAP    arch/hexagon/include/generated/asm/local.h
>   WRAP    arch/hexagon/include/generated/asm/local64.h
>   WRAP    arch/hexagon/include/generated/asm/mmiowb.h
>   WRAP    arch/hexagon/include/generated/asm/module.h
>   WRAP    arch/hexagon/include/generated/asm/module.lds.h
>   WRAP    arch/hexagon/include/generated/asm/msi.h
>   WRAP    arch/hexagon/include/generated/asm/pci.h
>   WRAP    arch/hexagon/include/generated/asm/percpu.h
>   WRAP    arch/hexagon/include/generated/asm/preempt.h
>   WRAP    arch/hexagon/include/generated/asm/rqspinlock.h
>   WRAP    arch/hexagon/include/generated/asm/runtime-const.h
>   WRAP    arch/hexagon/include/generated/asm/rwonce.h
>   WRAP    arch/hexagon/include/generated/asm/sections.h
>   WRAP    arch/hexagon/include/generated/asm/serial.h
>   WRAP    arch/hexagon/include/generated/asm/shmparam.h
>   WRAP    arch/hexagon/include/generated/asm/simd.h
>   WRAP    arch/hexagon/include/generated/asm/softirq_stack.h
>   WRAP    arch/hexagon/include/generated/asm/topology.h
>   WRAP    arch/hexagon/include/generated/asm/trace_clock.h
>   WRAP    arch/hexagon/include/generated/asm/vga.h
>   WRAP    arch/hexagon/include/generated/asm/video.h
>   WRAP    arch/hexagon/include/generated/asm/word-at-a-time.h
>   WRAP    arch/hexagon/include/generated/asm/xor.h
>   SYSTBL  arch/hexagon/include/generated/asm/syscall_table_32.h
>   HOSTCC  scripts/dtc/dtc-lexer.lex.o
>   HOSTCC  scripts/dtc/dtc-parser.tab.o
>   HOSTLD  scripts/dtc/fdtoverlay
>   HOSTLD  scripts/dtc/dtc
>   HOSTCC  scripts/kallsyms
>   HOSTCC  scripts/asn1_compiler
>   HOSTCC  scripts/insert-sys-cert
>   CC      scripts/mod/empty.o
>   HOSTCC  scripts/mod/mk_elfconfig
>   CC      scripts/mod/devicetable-offsets.s
>   UPD     scripts/mod/devicetable-offsets.h
>   MKELF   scripts/mod/elfconfig.h
>   HOSTCC  scripts/mod/modpost.o
>   HOSTCC  scripts/mod/file2alias.o
>   HOSTCC  scripts/mod/sumversion.o
>   HOSTCC  scripts/mod/symsearch.o
>   HOSTLD  scripts/mod/modpost
>   CC      kernel/bounds.s
>   CHKSHA1 ../include/linux/atomic/atomic-arch-fallback.h
>   CHKSHA1 ../include/linux/atomic/atomic-instrumented.h
>   CHKSHA1 ../include/linux/atomic/atomic-long.h
>   UPD     include/generated/timeconst.h
>   UPD     include/generated/bounds.h
>   CC      arch/hexagon/kernel/asm-offsets.s
>   UPD     include/generated/asm-offsets.h
>   CALL    ../scripts/checksyscalls.sh
>   CC      drivers/hid/hid-core.o
>   CC      drivers/hid/hid-input.o
>   CC      drivers/hid/hid-quirks.o
>   CC      drivers/hid/hid-debug.o
>   CC      drivers/hid/hid-haptic.o
>   CC [M]  drivers/hid/usbhid/hid-core.o
>   CC      drivers/hid/hidraw.o
>   CC [M]  drivers/hid/usbhid/hiddev.o
>   CC [M]  drivers/hid/usbhid/hid-pidff.o
>   CC      drivers/hid/hid-a4tech.o
>   CC      drivers/hid/hid-alps.o
>   CC      drivers/hid/hid-apple.o
>   CC      drivers/hid/hid-belkin.o
>   CC      drivers/hid/hid-cherry.o
>   CC      drivers/hid/hid-cmedia.o
>   CC      drivers/hid/hid-cougar.o
>   CC      drivers/hid/hid-ezkey.o
>   CC      drivers/hid/hid-icade.o
>   CC      drivers/hid/hid-ite.o
>   CC      drivers/hid/hid-jabra.o
>   CC      drivers/hid/hid-kensington.o
>   CC      drivers/hid/hid-ortek.o
>   CC      drivers/hid/hid-razer.o
>   CC      drivers/hid/hid-rmi.o
>   CC      drivers/hid/hid-saitek.o
>   CC      drivers/hid/hid-sjoy.o
>   CC      drivers/hid/hid-tivo.o
>   CC      drivers/hid/hid-udraw-ps3.o
>   CC      drivers/hid/hid-led.o
>   CC      drivers/hid/hid-wiimote-core.o
>   CC      drivers/hid/hid-wiimote-modules.o
>   CC      drivers/hid/hid-wiimote-debug.o
>   CC [M]  drivers/hid/uhid.o
>   CC [M]  drivers/hid/hid-generic.o
>   CC [M]  drivers/hid/hid-axff.o
>   CC [M]  drivers/hid/hid-appleir.o
>   CC [M]  drivers/hid/hid-asus.o
>   CC [M]  drivers/hid/hid-aureal.o
>   CC [M]  drivers/hid/hid-betopff.o
>   CC [M]  drivers/hid/hid-bigbenff.o
>   CC [M]  drivers/hid/hid-chicony.o
>   CC [M]  drivers/hid/hid-corsair.o
>   CC [M]  drivers/hid/hid-corsair-void.o
>   CC [M]  drivers/hid/hid-cp2112.o
>   CC [M]  drivers/hid/hid-cypress.o
>   CC [M]  drivers/hid/hid-emsff.o
>   CC [M]  drivers/hid/hid-elan.o
>   CC [M]  drivers/hid/hid-elo.o
>   CC [M]  drivers/hid/hid-gembird.o
>   CC [M]  drivers/hid/hid-gfrm.o
>   CC [M]  drivers/hid/hid-vivaldi-common.o
>   CC [M]  drivers/hid/hid-google-stadiaff.o
>   CC [M]  drivers/hid/hid-vivaldi.o
>   CC [M]  drivers/hid/hid-gt683r.o
>   CC [M]  drivers/hid/hid-gyration.o
>   CC [M]  drivers/hid/hid-holtek-kbd.o
>   CC [M]  drivers/hid/hid-holtek-mouse.o
>   CC [M]  drivers/hid/hid-holtekff.o
>   CC [M]  drivers/hid/hid-kye.o
>   CC [M]  drivers/hid/hid-kysona.o
>   CC [M]  drivers/hid/hid-letsketch.o
>   CC [M]  drivers/hid/hid-macally.o
>   CC [M]  drivers/hid/hid-magicmouse.o
>   CC [M]  drivers/hid/hid-mcp2221.o
>   CC [M]  drivers/hid/hid-megaworld.o
>   CC [M]  drivers/hid/hid-microsoft.o
>   CC [M]  drivers/hid/hid-nintendo.o
>   CC [M]  drivers/hid/hid-nti.o
>   CC [M]  drivers/hid/hid-pl.o
>   CC [M]  drivers/hid/hid-penmount.o
>   CC [M]  drivers/hid/hid-picolcd_core.o
>   CC [M]  drivers/hid/hid-picolcd_fb.o
>   CC [M]  drivers/hid/hid-picolcd_backlight.o
>   CC [M]  drivers/hid/hid-picolcd_leds.o
>   CC [M]  drivers/hid/hid-picolcd_cir.o
>   CC [M]  drivers/hid/hid-picolcd_debugfs.o
> ../drivers/hid/hid-haptic.c:13:6: error: redefinition of 'hid_haptic_feature_mapping'
>    13 | void hid_haptic_feature_mapping(struct hid_device *hdev,
>       |      ^
> ../drivers/hid/hid-haptic.h:83:6: note: previous definition is here
>    83 | void hid_haptic_feature_mapping(struct hid_device *hdev,
>       |      ^
> ../drivers/hid/hid-haptic.c:51:6: error: redefinition of 'hid_haptic_check_pressure_unit'
>    51 | bool hid_haptic_check_pressure_unit(struct hid_haptic_device *haptic,
>       |      ^
> ../drivers/hid/hid-haptic.h:89:6: note: previous definition is here
>    89 | bool hid_haptic_check_pressure_unit(struct hid_haptic_device *haptic,
>       |      ^
> ../drivers/hid/hid-haptic.c:65:5: error: redefinition of 'hid_haptic_input_mapping'
>    65 | int hid_haptic_input_mapping(struct hid_device *hdev,
>       |     ^
> ../drivers/hid/hid-haptic.h:95:5: note: previous definition is here
>    95 | int hid_haptic_input_mapping(struct hid_device *hdev,
>       |     ^
> ../drivers/hid/hid-haptic.c:81:5: error: redefinition of 'hid_haptic_input_configured'
>    81 | int hid_haptic_input_configured(struct hid_device *hdev,
>       |     ^
> ../drivers/hid/hid-haptic.h:104:5: note: previous definition is here
>   104 | int hid_haptic_input_configured(struct hid_device *hdev,
>       |     ^
> ../drivers/hid/hid-haptic.c:403:5: error: redefinition of 'hid_haptic_init'
>   403 | int hid_haptic_init(struct hid_device *hdev,
>       |     ^
> ../drivers/hid/hid-haptic.h:114:5: note: previous definition is here
>   114 | int hid_haptic_init(struct  CC [M]  drivers/hid/hid-redragon.o
>  hid_device *hdev, struct hid_haptic_device **haptic_ptr)
>       |     ^
> ../drivers/hid/hid-haptic.c:569:6: error: redefinition of 'hid_haptic_pressure_reset'
>   569 | void hid_haptic_pressure_reset(struct hid_haptic_device *haptic)
>       |      ^
> ../drivers/hid/hid-haptic.h:126:6: note: previous definition is here
>   126 | void hid_haptic_pressure_reset(struct hid_haptic_device *haptic) {}
>       |      ^
> ../drivers/hid/hid-haptic.c:575:6: error: redefinition of 'hid_haptic_pressure_increase'
>   575 | void hid_haptic_pressure_increase(struct hid_haptic_device *haptic,
>       |      ^
> ../drivers/hid/hid-haptic.h:128:6: note: previous definition is here
>   128 | void hid_haptic_pressure_increase(struct hid_haptic_device *haptic,
>       |   CC [M]  drivers/hid/hid-retrode.o
>      ^
> 7 errors generated.
> make[5]: *** [../scripts/Makefile.build:287: drivers/hid/hid-haptic.o] Error 1
>   CC [M]  drivers/hid/hid-roccat.o
>   CC [M]  drivers/hid/hid-roccat-common.o
>   CC [M]  drivers/hid/hid-roccat-arvo.o
>   CC [M]  drivers/hid/hid-roccat-isku.o
>   CC [M]  drivers/hid/hid-roccat-kone.o
>   CC [M]  drivers/hid/hid-roccat-koneplus.o
>   CC [M]  drivers/hid/hid-roccat-konepure.o
>   CC [M]  drivers/hid/hid-roccat-kovaplus.o
>   CC [M]  drivers/hid/hid-roccat-lua.o
>   CC [M]  drivers/hid/hid-roccat-pyra.o
>   CC [M]  drivers/hid/hid-roccat-ryos.o
>   CC [M]  drivers/hid/hid-roccat-savu.o
>   CC [M]  drivers/hid/hid-samsung.o
>   CC [M]  drivers/hid/hid-sony.o
>   CC [M]  drivers/hid/hid-steam.o
>   CC [M]  drivers/hid/hid-sunplus.o
>   CC [M]  drivers/hid/hid-gaff.o
>   CC [M]  drivers/hid/hid-tmff.o
>   CC [M]  drivers/hid/hid-thrustmaster.o
>   CC [M]  drivers/hid/hid-uclogic-core.o
>   CC [M]  drivers/hid/hid-uclogic-rdesc.o
>   CC [M]  drivers/hid/hid-uclogic-params.o
>   CC [M]  drivers/hid/hid-xinmo.o
>   CC [M]  drivers/hid/hid-zpff.o
>   CC [M]  drivers/hid/hid-vrc2.o
>   CC [M]  drivers/hid/wacom_sys.o
>   CC [M]  drivers/hid/wacom_wac.o
>   CC [M]  drivers/hid/hid-waltop.o
>   CC [M]  drivers/hid/hid-winwing.o
>   CC [M]  drivers/hid/hid-uclogic-rdesc-test.o
>   LD [M]  drivers/hid/usbhid/usbhid.o
>   LD [M]  drivers/hid/hid-uclogic-test.o
>   LD [M]  drivers/hid/hid-picolcd.o
>   LD [M]  drivers/hid/hid-uclogic.o
>   LD [M]  drivers/hid/wacom.o
> make[5]: Target 'drivers/hid/' not remade because of errors.
> make[4]: *** [../scripts/Makefile.build:554: drivers/hid] Error 2
> make[4]: Target 'drivers/hid/' not remade because of errors.
> make[3]: *** [../scripts/Makefile.build:554: drivers] Error 2
> make[3]: Target 'drivers/hid/' not remade because of errors.
> make[2]: *** [/home/xsang/linux/Makefile:2003: .] Error 2
> make[2]: Target 'drivers/hid/' not remade because of errors.
> make[1]: *** [/home/xsang/linux/Makefile:248: __sub-make] Error 2
> make[1]: Target 'drivers/hid/' not remade because of errors.
> make[1]: Leaving directory '/home/xsang/linux/build_dir'
> make: *** [Makefile:248: __sub-make] Error 2
> make: Target 'drivers/hid/' not remade because of errors.
>
>
>
> > --
> > Jonathan
> >
Hi Oliver,

I think I didn't have the right config so the issue wasn't reproducing
but I'm seeing it now so I'll fix and resubmit.

Thanks!
-- 
Jonathan

  reply	other threads:[~2025-08-01 21:31 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-07-14 15:09 [PATCH 00/11] HID: Implement haptic forcepad support Jonathan Denose
2025-07-14 15:09 ` [PATCH 01/11] HID: add haptics page defines Jonathan Denose
2025-07-14 15:09 ` [PATCH 02/11] Input: add FF_HID effect type Jonathan Denose
2025-07-14 15:09 ` [PATCH 03/11] Input: add INPUT_PROP_HAPTIC_TOUCHPAD Jonathan Denose
2025-07-14 15:09 ` [PATCH 04/11] HID: haptic: introduce hid_haptic_device Jonathan Denose
2025-07-14 15:09 ` [PATCH 05/11] HID: input: allow mapping of haptic output Jonathan Denose
2025-07-14 15:09 ` [PATCH 06/11] HID: haptic: initialize haptic device Jonathan Denose
2025-07-14 15:09 ` [PATCH 07/11] HID: input: calculate resolution for pressure Jonathan Denose
2025-07-14 15:09 ` [PATCH 08/11] HID: haptic: add functions handling events Jonathan Denose
2025-07-14 15:09 ` [PATCH 09/11] Input: MT - add INPUT_MT_TOTAL_FORCE flags Jonathan Denose
2025-07-14 15:09 ` [PATCH 10/11] HID: haptic: add hid_haptic_switch_mode Jonathan Denose
2025-07-14 15:09 ` [PATCH 11/11] HID: multitouch: add haptic multitouch support Jonathan Denose
2025-07-15 11:35   ` kernel test robot
2025-07-17 18:43     ` Jonathan Denose
2025-07-17 23:59       ` Philip Li
2025-07-21 16:06         ` Jonathan Denose
2025-07-31  2:55           ` Oliver Sang
2025-08-01 21:30             ` Jonathan Denose [this message]
2025-07-23 19:59 ` [PATCH 00/11] HID: Implement haptic forcepad support Jonathan Denose

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAMCVhVM4fbtAAVReSBJ7noGOjp+MO94kbrwQXs-zWvcHWCmwFw@mail.gmail.com \
    --to=jdenose@google.com \
    --cc=aczubak@google.com \
    --cc=bentiss@kernel.org \
    --cc=corbet@lwn.net \
    --cc=dmitry.torokhov@gmail.com \
    --cc=jikos@kernel.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lkp@intel.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=oliver.sang@intel.com \
    --cc=philip.li@intel.com \
    --cc=rydberg@bitmath.org \
    --cc=seobrien@google.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).