drm/amdgpu/pm: Account for extra separator characters in sysfs interface
authorMatt Coffin <mcoffin13@gmail.com>
Thu, 3 Sep 2020 21:06:14 +0000 (15:06 -0600)
committerAlex Deucher <alexander.deucher@amd.com>
Mon, 1 Feb 2021 16:51:46 +0000 (11:51 -0500)
commitaec1d8703e02426cc2cda618e4d79aa618858298
tree51a02424e949e16e09a1279c72cfd7c15f62bd96
parentce7c1d047629235158b7d6d05874704a0b74b3e5
drm/amdgpu/pm: Account for extra separator characters in sysfs interface

Currently, the the input to the pp_od_clk_voltage sysfs interface has
two adjacent separator characters, i.e. "\n\0", then we try to parse an
argument out of the empty string, whereas we really should just ignore
this case, and treat any number of adjacent separators as one separator
for arguments.

We do this here by simply skipping the argument parsing for all
empy-string arguments.

An example test case would be `echo 's  1 900' > pp_od_clk_voltage`
(note the two spaces after the 's'.

This also solves the issue where the written string ends with both a
newline, and a nul-terminator, '\n\0', as is the case for `echo 's 1
900' > pp_od_clk_voltage` since the recent rebase.

Got lost during code rebase/merge. No need to port this to other
branches.

Reviewed-by: Evan Quan <evan.quan@amd.com>
Signed-off-by: Matt Coffin <mcoffin13@gmail.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/pm/amdgpu_pm.c