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 3F311C54E5D for ; Sat, 16 Mar 2024 09:01:19 +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=Bim882fbmAVM/nYgX0ZfZ70KP3gZOu8802SehFCF0FI=; b=J7RlEHBdeFF8xo nIw54/NOaawBLC9tNSG3W+KTqwR45eYo7H7YpEh/jASf6jkWNK+fXO6rDtz7oZEmelQTOyLFgJDoA 8ZNgNzWUugTonviRqWksjqq+Zfd68ljfkStvJzlIhh+4YXWcXaT+chxeQC+dJ83ooXLAQmuv/Q0c4 PlglFWwZR/dtZXcwJ3s7lgTkq4+8VIhXW46XlwA+b1x45b0CTeHh1L1XbRPxjtXC4QYYgRdHcKpZm 8QDM1rwuQsSlyULlC9dkuTdFYL5Wsog9oITH9mKWJ6Dcj819pKetNigJmNIGZQ6QGtDsxrj4ySE/L gvEuOyuE6MBksP7Wjp7w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rlPuV-00000002o1U-39Cc; Sat, 16 Mar 2024 09:00:59 +0000 Received: from mgamail.intel.com ([198.175.65.18]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rlPuS-00000002o0D-1bxX for linux-arm-kernel@lists.infradead.org; Sat, 16 Mar 2024 09:00:58 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1710579656; x=1742115656; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=9tpnCC2OSnn/XSLWszg6PWWM++RoYjOZSlHEqh0s26Q=; b=XxUVElyp7b+5WIVR0tPmRcdPkQ6bxNt/hEKu/m6nfFMnsC9dacQ0BF9D tMbe6tT4Oyf7Gn3PAuJmK9w2YwBeLaXsnapbxfMYFBiNJ5yWEJ6HsS6TH 321qDSCKBOEsxmshHrRXT42220RAs7zqc+MB874cpmCW+vahoB0ssJH2X 96JdOVPTNKPqQu2lLwRGqqILygdu6bKOdA7/4o6K8PUxbowKQsGNO5JJI BUQcwkIr1PSR4bnu1dXSQiYrzU/p5fR88WcxfeznPGN81L2B8IvnaBmCa 0Rda0e4dPs8t96YK/rz1g6p2qz4+Lt0y1PB1GV72JCcOlbO2NdM/0K7Rx g==; X-IronPort-AV: E=McAfee;i="6600,9927,11014"; a="5588425" X-IronPort-AV: E=Sophos;i="6.07,129,1708416000"; d="scan'208";a="5588425" Received: from fmviesa004.fm.intel.com ([10.60.135.144]) by orvoesa110.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 16 Mar 2024 02:00:46 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.07,129,1708416000"; d="scan'208";a="17527675" Received: from lkp-server01.sh.intel.com (HELO b21307750695) ([10.239.97.150]) by fmviesa004.fm.intel.com with ESMTP; 16 Mar 2024 02:00:40 -0700 Received: from kbuild by b21307750695 with local (Exim 4.96) (envelope-from ) id 1rlPuA-000FFx-26; Sat, 16 Mar 2024 09:00:38 +0000 Date: Sat, 16 Mar 2024 17:00:14 +0800 From: kernel test robot To: Ayush Singh , linux-kernel@vger.kernel.org Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev, Ayush Singh , jkridner@beagleboard.org, robertcnelson@beagleboard.org, Vaishnav M A , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Vignesh Raghavendra , Tero Kristo , Derek Kiernan , Dragan Cvetic , Arnd Bergmann , Greg Kroah-Hartman , Liam Girdwood , Mark Brown , Jiri Slaby , Johan Hovold , Alex Elder , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-spi@vger.kernel.org, linux-serial@vger.kernel.org, greybus-dev@lists.linaro.org Subject: Re: [PATCH v3 7/8] mikrobus: Add mikrobus driver Message-ID: <202403161645.CxE8k6qy-lkp@intel.com> References: <20240315184908.500352-8-ayushdevel1325@gmail.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20240315184908.500352-8-ayushdevel1325@gmail.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240316_020056_566552_B2D2FBF9 X-CRM114-Status: GOOD ( 14.14 ) 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 Ayush, kernel test robot noticed the following build warnings: [auto build test WARNING on 61996c073c9b070922ad3a36c981ca6ddbea19a5] url: https://github.com/intel-lab-lkp/linux/commits/Ayush-Singh/dt-bindings-misc-Add-mikrobus-connector/20240316-025407 base: 61996c073c9b070922ad3a36c981ca6ddbea19a5 patch link: https://lore.kernel.org/r/20240315184908.500352-8-ayushdevel1325%40gmail.com patch subject: [PATCH v3 7/8] mikrobus: Add mikrobus driver config: hexagon-allmodconfig (https://download.01.org/0day-ci/archive/20240316/202403161645.CxE8k6qy-lkp@intel.com/config) compiler: clang version 19.0.0git (https://github.com/llvm/llvm-project 8f68022f8e6e54d1aeae4ed301f5a015963089b7) reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240316/202403161645.CxE8k6qy-lkp@intel.com/reproduce) 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 | Closes: https://lore.kernel.org/oe-kbuild-all/202403161645.CxE8k6qy-lkp@intel.com/ All warnings (new ones prefixed by >>): In file included from drivers/misc/mikrobus/mikrobus_id.c:17: In file included from drivers/misc/mikrobus/mikrobus_core.h:14: In file included from include/linux/i2c.h:19: In file included from include/linux/regulator/consumer.h:35: In file included from include/linux/suspend.h:5: In file included from include/linux/swap.h:9: In file included from include/linux/memcontrol.h:13: In file included from include/linux/cgroup.h:26: In file included from include/linux/kernel_stat.h:9: In file included from include/linux/interrupt.h:11: In file included from include/linux/hardirq.h:11: In file included from ./arch/hexagon/include/generated/asm/hardirq.h:1: In file included from include/asm-generic/hardirq.h:17: In file included from include/linux/irq.h:20: In file included from include/linux/io.h:13: In file included from arch/hexagon/include/asm/io.h:328: include/asm-generic/io.h:547:31: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic] 547 | val = __raw_readb(PCI_IOBASE + addr); | ~~~~~~~~~~ ^ include/asm-generic/io.h:560:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic] 560 | val = __le16_to_cpu((__le16 __force)__raw_readw(PCI_IOBASE + addr)); | ~~~~~~~~~~ ^ include/uapi/linux/byteorder/little_endian.h:37:51: note: expanded from macro '__le16_to_cpu' 37 | #define __le16_to_cpu(x) ((__force __u16)(__le16)(x)) | ^ In file included from drivers/misc/mikrobus/mikrobus_id.c:17: In file included from drivers/misc/mikrobus/mikrobus_core.h:14: In file included from include/linux/i2c.h:19: In file included from include/linux/regulator/consumer.h:35: In file included from include/linux/suspend.h:5: In file included from include/linux/swap.h:9: In file included from include/linux/memcontrol.h:13: In file included from include/linux/cgroup.h:26: In file included from include/linux/kernel_stat.h:9: In file included from include/linux/interrupt.h:11: In file included from include/linux/hardirq.h:11: In file included from ./arch/hexagon/include/generated/asm/hardirq.h:1: In file included from include/asm-generic/hardirq.h:17: In file included from include/linux/irq.h:20: In file included from include/linux/io.h:13: In file included from arch/hexagon/include/asm/io.h:328: include/asm-generic/io.h:573:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic] 573 | val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr)); | ~~~~~~~~~~ ^ include/uapi/linux/byteorder/little_endian.h:35:51: note: expanded from macro '__le32_to_cpu' 35 | #define __le32_to_cpu(x) ((__force __u32)(__le32)(x)) | ^ In file included from drivers/misc/mikrobus/mikrobus_id.c:17: In file included from drivers/misc/mikrobus/mikrobus_core.h:14: In file included from include/linux/i2c.h:19: In file included from include/linux/regulator/consumer.h:35: In file included from include/linux/suspend.h:5: In file included from include/linux/swap.h:9: In file included from include/linux/memcontrol.h:13: In file included from include/linux/cgroup.h:26: In file included from include/linux/kernel_stat.h:9: In file included from include/linux/interrupt.h:11: In file included from include/linux/hardirq.h:11: In file included from ./arch/hexagon/include/generated/asm/hardirq.h:1: In file included from include/asm-generic/hardirq.h:17: In file included from include/linux/irq.h:20: In file included from include/linux/io.h:13: In file included from arch/hexagon/include/asm/io.h:328: include/asm-generic/io.h:584:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic] 584 | __raw_writeb(value, PCI_IOBASE + addr); | ~~~~~~~~~~ ^ include/asm-generic/io.h:594:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic] 594 | __raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr); | ~~~~~~~~~~ ^ include/asm-generic/io.h:604:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic] 604 | __raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr); | ~~~~~~~~~~ ^ In file included from drivers/misc/mikrobus/mikrobus_id.c:17: In file included from drivers/misc/mikrobus/mikrobus_core.h:14: In file included from include/linux/i2c.h:19: In file included from include/linux/regulator/consumer.h:35: In file included from include/linux/suspend.h:5: In file included from include/linux/swap.h:9: In file included from include/linux/memcontrol.h:21: In file included from include/linux/mm.h:2208: include/linux/vmstat.h:522:36: warning: arithmetic between different enumeration types ('enum node_stat_item' and 'enum lru_list') [-Wenum-enum-conversion] 522 | return node_stat_name(NR_LRU_BASE + lru) + 3; // skip "nr_" | ~~~~~~~~~~~ ^ ~~~ >> drivers/misc/mikrobus/mikrobus_id.c:45:60: warning: format specifies type 'unsigned long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat] 45 | pr_info("mikrobus_id: writing manifest size = %lu bytes", count); | ~~~ ^~~~~ | %zu include/linux/printk.h:530:34: note: expanded from macro 'pr_info' 530 | printk(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__) | ~~~ ^~~~~~~~~~~ include/linux/printk.h:457:60: note: expanded from macro 'printk' 457 | #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__) | ~~~ ^~~~~~~~~~~ include/linux/printk.h:429:19: note: expanded from macro 'printk_index_wrap' 429 | _p_func(_fmt, ##__VA_ARGS__); \ | ~~~~ ^~~~~~~~~~~ 8 warnings generated. vim +45 drivers/misc/mikrobus/mikrobus_id.c 28 29 static ssize_t mikrobus_manifest_store(struct device *device, struct device_attribute *attr, 30 const char *buf, size_t count) 31 { 32 u8 write_request[] = { W1_MIKROBUS_ID_WRITE_EEPROM, 33 W1_MIKROBUS_EEPROM_MANIFEST_START_PAGE }; 34 u8 release_command = W1_MIKROBUS_ID_RELEASE_EEPROM; 35 struct w1_slave *sl = dev_to_w1_slave(device); 36 u16 crc, crc_read, pos = 0; 37 u8 status = 0; 38 int cnt; 39 40 if (count > W1_MIKROBUS_ID_EEPROM_SIZE) 41 return -ENOMEM; 42 43 mutex_lock(&sl->master->bus_mutex); 44 > 45 pr_info("mikrobus_id: writing manifest size = %lu bytes", count); 46 while (pos < count) { 47 if (w1_reset_select_slave(sl)) 48 break; 49 50 w1_write_block(sl->master, write_request, sizeof(write_request)); 51 crc = crc16(0, write_request, sizeof(write_request)) ^ 0xFFFF; 52 w1_read_block(sl->master, (u8 *)&crc_read, sizeof(crc_read)); 53 54 if (crc != crc_read) 55 break; 56 57 for (cnt = 0; cnt < W1_MIKROBUS_ID_EEPROM_PAGE_SIZE; cnt++) 58 w1_write_8(sl->master, (u8)buf[cnt]); 59 60 crc = crc16(0, buf, W1_MIKROBUS_ID_EEPROM_PAGE_SIZE) ^ 0xFFFF; 61 usleep_range(1 * USEC_PER_MSEC, 2 * USEC_PER_MSEC); 62 w1_read_block(sl->master, (u8 *)&crc_read, sizeof(crc_read)); 63 64 if (crc != crc_read) 65 break; 66 67 w1_write_8(sl->master, release_command); 68 usleep_range(10 * USEC_PER_MSEC, 15 * USEC_PER_MSEC); 69 status = w1_read_8(sl->master); 70 w1_read_block(sl->master, (u8 *)&crc_read, sizeof(crc_read)); 71 crc = crc16(0, (u8 *)&release_command, sizeof(release_command)) ^ 0xFFFF; 72 73 if (status != W1_MIKROBUS_ID_RELEASE_EEPROM) 74 break; 75 76 if (crc != crc_read) 77 break; 78 79 buf += W1_MIKROBUS_ID_EEPROM_PAGE_SIZE; 80 pos += W1_MIKROBUS_ID_EEPROM_PAGE_SIZE; 81 write_request[1]++; 82 } 83 84 pr_info("mikrobus_id: manifest written bytes: %d", pos); 85 mutex_unlock(&sl->master->bus_mutex); 86 87 return count > pos ? count : pos; 88 } 89 static DEVICE_ATTR_WO(mikrobus_manifest); 90 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel