On Tue, Jan 23, 2024 at 10:17:25AM -0600, Chris Morgan wrote: > On Sat, Jan 20, 2024 at 06:55:32AM +0800, kernel test robot wrote: > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git next > > head: 3af6e24a456437d323d1080bd254053f7af05234 > > commit: 6380a59c534ecab1462608a1f76490289a45a377 [134/135] Input: adc-joystick - handle inverted axes > > config: i386-randconfig-011-20240120 (https://download.01.org/0day-ci/archive/20240120/202401200614.B4PnBzhk-lkp@xxxxxxxxx/config) > > compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 > > reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240120/202401200614.B4PnBzhk-lkp@xxxxxxxxx/reproduce) > > > > 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/202401200614.B4PnBzhk-lkp@xxxxxxxxx/ > > > > Assume this means we need to explicitly `#include <linux/minmax.h>`, right? > > Should I submit a new patch or a bugfix for this patch? As I mentioned in another email I think usage of min/max_array() is too clever here. How about we simply "swap(range[0], range[1])" when axis is inverted? Thanks. -- Dmitry