[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