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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id E2A03C61DA4 for ; Fri, 3 Feb 2023 23:44:38 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233034AbjBCXoh (ORCPT ); Fri, 3 Feb 2023 18:44:37 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50174 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232932AbjBCXoc (ORCPT ); Fri, 3 Feb 2023 18:44:32 -0500 Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5110641098; Fri, 3 Feb 2023 15:44:31 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1675467871; x=1707003871; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=uZ0qNhDCqz4L2s+QoHLo0NAhbxb3fK+YiXc/q8MtHUA=; b=G4aWql6NAzhhEtozfewYvinxO9F7kDWeKkQrj37EEonNBoFUwoUfVJE+ oGF3m2T42Tvmsf1bOT11+v/zUwg5iKrliTjm4BH4qtlIiDvPDUmj6Gqya P4GwZC2KJ5CQ8+rAxwcvGzLolbAj17rGDUxM08g04GB+KDtHv8OZRAGL2 dgSW2MiRxqhY+nzDOc18NP6xUr1cY+4lfy4bMG+cv9bT4nO+yD2WMAEzT cCoSWKpchKYi/kjJ9WmM9kz3YhE81aIWtwwNC5klJkCOI3SIn8E5jJG1G GRqeAPxrSG7g7sLp65ZDERpepMi94fIdQwI8p8XIPATx40+sYAQalfXEA A==; X-IronPort-AV: E=McAfee;i="6500,9779,10610"; a="331016972" X-IronPort-AV: E=Sophos;i="5.97,271,1669104000"; d="scan'208";a="331016972" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 03 Feb 2023 15:44:30 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6500,9779,10610"; a="698243366" X-IronPort-AV: E=Sophos;i="5.97,271,1669104000"; d="scan'208";a="698243366" Received: from lkp-server01.sh.intel.com (HELO 4455601a8d94) ([10.239.97.150]) by orsmga001.jf.intel.com with ESMTP; 03 Feb 2023 15:44:26 -0800 Received: from kbuild by 4455601a8d94 with local (Exim 4.96) (envelope-from ) id 1pO5jA-0000qp-2x; Fri, 03 Feb 2023 23:44:20 +0000 Date: Sat, 4 Feb 2023 07:43:25 +0800 From: kernel test robot To: Krishna Yarlagadda , robh+dt@kernel.org, broonie@kernel.org, peterhuewe@gmx.de, jgg@ziepe.ca, jarkko@kernel.org, krzysztof.kozlowski+dt@linaro.org, linux-spi@vger.kernel.org, linux-tegra@vger.kernel.org, linux-integrity@vger.kernel.org, linux-kernel@vger.kernel.org Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev, thierry.reding@gmail.com, jonathanh@nvidia.com, skomatineni@nvidia.com, ldewangan@nvidia.com, Krishna Yarlagadda Subject: Re: [Patch V2 2/4] tpm: tegra: Support SPI tpm wait state detect Message-ID: <202302040739.VbsIprzH-lkp@intel.com> References: <20230203130133.32901-3-kyarlagadda@nvidia.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230203130133.32901-3-kyarlagadda@nvidia.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Krishna, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on char-misc/char-misc-testing] [also build test WARNING on char-misc/char-misc-next char-misc/char-misc-linus broonie-spi/for-next robh/for-next linus/master v6.2-rc6 next-20230203] [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/Krishna-Yarlagadda/dt-bindings-tpm-Add-compatible-for-Tegra-TPM/20230203-210314 patch link: https://lore.kernel.org/r/20230203130133.32901-3-kyarlagadda%40nvidia.com patch subject: [Patch V2 2/4] tpm: tegra: Support SPI tpm wait state detect config: x86_64-randconfig-k001 (https://download.01.org/0day-ci/archive/20230204/202302040739.VbsIprzH-lkp@intel.com/config) compiler: clang version 14.0.6 (https://github.com/llvm/llvm-project f28c006a5895fc0e329fe15fead81e37457cb1d1) 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/825363f7e8d0d426c45bbba6cb3c5d9b79b7e6aa git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Krishna-Yarlagadda/dt-bindings-tpm-Add-compatible-for-Tegra-TPM/20230203-210314 git checkout 825363f7e8d0d426c45bbba6cb3c5d9b79b7e6aa # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=x86_64 olddefconfig COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/char/tpm/ If you fix the issue, kindly add following tag where applicable | Reported-by: kernel test robot All warnings (new ones prefixed by >>): In file included from drivers/char/tpm/tpm_tis_spi_tegra.c:18: In file included from drivers/char/tpm/tpm_tis_core.h:22: In file included from drivers/char/tpm/tpm.h:28: include/linux/tpm_eventlog.h:167:6: warning: variable 'mapping_size' set but not used [-Wunused-but-set-variable] int mapping_size; ^ >> drivers/char/tpm/tpm_tis_spi_tegra.c:23:5: warning: no previous prototype for function 'tpm_tis_spi_tegra_transfer' [-Wmissing-prototypes] int tpm_tis_spi_tegra_transfer(struct tpm_tis_data *data, u32 addr, u16 len, ^ drivers/char/tpm/tpm_tis_spi_tegra.c:23:1: note: declare 'static' if the function is not intended to be used outside of this translation unit int tpm_tis_spi_tegra_transfer(struct tpm_tis_data *data, u32 addr, u16 len, ^ static 2 warnings generated. vim +/tpm_tis_spi_tegra_transfer +23 drivers/char/tpm/tpm_tis_spi_tegra.c 22 > 23 int tpm_tis_spi_tegra_transfer(struct tpm_tis_data *data, u32 addr, u16 len, 24 u8 *in, const u8 *out) 25 { 26 struct tpm_tis_spi_phy *phy = to_tpm_tis_spi_phy(data); 27 int ret = 0; 28 struct spi_message m; 29 struct spi_transfer spi_xfer[3]; 30 u8 transfer_len; 31 32 spi_bus_lock(phy->spi_device->master); 33 34 while (len) { 35 transfer_len = min_t(u16, len, MAX_SPI_FRAMESIZE); 36 37 spi_message_init(&m); 38 phy->iobuf[0] = (in ? 0x80 : 0) | (transfer_len - 1); 39 phy->iobuf[1] = 0xd4; 40 phy->iobuf[2] = addr >> 8; 41 phy->iobuf[3] = addr; 42 43 memset(&spi_xfer, 0, sizeof(spi_xfer)); 44 45 spi_xfer[0].tx_buf = phy->iobuf; 46 spi_xfer[0].len = 1; 47 spi_message_add_tail(&spi_xfer[0], &m); 48 49 spi_xfer[1].tx_buf = phy->iobuf + 1; 50 spi_xfer[1].len = 3; 51 spi_message_add_tail(&spi_xfer[1], &m); 52 53 if (out) { 54 spi_xfer[2].tx_buf = &phy->iobuf[4]; 55 spi_xfer[2].rx_buf = NULL; 56 memcpy(&phy->iobuf[4], out, transfer_len); 57 out += transfer_len; 58 } 59 if (in) { 60 spi_xfer[2].tx_buf = NULL; 61 spi_xfer[2].rx_buf = &phy->iobuf[4]; 62 } 63 spi_xfer[2].len = transfer_len; 64 spi_message_add_tail(&spi_xfer[2], &m); 65 66 reinit_completion(&phy->ready); 67 ret = spi_sync_locked(phy->spi_device, &m); 68 if (ret < 0) 69 goto exit; 70 71 if (in) { 72 memcpy(in, &phy->iobuf[4], transfer_len); 73 in += transfer_len; 74 } 75 76 len -= transfer_len; 77 } 78 79 exit: 80 spi_bus_unlock(phy->spi_device->master); 81 return ret; 82 } 83 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests