2015-12-17 03:34:59

by Yu, Xiangliang

[permalink] [raw]
Subject: [PATCH 2/3] NTB: Add AMD NTB support in Kconfig and Makefile

This patch is to enable AMD NTB support in Kconfig and Makefile.

Signed-off-by: Xiangliang Yu <[email protected]>
---
drivers/ntb/hw/Kconfig | 1 +
drivers/ntb/hw/Makefile | 1 +
2 files changed, 2 insertions(+)

diff --git a/drivers/ntb/hw/Kconfig b/drivers/ntb/hw/Kconfig
index 4d5535c..0c5c2a6 100644
--- a/drivers/ntb/hw/Kconfig
+++ b/drivers/ntb/hw/Kconfig
@@ -1 +1,2 @@
source "drivers/ntb/hw/intel/Kconfig"
+source "drivers/ntb/hw/amd/Kconfig"
diff --git a/drivers/ntb/hw/Makefile b/drivers/ntb/hw/Makefile
index 175d7c9..636be7d 100644
--- a/drivers/ntb/hw/Makefile
+++ b/drivers/ntb/hw/Makefile
@@ -1 +1,2 @@
obj-$(CONFIG_NTB_INTEL) += intel/
+obj-$(CONFIG_NTB_AMD) += amd/
--
1.9.1


2015-12-17 11:02:27

by kernel test robot

[permalink] [raw]
Subject: Re: [PATCH 2/3] NTB: Add AMD NTB support in Kconfig and Makefile

Hi Xiangliang,

[auto build test WARNING on v4.4-rc5]
[also build test WARNING on next-20151217]

url: https://github.com/0day-ci/linux/commits/Xiangliang-Yu/NTB-Add-AMD-PCI-Express-NTB-driver/20151217-113608
config: x86_64-randconfig-n0-12171710 (attached as .config)
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64

Note: it may well be a FALSE warning. FWIW you are at least aware of it now.
http://gcc.gnu.org/wiki/Better_Uninitialized_Warnings

All warnings (new ones prefixed by >>):

drivers/ntb/hw/amd/ntb_hw_amd.c:486:12: warning: 'amd_flush_peer_requests' defined but not used [-Wunused-function]
static int amd_flush_peer_requests(struct amd_ntb_dev *ndev)
^
drivers/ntb/hw/amd/ntb_hw_amd.c:507:12: warning: 'amd_wakeup_peer_side' defined but not used [-Wunused-function]
static int amd_wakeup_peer_side(struct amd_ntb_dev *ndev)
^
drivers/ntb/hw/amd/ntb_hw_amd.c: In function 'amd_ntb_mw_set_trans':
>> drivers/ntb/hw/amd/ntb_hw_amd.c:187:6: warning: 'limit' may be used uninitialized in this function [-Wmaybe-uninitialized]
if (reg_val != limit) {
^

vim +/limit +187 drivers/ntb/hw/amd/ntb_hw_amd.c

a635755a Xiangliang Yu 2015-12-17 171 if (bar != 1) {
a635755a Xiangliang Yu 2015-12-17 172 /* Set the limit if supported */
a635755a Xiangliang Yu 2015-12-17 173 if (limit_reg)
a635755a Xiangliang Yu 2015-12-17 174 limit = base_addr + size;
a635755a Xiangliang Yu 2015-12-17 175
a635755a Xiangliang Yu 2015-12-17 176 /* set and verify setting the translation address */
a635755a Xiangliang Yu 2015-12-17 177 iowrite64(addr, peer_mmio + xlat_reg);
a635755a Xiangliang Yu 2015-12-17 178 reg_val = ioread64(peer_mmio + xlat_reg);
a635755a Xiangliang Yu 2015-12-17 179 if (reg_val != addr) {
a635755a Xiangliang Yu 2015-12-17 180 iowrite64(0, peer_mmio + xlat_reg);
a635755a Xiangliang Yu 2015-12-17 181 return -EIO;
a635755a Xiangliang Yu 2015-12-17 182 }
a635755a Xiangliang Yu 2015-12-17 183
a635755a Xiangliang Yu 2015-12-17 184 /* set and verify setting the limit */
a635755a Xiangliang Yu 2015-12-17 185 iowrite64(limit, mmio + limit_reg);
a635755a Xiangliang Yu 2015-12-17 186 reg_val = ioread64(mmio + limit_reg);
a635755a Xiangliang Yu 2015-12-17 @187 if (reg_val != limit) {
a635755a Xiangliang Yu 2015-12-17 188 iowrite64(base_addr, mmio + limit_reg);
a635755a Xiangliang Yu 2015-12-17 189 iowrite64(0, peer_mmio + xlat_reg);
a635755a Xiangliang Yu 2015-12-17 190 return -EIO;
a635755a Xiangliang Yu 2015-12-17 191 }
a635755a Xiangliang Yu 2015-12-17 192 } else {
a635755a Xiangliang Yu 2015-12-17 193 /* split bar addr range must all be 32 bit */
a635755a Xiangliang Yu 2015-12-17 194 if (addr & (~0ull << 32))
a635755a Xiangliang Yu 2015-12-17 195 return -EINVAL;

:::::: The code at line 187 was first introduced by commit
:::::: a635755a710138a6f505cc7e62693741786fb19e NTB: Add AMD PCI-Express NTB driver

:::::: TO: Xiangliang Yu <[email protected]>
:::::: CC: 0day robot <[email protected]>

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation


Attachments:
(No filename) (3.23 kB)
.config.gz (28.02 kB)
Download all attachments