Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Wed, 29 Mar 2023 09:17:51 +0200
From:   Johan Hovold <johan@kernel.org>
To:     Guiting Shen <aarongt.shen@gmail.com>
Cc:     lee@kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] mfd:i2c: remove redundant dev_set_drvdata() function
Message-ID: <ZCPmH0HnqOYp4hpG@hovoldconsulting.com>
References: <20230329055724.43998-1-aarongt.shen@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20230329055724.43998-1-aarongt.shen@gmail.com>
Precedence: bulk

On Wed, Mar 29, 2023 at 01:57:24PM +0800, Guiting Shen wrote:
> the i2c_set_clientdata() is the inline function which is
> complemented by the dev_set_drvdata() internally. Do not
> need to use the i2c_set_clientdata() and dev_set_drvdata()
> at the same time.

In the future please break your commit messages at 72 columns or so (not
60).

Also the Subject should have a space after each colon (or that should
have been a slash, but in this case you should probably just drop the
i2c bit) and it's really not the function that is redundant but the fact
that it is called. Perhaps something like:

	mfd: remove redundant dev_set_drvdata() from i2c drivers
	
> Signed-off-by: Guiting Shen <aarongt.shen@gmail.com>
> ---
>  drivers/mfd/88pm80x.c       | 1 -
>  drivers/mfd/88pm860x-core.c | 1 -
>  drivers/mfd/aat2870-core.c  | 2 --
>  drivers/mfd/lm3533-core.c   | 2 --
>  drivers/mfd/max8907.c       | 2 --
>  drivers/mfd/max8925-i2c.c   | 1 -
>  drivers/mfd/wm8400-core.c   | 2 --
>  drivers/mfd/wm8994-core.c   | 2 --
>  8 files changed, 13 deletions(-)

Looks good to me otherwise:

Reviewed-by: Johan Hovold <johan@kernel.org>

Johan