linux-kernel - [PATCH 1/1] regmap: cache: downgrade log level for no cache defaults message

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-Id: <20220921071054.3132504-1-alexander.stein@ew.tq-group.com>
Date:   Wed, 21 Sep 2022 09:10:54 +0200
From:   Alexander Stein <alexander.stein@...tq-group.com>
To:     Mark Brown <broonie@...nel.org>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        "Rafael J . Wysocki" <rafael@...nel.org>
Cc:     Alexander Stein <alexander.stein@...tq-group.com>,
        linux-kernel@...r.kernel.org
Subject: [PATCH 1/1] regmap: cache: downgrade log level for no cache defaults message

Having no cache defaults is not a mistake, especially for devices
which do have some kind of strapping values which can't be provided
statically. For this case a warning is the wrong log level, the initial
cache values do need to be read form hardware. Reduce level to info.

Signed-off-by: Alexander Stein <alexander.stein@...tq-group.com>
---
 drivers/base/regmap/regcache.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/base/regmap/regcache.c b/drivers/base/regmap/regcache.c
index 362e043e26d8..89b92eda8b02 100644
--- a/drivers/base/regmap/regcache.c
+++ b/drivers/base/regmap/regcache.c
@@ -54,7 +54,7 @@ static int regcache_hw_init(struct regmap *map)
 
 	if (!map->reg_defaults_raw) {
 		bool cache_bypass = map->cache_bypass;
-		dev_warn(map->dev, "No cache defaults, reading back from HW\n");
+		dev_info(map->dev, "No cache defaults, reading back from HW\n");
 
 		/* Bypass the cache access till data read from HW */
 		map->cache_bypass = true;
-- 
2.25.1