NAK: [SRU][F:linux-bluefield][PATCH v1 0/2] UBUNTU: SAUCE: ipmb_dev_int.c: sync up with upstream

[Asmaa Mnebhi]

Oh... this is my bad, I gave the patches the same subject as in upstreaming:
[SRU][F:linux-bluefield][PATCH v1 1/2] ipmi: remove open coded version of SMBus block write
[SRU][F:linux-bluefield][PATCH v1 2/2] ipmi: use simple i2c probe function

I will submit another patch to keep the subject the same as in the cover letter.

-----Original Message-----
From: Tim Gardner <tim.gardner at canonical.com> 
Sent: Tuesday, July 5, 2022 10:14 AM
To: Asmaa Mnebhi <asmaa at nvidia.com>; kernel-team at lists.ubuntu.com
Subject: NAK: [SRU][F:linux-bluefield][PATCH v1 0/2] UBUNTU: SAUCE: ipmb_dev_int.c: sync up with upstream

On 7/1/22 08:22, Asmaa Mnebhi wrote:
> Buglink: 
> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbugs
> .launchpad.net%2Fbugs%2F1980525&data=05%7C01%7Casmaa%40nvidia.com%
> 7Cfcb212cbeaf441397dbf08da5e90a5c5%7C43083d15727340c1b7db39efd9ccc17a%
> 7C0%7C0%7C637926272602726927%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwM
> DAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&
> sdata=w5%2Fd%2FcsPAD3VzYDDjUc7%2FMoEG351VeDohYtYUi3TZOI%3D&reserve
> d=0
> 
> SRU Justification:
> 
> [Impact]
> 
> Sync up the ipmb_dev_int.c with the latest version available in the upstream master branch.
> 
> [Fix]
> 
> * cherry-pick fc26067c7417e7fafed7bcc97bda155d91988734
>    ipmi: remove open coded version of SMBus block write
> * backport ipmb_dev_int.c related changes from
>    commit 0924c5a0cbed5831a2a0ff2f9a19692265f1ab92
>    The above commit makes modifications to 3 files:
>    ipmb_dev_int.c, ipmi_ipmb.c and ipmi_ssif.c
>    ipmi_ipmb.c doesn't exist on 5.4, so we just pull the change 
> targeting ipmb_dev_int.c
> 
> [Test Case]
> 
> * check ipmb_dev_int is loaded without failure (check dmesg and lsmod)
> * verify that /dev/ipmb-2 is created
> * run ipmi command from bmc to send requests to BF DPU and verify that 
> the response is as expected
> 
> [Regression Potential]
> 
> Any of the test cases above could be impacted due to these changes.
> 

You appear to be missing the second patch. I checked the list archive as well.

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