Hi, On 7/18/22 19:05, Pandruvada, Srinivas wrote: > Hi Hans, > > There are two minor fixes: > - One for just removing unneeded semicolon > - Another for array range check > > These changes are on top of > https://kernel.googlesource.com/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86/+/refs/heads/for-next > > The following changes since commit > 7a4a04f4e90d746928501a6d551de87d02c28323: > > platform/surface: tabletsw: Fix __le32 integer access (2022-07-17 > 16:20:35 +0200) > > are available in the Git repository at: > > https://github.com/spandruvada/linux-kernel.git intel-sst > > for you to fetch changes up to > 7d565542f8bdc097d611030572ecf8d9706f14dc: > > tools/power/x86/intel-speed-select:Unneeded semicolon (2022-07-18 > 09:51:09 -0700) Thanks. Both commits did not include your Signed-off-by though, if you apply patches of others you must at your S-o-b since you are part of the chain how they got upstream. After pulling I ended up with 2 commits where the S-o-b of the committer of the patch was not present. Instead I've picked both patches directly from the list, now setting me as the committer (and added my own S-o-b). So this is resolved now and both patches are in my review-hans branch now. But the next time when applying patches from others for a pull-req please don't forget to add your own S-o-b. Regards, Hans > > ---------------------------------------------------------------- > Dan Carpenter (1): > tools/power/x86/intel-speed-select: Fix off by one check > > Xin Gao (1): > tools/power/x86/intel-speed-select:Unneeded semicolon > > tools/power/x86/intel-speed-select/hfi-events.c | 2 +- > tools/power/x86/intel-speed-select/isst-daemon.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > Thanks, > Srinivas
