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 182B8C00140 for ; Tue, 2 Aug 2022 23:45:06 +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=Yy8Fehegm3q0Cj2yGJhwlju+0adzw7m54WKILAifN1M=; b=sox5xN6WTRcCe8 OeTkSrCujXfxPYuYPp84yyDX6FZOJIC4L0vsYEc1PBeXa4aFtnOWC1FuEPijMCX6bivPvynrS0Oou cbfrtT01U8b9PN+h8DOiN6WzDH/GvlMc74+cebhnRYHHVvvxPwq/YG6sf/8gLEa3iZooBYYd58CSF pdP+Uq/6yh62IhuLbM27H4zFlKB1tGmimuii7EMoUCtHS3RawZZDvp5FA/G3g8Dw3Dc6nw4wT/GH6 a/9C0VGfb6kC9ssOThmHLylWrFcIQF3jpesdHSSD2AQEiWpS0aSxDHK8550J8ZkdkNcTh7qZ8ECd6 41/01ERLxzTzVmkIMKfQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oJ1YK-000jul-TR; Tue, 02 Aug 2022 23:43:57 +0000 Received: from mga05.intel.com ([192.55.52.43]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oJ1Y6-000jrE-3v; Tue, 02 Aug 2022 23:43:44 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1659483822; x=1691019822; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=/LJASFRzIeInreQ73D8eRDuT2RqFrVAf9oHkIaKOgWU=; b=Lq8lK0uOHkbLE6nCNN5aThZLq90skQkOeEoksSD0l2+KZlGWiJyIbHhE ka+IVjXoI3JwdCzW4eWeOBYMXisyVPhc/8e5clGA4VQNEMP3ZvxH8fi7e zRCSK2QpAgeE1nDoemcR+IQxC40EoxHnELHrw20hTuSRwhnycGNjQdZJh 27SZqw1HyvDvFKo/mYS56pEzUomq+2jW/ORyaCT67BCMqw2HqQqLCked0 4VipQ5B4upheRQyhRNJtOWRIZRJ5A85fP3or6Z6FFvH9k94IvmonjV0hB q4tsLf1xYL6qD6trtyLmV1RXaKIcuL7SEMKAXhSSRQrRvjidJSAn/yGHn w==; X-IronPort-AV: E=McAfee;i="6400,9594,10427"; a="375843686" X-IronPort-AV: E=Sophos;i="5.93,212,1654585200"; d="scan'208";a="375843686" Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by fmsmga105.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Aug 2022 16:43:40 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.93,212,1654585200"; d="scan'208";a="930169030" Received: from lkp-server01.sh.intel.com (HELO e0eace57cfef) ([10.239.97.150]) by fmsmga005.fm.intel.com with ESMTP; 02 Aug 2022 16:43:38 -0700 Received: from kbuild by e0eace57cfef with local (Exim 4.96) (envelope-from ) id 1oJ1Y2-000GZP-0X; Tue, 02 Aug 2022 23:43:38 +0000 Date: Wed, 3 Aug 2022 07:43:06 +0800 From: kernel test robot To: Andy Shevchenko , Gene Chen , linux-leds@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org Cc: kbuild-all@lists.01.org, Pavel Machek , Matthias Brugger Subject: Re: [PATCH v1 1/1] leds: mt6360: Get rid of custom led_init_default_state_get() Message-ID: <202208030728.Yo6CynF9-lkp@intel.com> References: <20220802212532.7091-1-andriy.shevchenko@linux.intel.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20220802212532.7091-1-andriy.shevchenko@linux.intel.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220802_164342_217023_A7D0057B X-CRM114-Status: GOOD ( 14.65 ) 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 Andy, I love your patch! Yet something to improve: [auto build test ERROR on pavel-leds/for-next] [also build test ERROR on linus/master v5.19 next-20220728] [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#_base_tree_information] url: https://github.com/intel-lab-lkp/linux/commits/Andy-Shevchenko/leds-mt6360-Get-rid-of-custom-led_init_default_state_get/20220803-052854 base: git://git.kernel.org/pub/scm/linux/kernel/git/pavel/linux-leds.git for-next config: arc-randconfig-r043-20220802 (https://download.01.org/0day-ci/archive/20220803/202208030728.Yo6CynF9-lkp@intel.com/config) compiler: arceb-elf-gcc (GCC) 12.1.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/intel-lab-lkp/linux/commit/260d7c20cbd2b5da92dfa5ffa7c73499661c838f git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Andy-Shevchenko/leds-mt6360-Get-rid-of-custom-led_init_default_state_get/20220803-052854 git checkout 260d7c20cbd2b5da92dfa5ffa7c73499661c838f # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=arc SHELL=/bin/bash drivers/leds/flash/ If you fix the issue, kindly add following tag where applicable Reported-by: kernel test robot All errors (new ones prefixed by >>): drivers/leds/flash/leds-mt6360.c: In function 'mt6360_led_probe': >> drivers/leds/flash/leds-mt6360.c:832:38: error: implicit declaration of function 'led_init_default_state_get'; did you mean 'led_get_default_pattern'? [-Werror=implicit-function-declaration] 832 | led->default_state = led_init_default_state_get(child); | ^~~~~~~~~~~~~~~~~~~~~~~~~~ | led_get_default_pattern cc1: some warnings being treated as errors vim +832 drivers/leds/flash/leds-mt6360.c 770 771 static int mt6360_led_probe(struct platform_device *pdev) 772 { 773 struct mt6360_priv *priv; 774 struct fwnode_handle *child; 775 size_t count; 776 int i = 0, ret; 777 778 count = device_get_child_node_count(&pdev->dev); 779 if (!count || count > MT6360_MAX_LEDS) { 780 dev_err(&pdev->dev, 781 "No child node or node count over max led number %zu\n", 782 count); 783 return -EINVAL; 784 } 785 786 priv = devm_kzalloc(&pdev->dev, 787 struct_size(priv, leds, count), GFP_KERNEL); 788 if (!priv) 789 return -ENOMEM; 790 791 priv->leds_count = count; 792 priv->dev = &pdev->dev; 793 mutex_init(&priv->lock); 794 795 priv->regmap = dev_get_regmap(pdev->dev.parent, NULL); 796 if (!priv->regmap) { 797 dev_err(&pdev->dev, "Failed to get parent regmap\n"); 798 return -ENODEV; 799 } 800 801 device_for_each_child_node(&pdev->dev, child) { 802 struct mt6360_led *led = priv->leds + i; 803 struct led_init_data init_data = { .fwnode = child, }; 804 u32 reg, led_color; 805 806 ret = fwnode_property_read_u32(child, "color", &led_color); 807 if (ret) 808 goto out_flash_release; 809 810 if (led_color == LED_COLOR_ID_RGB || 811 led_color == LED_COLOR_ID_MULTI) 812 reg = MT6360_VIRTUAL_MULTICOLOR; 813 else { 814 ret = fwnode_property_read_u32(child, "reg", ®); 815 if (ret) 816 goto out_flash_release; 817 818 if (reg >= MT6360_MAX_LEDS) { 819 ret = -EINVAL; 820 goto out_flash_release; 821 } 822 } 823 824 if (priv->leds_active & BIT(reg)) { 825 ret = -EINVAL; 826 goto out_flash_release; 827 } 828 priv->leds_active |= BIT(reg); 829 830 led->led_no = reg; 831 led->priv = priv; > 832 led->default_state = led_init_default_state_get(child); 833 834 if (reg == MT6360_VIRTUAL_MULTICOLOR || 835 reg <= MT6360_LED_ISNKML) 836 ret = mt6360_init_isnk_properties(led, &init_data); 837 else 838 ret = mt6360_init_flash_properties(led, &init_data); 839 840 if (ret) 841 goto out_flash_release; 842 843 ret = mt6360_led_register(&pdev->dev, led, &init_data); 844 if (ret) 845 goto out_flash_release; 846 847 i++; 848 } 849 850 platform_set_drvdata(pdev, priv); 851 return 0; 852 853 out_flash_release: 854 mt6360_v4l2_flash_release(priv); 855 return ret; 856 } 857 -- 0-DAY CI Kernel Test Service https://01.org/lkp _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel