ACK/Cmnt: [SRU][J:linux-bluefield][PATCH 0/1] UBUNTU: SAUCE: mlxbf-ptm: power and thermal management debugfs driver

[Tim Gardner]

On 1/16/23 8:12 AM, Jitendra Lanka wrote:
> BugLink: https://bugs.launchpad.net/bugs/2002852
> 
> SRU Justification:
> 
> [Impact]
> 
> mlxbf-ptm is a kernel driver that provides debufgs interface for system software to monitor Bluefield devices' power and thermal management parameters.
> 
> [Fix]
> 
> * New driver added
> 
> [Test Case]
> 
> * After installing the kernel module, debugfs entry at /sys/kernel/debug/mlxbf-ptm will be created
> * monitors/status folder contains read-only parameters reported from the Bluefield device
> 
> [Regression Potential]
> 
> * Minimal - as these are read-only parameters that report Bluefield device information.
> 
> [Other]
> * This code is likely to change depending on feedback we received from maintainers.
> 
> Jitendra Lanka (1):
>    mlxbf-ptm: power and thermal management debugfs driver
> 
>   debian.bluefield/config/config.common.ubuntu |   1 +
>   drivers/platform/mellanox/Kconfig            |  10 +
>   drivers/platform/mellanox/Makefile           |   1 +
>   drivers/platform/mellanox/mlxbf-ptm.c        | 215 +++++++++++++++++++
>   4 files changed, 227 insertions(+)
>   create mode 100644 drivers/platform/mellanox/mlxbf-ptm.c
> 
Acked-by: Tim Gardner <tim.gardner at canonical.com>

I suggest dropping 'Change-Id: 
I241e1406962548cef9b33c4b3dea925e675c3c88' when applying the patch.

You should run scripts/checkpatch.pl on your patches. You'll see that 
this one generates quite a few complaints (though I don't agree with 
it's suggestion that you use 0444 instead of S_IRUGO).

--
-----------
Tim Gardner
Canonical, Inc