diff mbox series

[v1,1/8] hwmon: (adt7410) Pass name to core driver

Message ID 20211221201113.752827-1-demonsingur@gmail.com (mailing list archive)
State Rejected
Headers show
Series [v1,1/8] hwmon: (adt7410) Pass name to core driver | expand

Commit Message

Cosmin Tanislav Dec. 21, 2021, 8:11 p.m. UTC
From: Cosmin Tanislav <cosmin.tanislav@analog.com>

It will later be used to register hwmon device using
hwmon_device_register_with_info.

Signed-off-by: Cosmin Tanislav <cosmin.tanislav@analog.com>
---
 drivers/hwmon/adt7410.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Guenter Roeck Dec. 21, 2021, 8:47 p.m. UTC | #1
On 12/21/21 12:11 PM, Cosmin Tanislav wrote:
> From: Cosmin Tanislav <cosmin.tanislav@analog.com>
> 
> It will later be used to register hwmon device using
> hwmon_device_register_with_info.
> 
> Signed-off-by: Cosmin Tanislav <cosmin.tanislav@analog.com>
> ---
>   drivers/hwmon/adt7410.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/hwmon/adt7410.c b/drivers/hwmon/adt7410.c
> index 973db057427b..ef4b5af865e9 100644
> --- a/drivers/hwmon/adt7410.c
> +++ b/drivers/hwmon/adt7410.c
> @@ -45,7 +45,7 @@ static int adt7410_i2c_probe(struct i2c_client *client)
>   			I2C_FUNC_SMBUS_BYTE_DATA | I2C_FUNC_SMBUS_WORD_DATA))
>   		return -ENODEV;
>   
> -	return adt7x10_probe(&client->dev, NULL, client->irq, &adt7410_i2c_ops);
> +	return adt7x10_probe(&client->dev, client->name, client->irq, &adt7410_i2c_ops);
>   }
>   
>   static int adt7410_i2c_remove(struct i2c_client *client)
> 

This series looks very similar to the previous series, yet there is
no change log, and it again claims to be v1 of a patch series. Either
explain why this series is so different to the similar looking patch
series starting with "hwmon: adt7x10: store bus_dev in private data",
that it doesn't deserve to be called v2, or provide change logs and
version your series.

If you expect me to figure out what changed, sorry, that is not how
it works. I won't even look at this series.

Guenter
Cosmin Tanislav Dec. 21, 2021, 8:58 p.m. UTC | #2
On 12/21/21 22:47, Guenter Roeck wrote:
> On 12/21/21 12:11 PM, Cosmin Tanislav wrote:
>> From: Cosmin Tanislav <cosmin.tanislav@analog.com>
>>
>> It will later be used to register hwmon device using
>> hwmon_device_register_with_info.
>>
>> Signed-off-by: Cosmin Tanislav <cosmin.tanislav@analog.com>
>> ---
>>   drivers/hwmon/adt7410.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/hwmon/adt7410.c b/drivers/hwmon/adt7410.c
>> index 973db057427b..ef4b5af865e9 100644
>> --- a/drivers/hwmon/adt7410.c
>> +++ b/drivers/hwmon/adt7410.c
>> @@ -45,7 +45,7 @@ static int adt7410_i2c_probe(struct i2c_client *client)
>>               I2C_FUNC_SMBUS_BYTE_DATA | I2C_FUNC_SMBUS_WORD_DATA))
>>           return -ENODEV;
>> -    return adt7x10_probe(&client->dev, NULL, client->irq, 
>> &adt7410_i2c_ops);
>> +    return adt7x10_probe(&client->dev, client->name, client->irq, 
>> &adt7410_i2c_ops);
>>   }
>>   static int adt7410_i2c_remove(struct i2c_client *client)
>>
> 
> This series looks very similar to the previous series, yet there is
> no change log, and it again claims to be v1 of a patch series. Either
> explain why this series is so different to the similar looking patch
> series starting with "hwmon: adt7x10: store bus_dev in private data",
> that it doesn't deserve to be called v2, or provide change logs and
> version your series.
> 
> If you expect me to figure out what changed, sorry, that is not how
> it works. I won't even look at this series.
> 
> Guenter

I'm sorry, I was in a rush to push the new patchset and I mistyped the
format-patch command and then I also forgot to add a cover letter.
I'll send the proper patches in a few.
diff mbox series

Patch

diff --git a/drivers/hwmon/adt7410.c b/drivers/hwmon/adt7410.c
index 973db057427b..ef4b5af865e9 100644
--- a/drivers/hwmon/adt7410.c
+++ b/drivers/hwmon/adt7410.c
@@ -45,7 +45,7 @@  static int adt7410_i2c_probe(struct i2c_client *client)
 			I2C_FUNC_SMBUS_BYTE_DATA | I2C_FUNC_SMBUS_WORD_DATA))
 		return -ENODEV;
 
-	return adt7x10_probe(&client->dev, NULL, client->irq, &adt7410_i2c_ops);
+	return adt7x10_probe(&client->dev, client->name, client->irq, &adt7410_i2c_ops);
 }
 
 static int adt7410_i2c_remove(struct i2c_client *client)