On 07/07/2024 17:20, kernel test robot wrote: > tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master > head: 0b58e108042b0ed28a71cd7edf5175999955b233 > commit: 09ea421652a832083ea380a72addf383965f3682 [7766/10451] clk: qcom: camcc-sm8650: Add SM8650 camera clock controller driver > config: x86_64-allyesconfig (https://download.01.org/0day-ci/archive/20240707/202407072331.baglL4Sd-lkp@xxxxxxxxx/config) > compiler: clang version 18.1.5 (https://github.com/llvm/llvm-project 617a15a9eac96088ae5e9134248d8236e34b91b1) > reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240707/202407072331.baglL4Sd-lkp@xxxxxxxxx/reproduce) This and this: https://lore.kernel.org/all/202407072212.WptVaUDt-lkp@xxxxxxxxx/ are reports on linux-next. They were addressed to you/ Jagadeesh, how did you respond to public reports that your commit has build failures in linux-next? It reached now Linus' tree causing huge rant, unfortunately not in the direction we would like that rant to go. If you receive a report that your code does not even compile, it is YOUR DAMN MOST IMPORTANT TASK to address it IMMEDIATELY. I don't think such code should be ever accepted if contributors do not take responsibility for their commits. Krzysztof > > 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/202407072331.baglL4Sd-lkp@xxxxxxxxx/ > > Note: the linux-next/master HEAD 0b58e108042b0ed28a71cd7edf5175999955b233 builds fine. > It may have been fixed somewhere. > > All errors (new ones prefixed by >>): > >>> drivers/clk/qcom/camcc-sm8650.c:3573:29: error: incompatible pointer types passing 'struct device *' to parameter of type 'struct platform_device *' [-Werror,-Wincompatible-pointer-types] > 3573 | ret = qcom_cc_really_probe(&pdev->dev, &cam_cc_sm8650_desc, regmap); > | ^~~~~~~~~~ > drivers/clk/qcom/common.h:63:57: note: passing argument to parameter 'pdev' here > 63 | extern int qcom_cc_really_probe(struct platform_device *pdev, > | ^ > 1 error generated. >
