Buildresult: linus/32r1_defconfig/mips-gcc8 built on Apr 1 2023, 13:47

kisskb

Revisions | Branches | Compilers | Configs | Build Results | Build Failures |

Status:	OK
Date/Time:	Apr 1 2023, 13:47
Duration:	0:02:05.652065
Builder:	alpine1
Revision:	media: i2c: imx290: fix conditional function defintions (`7b50567bdcad8925ca1e075feb7171c12015afd1)`
Target:	linus/32r1_defconfig/mips-gcc8
Branch:	linus
Compiler:	mips-gcc8 (mips-linux-gcc (GCC) 8.5.0 / GNU ld (GNU Binutils) 2.36.1)
Config:	32r1_defconfig (download)
Log:	Download original Possible warnings (2) ./.config.32r1_defconfig:93:warning: override: CPU_BIG_ENDIAN changes choice state .config:92:warning: override: CPU_BIG_ENDIAN changes choice state Full Log # git rev-parse -q --verify 7b50567bdcad8925ca1e075feb7171c12015afd1^{commit} 7b50567bdcad8925ca1e075feb7171c12015afd1 already have revision, skipping fetch # git checkout -q -f -B kisskb 7b50567bdcad8925ca1e075feb7171c12015afd1 # git clean -qxdf # < git log -1 # commit 7b50567bdcad8925ca1e075feb7171c12015afd1 # Author: Arnd Bergmann <arnd@arndb.de> # Date: Tue Feb 7 17:13:12 2023 +0100 # # media: i2c: imx290: fix conditional function defintions # # The runtime suspend/resume functions are only referenced from the # dev_pm_ops, but they use the old SET_RUNTIME_PM_OPS() helper that # requires a __maybe_unused annotation to avoid a warning: # # drivers/media/i2c/imx290.c:1082:12: error: unused function 'imx290_runtime_resume' [-Werror,-Wunused-function] # static int imx290_runtime_resume(struct device dev) # ^ # drivers/media/i2c/imx290.c:1090:12: error: unused function 'imx290_runtime_suspend' [-Werror,-Wunused-function] # static int imx290_runtime_suspend(struct device dev) # ^ # # Convert this to the new RUNTIME_PM_OPS() helper that so this is not # required. To improve this further, also use the pm_ptr() helper that # lets the dev_pm_ops get dropped entirely when CONFIG_PM is disabled. # # A related mistake happened in the of_match_ptr() macro here, which like # SET_RUNTIME_PM_OPS() requires the match table to be marked as # __maybe_unused, though I could not reproduce building this without # CONFIG_OF. Remove the of_match_ptr() here as there is no point in # dropping the match table in configurations without CONFIG_OF. # # Fixes: 02852c01f654 ("media: i2c: imx290: Initialize runtime PM before subdev") # Signed-off-by: Arnd Bergmann <arnd@arndb.de> # Reported-by: Guenter Roeck <linux@roeck-us.net> # Reported-by: Sudip Mukherjee <sudipm.mukherjee@gmail.com> # Reviewed-by: Manivannan Sadhasivam <mani@kernel.org> # Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> # Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> # < /opt/cross/kisskb/korg/gcc-8.5.0-nolibc/mips-linux/bin/mips-linux-gcc --version # < /opt/cross/kisskb/korg/gcc-8.5.0-nolibc/mips-linux/bin/mips-linux-ld --version # < git log --format=%s --max-count=1 7b50567bdcad8925ca1e075feb7171c12015afd1 # < make -s -j 160 ARCH=mips O=/kisskb/build/linus_32r1_defconfig_mips-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.5.0-nolibc/mips-linux/bin/mips-linux- 32r1_defconfig ./.config.32r1_defconfig:93:warning: override: CPU_BIG_ENDIAN changes choice state .config:92:warning: override: CPU_BIG_ENDIAN changes choice state # < make -s -j 160 ARCH=mips O=/kisskb/build/linus_32r1_defconfig_mips-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.5.0-nolibc/mips-linux/bin/mips-linux- help # make -s -j 160 ARCH=mips O=/kisskb/build/linus_32r1_defconfig_mips-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.5.0-nolibc/mips-linux/bin/mips-linux- olddefconfig # make -s -j 160 ARCH=mips O=/kisskb/build/linus_32r1_defconfig_mips-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.5.0-nolibc/mips-linux/bin/mips-linux- Completed OK # rm -rf /kisskb/build/linus_32r1_defconfig_mips-gcc8 # Build took: 0:02:05.652065

© Michael Ellerman 2006-2018.