NACK/Cmnt: [SRU][F:linux-bluefield][PATCH v1 0/1] UBUNTU: SAUCE: Revert "UBUNTU: SAUCE: mlxbf_gige: clear MDIO gateway lock after read"

[Stefan Bader]

On 28.09.22 22:37, David Thompson wrote:
> BugLink: https://bugs.launchpad.net/bugs/1991151
> 
> SRU Justification:
> 
> [Impact]
> 
> The Jammy repo contains the following SAUCE patch:
> UBUNTU: SAUCE: mlxbf_gige: clear MDIO gateway lock after read
> 
> This patch is now in mainline Linux kernel, so the SAUCE patch
> can be reverted and replaced with the upstream commit SHA.
> 
> [Fix]
> 
> The fix requires two parts, the first of which is addressed by
> this bug. A second bug will be created to add the upstream
> commit to the repo. The fix to this bug is a revert of the SAUCE patch
> 8d2dbbd437a1 UBUNTU: SAUCE: mlxbf_gige: clear MDIO gateway lock after read
> 
> [Test Case]
> 
> Boot platform and bring up "oob_net0" interface properly
> Test that network traffic works properly
> 
> [Regression Potential]
> 
> Low, as this logic is well tested.
> 
> [Other]
> 
> Will add a note to this bug with the second bug's number,
> after the second bug is created. The second bug is for
> adding the upstream commit to the Jammy repo.
> 
> David Thompson (1):
>    UBUNTU: SAUCE: Revert "UBUNTU: SAUCE: mlxbf_gige: clear MDIO gateway
>      lock after read"
> 
>   drivers/net/ethernet/mellanox/mlxbf_gige/mlxbf_gige_mdio.c | 6 ------
>   1 file changed, 6 deletions(-)
> 

I think this one was now replaced by a series which correctly addresses Jammy. 
Combining the revert with the replacement. But also the subject on this thread 
was confusingly wrong. Doing an explicit NACK here to clear things up.

-Stefan

-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20220930/99ab0f49/attachment.sig>