# 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 # 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 # Reported-by: Guenter Roeck # Reported-by: Sudip Mukherjee # Reviewed-by: Manivannan Sadhasivam # Reviewed-by: Laurent Pinchart # Signed-off-by: Linus Torvalds # < /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_64r1_defconfig_mips-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.5.0-nolibc/mips-linux/bin/mips-linux- 64r1_defconfig ./.config.64r1_defconfig:96:warning: override: CPU_BIG_ENDIAN changes choice state .config:95:warning: override: CPU_BIG_ENDIAN changes choice state # < make -s -j 160 ARCH=mips O=/kisskb/build/linus_64r1_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_64r1_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_64r1_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_64r1_defconfig_mips-gcc8 # Build took: 0:01:13.352030