[PATCH dev-5.4] hwmon: peci: change label strings to reflect natural numbers

Jae Hyun Yoo jae.hyun.yoo at linux.intel.com
Thu Feb 27 09:13:18 AEDT 2020


On 2/26/2020 2:07 PM, Jae Hyun Yoo wrote:
> Hi Joel,
> 
> On 2/26/2020 1:54 PM, Joel Stanley wrote:
>> On Tue, 11 Feb 2020 at 23:47, Jae Hyun Yoo 
>> <jae.hyun.yoo at linux.intel.com> wrote:
>>>
>>> This commit changes label strings to reflect user friendly natural
>>> numbers like 'Core 1' instead of 'Core 0' and 'DIMM A1' instead of
>>> 'DIMM A0'.
>>>
>>> Signed-off-by: Jae Hyun Yoo <jae.hyun.yoo at linux.intel.com>
>>> ---
>>>   drivers/hwmon/peci-cputemp.c  | 2 +-
>>>   drivers/hwmon/peci-dimmtemp.c | 2 +-
>>>   2 files changed, 2 insertions(+), 2 deletions(-)
>>>   mode change 100644 => 100755 drivers/hwmon/peci-cputemp.c
>>>   mode change 100644 => 100755 drivers/hwmon/peci-dimmtemp.c
>>
>> I don't think this was intended.
>>
>> I fixed it up for you, but please try to figure out what went wrong so
>> it doesn't happen in the future.
> 
> It happened while upstreaming it. I changed the static label string
> table as a dynamic table to address review comments, and I missed
> this mismatch. My bad. It'll be fixed in the next upstreaming spin.

Oh, you pointed out the file mode '755'. Sorry, I changed my samba
configuration so it'll not be happened again.

Thanks :)

Jae

>>>
>>> diff --git a/drivers/hwmon/peci-cputemp.c b/drivers/hwmon/peci-cputemp.c
>>> old mode 100644
>>> new mode 100755
>>> index 78e442f433a7..b9fe91281d58
>>> --- a/drivers/hwmon/peci-cputemp.c
>>> +++ b/drivers/hwmon/peci-cputemp.c
>>> @@ -363,7 +363,7 @@ static int create_core_temp_label(struct 
>>> peci_cputemp *priv, int idx)
>>>          if (!priv->coretemp_label[idx])
>>>                  return -ENOMEM;
>>>
>>> -       sprintf(priv->coretemp_label[idx], "Core %d", idx);
>>> +       sprintf(priv->coretemp_label[idx], "Core %d", idx + 1);
>>>
>>>          return 0;
>>>   }
>>> diff --git a/drivers/hwmon/peci-dimmtemp.c 
>>> b/drivers/hwmon/peci-dimmtemp.c
>>> old mode 100644
>>> new mode 100755
>>> index 8ceab08d06e1..45eabd2ec5c8
>>> --- a/drivers/hwmon/peci-dimmtemp.c
>>> +++ b/drivers/hwmon/peci-dimmtemp.c
>>> @@ -260,7 +260,7 @@ static int create_dimm_temp_label(struct 
>>> peci_dimmtemp *priv, int chan)
>>>          rank = chan / priv->gen_info->dimm_idx_max;
>>>          idx = chan % priv->gen_info->dimm_idx_max;
>>>
>>> -       sprintf(priv->dimmtemp_label[chan], "DIMM %c%d", 'A' + rank, 
>>> idx);
>>> +       sprintf(priv->dimmtemp_label[chan], "DIMM %c%d", 'A' + rank, 
>>> idx + 1);
>>>
>>>          return 0;
>>>   }
>>> -- 
>>> 2.17.1
>>>


More information about the openbmc mailing list