Commit afd43fb2 authored by Xiaofei Tan's avatar Xiaofei Tan Committed by Junhao He
Browse files

soc: hisilicon: Fix one printing issue and simplify some printing 

driver inclusion
category: bugfix
bugzilla: https://gitee.com/openeuler/kernel/issues/IBS9WQ


CVE: NA

----------------------------------------------------------------------

It's the opposite of what acls_repair_store printing really means.
Fix it and add some printing simplification.

Fixes: efb5bc7b ("soc: hisilicon: Support memory repair driver on Kunpeng SoC")
Signed-off-by: default avatarXiaofei Tan <tanxiaofei@huawei.com>
Signed-off-by: default avatarQizhi Zhang <zhangqizhi3@h-partners.com>
parent ed789a9f

drivers/soc/hisilicon/hisi_mem_ras.c

+3 −3
Original line number Diff line number Diff line
@@ -229,7 +229,7 @@ static struct page *trans_and_check_paddr(struct device *dev, const char *buf, u 


	page = pfn_to_online_page(__phys_to_pfn(paddr_tmp));

	if (!page) {

		dev_info(dev, "The page of paddr 0x%llx is not online page.\n", paddr_tmp);

		dev_info(dev, "The page of paddr 0x%llx not online.\n", paddr_tmp);

		return NULL;

	}
@@ -283,7 +283,7 @@ static ssize_t acls_repair_store(struct device *dev, struct device_attribute *at 
		return -EINVAL;



	if (hdev->mem_type == MEMORY_TYPE_HBM && !PageHWPoison(page)) {

		dev_info(dev, "The page of paddr 0x%llx is not unpoisoned.\n", paddr);

		dev_info(dev, "The page of paddr 0x%llx not poisoned.\n", paddr);

		return -EIO;

	}
@@ -350,7 +350,7 @@ static ssize_t sppr_repair_store(struct device *dev, struct device_attribute *at 
		return -EINVAL;



	if (hdev->mem_type == MEMORY_TYPE_HBM && !PageHWPoison(page)) {

		dev_info(dev, "The page of paddr 0x%llx is not poisoned.\n", paddr);

		dev_info(dev, "The page of paddr 0x%llx not poisoned.\n", paddr);

		return -EIO;

	}