diff options
author | Robert Richter <rrichter@amd.com> | 2022-12-05 23:21:40 +0300 |
---|---|---|
committer | Dan Williams <dan.j.williams@intel.com> | 2022-12-06 02:37:09 +0300 |
commit | d3cdf4585f2f9122d1165acca40e801c75afa320 (patch) | |
tree | f4033fd4446d34237aff8db2434e87a98a495743 /drivers/cxl/acpi.c | |
parent | 878b2de67af7095ddd35da859a9ef8db15a36163 (diff) | |
download | linux-d3cdf4585f2f9122d1165acca40e801c75afa320.tar.xz |
cxl/acpi: Warn about an invalid CHBCR in an existing CHBS entry
After parsing for a CHBCR in cxl_get_chbcr() the case of (ctx.chbcr ==
CXL_RESOURCE_NONE) is a slighly different error reason than the
!ctx.chbcr case. In the first case the CHBS was found but the CHBCR
was invalid or something else failed to determine it, while in the
latter case no CHBS entry exists at all.
Update the warning message to reflect this. The log messages for both
cases can be differentiated now and the reason for a failure can be
determined better.
Signed-off-by: Robert Richter <rrichter@amd.com>
Reviewed-by: Dave Jiang <dave.jiang@intel.com>
Link: https://lore.kernel.org/r/167027170051.3542509.10494781536638424397.stgit@dwillia2-xfh.jf.intel.com
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'drivers/cxl/acpi.c')
-rw-r--r-- | drivers/cxl/acpi.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/cxl/acpi.c b/drivers/cxl/acpi.c index 420e322c85a1..f495f3ce45d1 100644 --- a/drivers/cxl/acpi.c +++ b/drivers/cxl/acpi.c @@ -455,7 +455,8 @@ static int add_host_bridge_dport(struct device *match, void *arg) dev_dbg(match, "RCRB found for UID %lld: %pa\n", uid, &ctx.rcrb); if (ctx.chbcr == CXL_RESOURCE_NONE) { - dev_warn(match, "No CHBS found for Host Bridge (UID %lld)\n", uid); + dev_warn(match, "CHBCR invalid for Host Bridge (UID %lld)\n", + uid); return 0; } |