From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 7076FC433EF for ; Sun, 13 Mar 2022 19:01:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=XIbfHtQCn1Ym+tRJA+c3IgC9z1BsKwQkN+d06JSdcQ0=; b=dIgzpIEG1omkXk cITVYRxsYUtTH1iFf8nn0ueT2+ggwnbNO7f+KfZGAodltt3xKC2Z1H1JJXI2eg4hPpze70+YWszfy Gr8pDfsACNNvprwDGVnrOeFG8C8xBzl5knzdbcDfJs4x5emKjrRKpEPUTISkya8VtbU75bKt6O546 cwTI35FQGewOjhIiW3wUgo2/5UoG72VcosbjwcdwWzi4b/kSKEG4p6GlANQ0oM3LxMcZQAOI77yCo 9566flB+Y8CGaa97Guo8etuzxB5lMqhnDaVWTj9c/yAUtrFrmFazjJFZQQ5EumTLCekZiajVdYaIS l2fMhv+49AHeslr0Girg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nTTRu-003Frk-TE; Sun, 13 Mar 2022 19:00:15 +0000 Received: from mga01.intel.com ([192.55.52.88]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nTTRr-003FqG-4O; Sun, 13 Mar 2022 19:00:13 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1647198011; x=1678734011; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=7A56rvBq2Ze2rosgzGpqZSzGRYf4vonTsoVqZ1kpJGA=; b=Bw9rE3YOsiPwcgTM6f0qejRSCbcZ6XX0qEP64dADPvAk+4ofXRNH0fzN 8yxMG0xR+kjX7POmiQ4R3PSsnxOWhkS/xMWD0opDQ05v052G+eBLizw6j FCwibnhk9Deh5ltAT6lpaHshTXHhxf2SsrE3rlLBdz+/ceCd5U89CeDPi uCWxSEktSfwYX0QCsrMWVtd+2Wpxr6sA/OOQO58H00qS4WfEiUnCLSZpF yo0xE1kW2BuFfyjHetljmaZMHc35W89EFhofpK23aZy/IFRvefXYQSISm 5Q2z1xlWyOR0X+9KQXwP6f5W4bPOCR2UY34Aoonp/djuUBJ6KDE6tPBzD A==; X-IronPort-AV: E=McAfee;i="6200,9189,10285"; a="280647315" X-IronPort-AV: E=Sophos;i="5.90,179,1643702400"; d="scan'208";a="280647315" Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 Mar 2022 12:00:07 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.90,179,1643702400"; d="scan'208";a="612673907" Received: from lkp-server02.sh.intel.com (HELO 89b41b6ae01c) ([10.239.97.151]) by fmsmga004.fm.intel.com with ESMTP; 13 Mar 2022 12:00:03 -0700 Received: from kbuild by 89b41b6ae01c with local (Exim 4.92) (envelope-from ) id 1nTTRi-0009DE-LF; Sun, 13 Mar 2022 19:00:02 +0000 Date: Mon, 14 Mar 2022 02:59:45 +0800 From: kernel test robot To: Jiaxin Yu , broonie@kernel.org, robh+dt@kernel.org, angelogioacchino.delregno@collabora.com Cc: kbuild-all@lists.01.org, aaronyu@google.com, matthias.bgg@gmail.com, trevor.wu@mediatek.com, tzungbi@google.com, julianbraha@gmail.com, alsa-devel@alsa-project.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org, Project_Global_Chrome_Upstream_Group@mediatek.com, Jiaxin Yu Subject: Re: [v3 13/19] ASoC: mediatek: mt8186: add platform driver Message-ID: <202203140250.ob821GmA-lkp@intel.com> References: <20220313151023.21229-14-jiaxin.yu@mediatek.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20220313151023.21229-14-jiaxin.yu@mediatek.com> User-Agent: Mutt/1.10.1 (2018-07-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220313_120011_244179_338E9875 X-CRM114-Status: GOOD ( 18.33 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Jiaxin, Thank you for the patch! Yet something to improve: [auto build test ERROR on broonie-sound/for-next] [also build test ERROR on broonie-spi/for-next robh/for-next v5.17-rc7 next-20220310] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/Jiaxin-Yu/ASoC-mediatek-Add-support-for-MT8186-SoC/20220313-231330 base: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next config: m68k-allyesconfig (https://download.01.org/0day-ci/archive/20220314/202203140250.ob821GmA-lkp@intel.com/config) compiler: m68k-linux-gcc (GCC) 11.2.0 reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # https://github.com/0day-ci/linux/commit/088850061de683cf1cac329e1187f45fb1d25085 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Jiaxin-Yu/ASoC-mediatek-Add-support-for-MT8186-SoC/20220313-231330 git checkout 088850061de683cf1cac329e1187f45fb1d25085 # save the config file to linux build tree mkdir build_dir COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=m68k SHELL=/bin/bash If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot All errors (new ones prefixed by >>): In file included from include/linux/device.h:15, from include/linux/dma-mapping.h:7, from sound/soc/mediatek/mt8186/mt8186-afe-pcm.c:9: sound/soc/mediatek/mt8186/mt8186-afe-pcm.c: In function 'mt8186_afe_pcm_dev_probe': >> sound/soc/mediatek/mt8186/mt8186-afe-pcm.c:2914:40: error: 'struct dev_pm_info' has no member named 'runtime_error' 2914 | ret, dev->power.runtime_error); | ^ include/linux/dev_printk.h:110:37: note: in definition of macro 'dev_printk_index_wrap' 110 | _p_func(dev, fmt, ##__VA_ARGS__); \ | ^~~~~~~~~~~ sound/soc/mediatek/mt8186/mt8186-afe-pcm.c:2913:17: note: in expansion of macro 'dev_err' 2913 | dev_err(dev, "get_ret:%d, rpm_error:%d\n", | ^~~~~~~ sound/soc/mediatek/mt8186/mt8186-afe-pcm.c:2948:40: error: 'struct dev_pm_info' has no member named 'runtime_error' 2948 | ret, dev->power.runtime_error); | ^ include/linux/dev_printk.h:110:37: note: in definition of macro 'dev_printk_index_wrap' 110 | _p_func(dev, fmt, ##__VA_ARGS__); \ | ^~~~~~~~~~~ sound/soc/mediatek/mt8186/mt8186-afe-pcm.c:2947:17: note: in expansion of macro 'dev_err' 2947 | dev_err(dev, "put_ret:%d, rpm_error:%d\n", | ^~~~~~~ vim +2914 sound/soc/mediatek/mt8186/mt8186-afe-pcm.c 2803 2804 static int mt8186_afe_pcm_dev_probe(struct platform_device *pdev) 2805 { 2806 struct mtk_base_afe *afe; 2807 struct mt8186_afe_private *afe_priv; 2808 struct resource *res; 2809 struct reset_control *rstc; 2810 struct device *dev = &pdev->dev; 2811 int i, ret, irq_id; 2812 2813 ret = dma_set_mask_and_coherent(dev, DMA_BIT_MASK(34)); 2814 if (ret) 2815 return ret; 2816 2817 afe = devm_kzalloc(dev, sizeof(*afe), GFP_KERNEL); 2818 if (!afe) 2819 return -ENOMEM; 2820 platform_set_drvdata(pdev, afe); 2821 2822 afe->platform_priv = devm_kzalloc(dev, sizeof(*afe_priv), GFP_KERNEL); 2823 if (!afe->platform_priv) 2824 return -ENOMEM; 2825 2826 afe_priv = afe->platform_priv; 2827 afe->dev = &pdev->dev; 2828 2829 res = platform_get_resource(pdev, IORESOURCE_MEM, 0); 2830 afe->base_addr = devm_ioremap_resource(&pdev->dev, res); 2831 if (IS_ERR(afe->base_addr)) 2832 return PTR_ERR(afe->base_addr); 2833 2834 /* init audio related clock */ 2835 ret = mt8186_init_clock(afe); 2836 if (ret) { 2837 dev_err(dev, "init clock error, ret %d\n", ret); 2838 return ret; 2839 } 2840 2841 /* init memif */ 2842 afe->memif_32bit_supported = 0; 2843 afe->memif_size = MT8186_MEMIF_NUM; 2844 afe->memif = devm_kcalloc(dev, afe->memif_size, sizeof(*afe->memif), 2845 GFP_KERNEL); 2846 2847 if (!afe->memif) 2848 return -ENOMEM; 2849 2850 for (i = 0; i < afe->memif_size; i++) { 2851 afe->memif[i].data = &memif_data[i]; 2852 afe->memif[i].irq_usage = memif_irq_usage[i]; 2853 afe->memif[i].const_irq = 1; 2854 } 2855 2856 mutex_init(&afe->irq_alloc_lock); /* needed when dynamic irq */ 2857 2858 /* init irq */ 2859 afe->irqs_size = MT8186_IRQ_NUM; 2860 afe->irqs = devm_kcalloc(dev, afe->irqs_size, sizeof(*afe->irqs), 2861 GFP_KERNEL); 2862 2863 if (!afe->irqs) 2864 return -ENOMEM; 2865 2866 for (i = 0; i < afe->irqs_size; i++) 2867 afe->irqs[i].irq_data = &irq_data[i]; 2868 2869 /* request irq */ 2870 irq_id = platform_get_irq(pdev, 0); 2871 if (irq_id <= 0) 2872 return dev_err_probe(dev, irq_id < 0 ? irq_id : -ENXIO, 2873 "no irq found"); 2874 2875 ret = devm_request_irq(dev, irq_id, mt8186_afe_irq_handler, 2876 IRQF_TRIGGER_NONE, 2877 "Afe_ISR_Handle", (void *)afe); 2878 if (ret) 2879 return dev_err_probe(dev, ret, "could not request_irq for Afe_ISR_Handle\n"); 2880 2881 ret = enable_irq_wake(irq_id); 2882 if (ret < 0) 2883 return dev_err_probe(dev, ret, "enable_irq_wake %d\n", irq_id); 2884 2885 /* init sub_dais */ 2886 INIT_LIST_HEAD(&afe->sub_dais); 2887 2888 for (i = 0; i < ARRAY_SIZE(dai_register_cbs); i++) { 2889 ret = dai_register_cbs[i](afe); 2890 if (ret) 2891 return dev_err_probe(dev, ret, "dai register i %d fail\n", i); 2892 } 2893 2894 /* init dai_driver and component_driver */ 2895 ret = mtk_afe_combine_sub_dai(afe); 2896 if (ret) 2897 return dev_err_probe(dev, ret, "mtk_afe_combine_sub_dai fail\n"); 2898 2899 /* reset controller to reset audio regs before regmap cache */ 2900 rstc = devm_reset_control_get_exclusive(dev, "audiosys"); 2901 if (IS_ERR(rstc)) 2902 return dev_err_probe(dev, PTR_ERR(rstc), "could not get audiosys reset\n"); 2903 2904 ret = reset_control_reset(rstc); 2905 if (ret) 2906 return dev_err_probe(dev, ret, "failed to trigger audio reset\n"); 2907 2908 /* enable clock for regcache get default value from hw */ 2909 afe_priv->pm_runtime_bypass_reg_ctl = true; 2910 pm_runtime_enable(dev); 2911 ret = pm_runtime_get_sync(dev); 2912 if (ret) 2913 dev_err(dev, "get_ret:%d, rpm_error:%d\n", > 2914 ret, dev->power.runtime_error); 2915 2916 afe->regmap = devm_regmap_init_mmio(dev, afe->base_addr, 2917 &mt8186_afe_regmap_config); 2918 if (IS_ERR(afe->regmap)) { 2919 ret = PTR_ERR(afe->regmap); 2920 goto err_pm_disable; 2921 } 2922 2923 /* others */ 2924 afe->mtk_afe_hardware = &mt8186_afe_hardware; 2925 afe->memif_fs = mt8186_memif_fs; 2926 afe->irq_fs = mt8186_irq_fs; 2927 afe->get_dai_fs = mt8186_get_dai_fs; 2928 afe->get_memif_pbuf_size = mt8186_get_memif_pbuf_size; 2929 2930 afe->runtime_resume = mt8186_afe_runtime_resume; 2931 afe->runtime_suspend = mt8186_afe_runtime_suspend; 2932 2933 /* register platform */ 2934 dev_info(dev, "%s(), devm_snd_soc_register_component\n", __func__); 2935 2936 ret = devm_snd_soc_register_component(dev, 2937 &mt8186_afe_component, 2938 afe->dai_drivers, 2939 afe->num_dai_drivers); 2940 if (ret) { 2941 dev_err(dev, "err_dai_component\n"); 2942 goto err_pm_disable; 2943 } 2944 2945 ret = pm_runtime_put_sync(dev); 2946 if (ret) 2947 dev_err(dev, "put_ret:%d, rpm_error:%d\n", 2948 ret, dev->power.runtime_error); 2949 afe_priv->pm_runtime_bypass_reg_ctl = false; 2950 2951 regcache_cache_only(afe->regmap, true); 2952 regcache_mark_dirty(afe->regmap); 2953 2954 return 0; 2955 2956 err_pm_disable: 2957 pm_runtime_put_sync(dev); 2958 pm_runtime_disable(dev); 2959 2960 return ret; 2961 } 2962 --- 0-DAY CI Kernel Test Service https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel