linux-kernel - Re: [PATCH 11/21] peci: remove unneeded 'fast_io' parameter in regmap

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-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <be90ba43-316b-45fa-963d-7faa14b2cb4b@molgen.mpg.de>
Date: Thu, 14 Aug 2025 07:11:03 +0200
From: Paul Menzel <pmenzel@...gen.mpg.de>
To: Wolfram Sang <wsa+renesas@...g-engineering.com>
Cc: linux-kernel@...r.kernel.org, Mark Brown <broonie@...nel.org>,
 Avi Fishman <avifishman70@...il.com>, Tomer Maimon <tmaimon77@...il.com>,
 Tali Perry <tali.perry1@...il.com>, Patrick Venture <venture@...gle.com>,
 Nancy Yuen <yuenn@...gle.com>, Benjamin Fair <benjaminfair@...gle.com>,
 Iwona Winiarska <iwona.winiarska@...el.com>, openbmc@...ts.ozlabs.org
Subject: Re: [PATCH 11/21] peci: remove unneeded 'fast_io' parameter in
 regmap_config

Dear Wolfram,


Thank you for the patch.

Am 13.08.25 um 18:14 schrieb Wolfram Sang:
> When using MMIO with regmap, fast_io is implied. No need to set it
> again.
> 
> Signed-off-by: Wolfram Sang <wsa+renesas@...g-engineering.com>
> ---
> No dependencies, can be applied directly to the subsystem tree. Buildbot is
> happy, too.
> 
>   drivers/peci/controller/peci-npcm.c | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/peci/controller/peci-npcm.c b/drivers/peci/controller/peci-npcm.c
> index c77591ca583d..931868991241 100644
> --- a/drivers/peci/controller/peci-npcm.c
> +++ b/drivers/peci/controller/peci-npcm.c
> @@ -221,7 +221,6 @@ static const struct regmap_config npcm_peci_regmap_config = {
>   	.reg_bits = 8,
>   	.val_bits = 8,
>   	.max_register = NPCM_PECI_MAX_REG,
> -	.fast_io = true,
>   };
>   
>   static const struct peci_controller_ops npcm_ops = {

Reviewed-by: Paul Menzel <pmenzel@...gen.mpg.de>


Kind regards,

Paul