lkml.org 
[lkml]   [2023]   [Mar]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] mfd:i2c: remove redundant dev_set_drvdata() function
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

\
 
 \ /
  Last update: 2023-03-29 09:18    [W:0.162 / U:0.036 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site