NAK/Cmnt: [SRU][J:linux-bluefield][PATCH v2 0/1] Add test script and yaml for dpll

[William Tu]

Hi Bartlomiej,

Thanks for your feedback.
I will create a new buglink, fix the commit, and make the commit message more clear.

William


From: Bartlomiej Zolnierkiewicz <bartlomiej.zolnierkiewicz at canonical.com>
Date: Wednesday, March 6, 2024 at 5:13 AM
To: William Tu <witu at nvidia.com>
Cc: Ubuntu Kernel Team <kernel-team at lists.ubuntu.com>, Bodong Wang <bodong at nvidia.com>, Vladimir Sokolovsky <vlad at nvidia.com>, dann frazier <dann.frazier at canonical.com>
Subject: NAK/Cmnt: [SRU][J:linux-bluefield][PATCH v2 0/1] Add test script and yaml for dpll
External email: Use caution opening links or attachments


Hi William,

This submission should be either "UBUNTU: SAUCE: .." patch or a more
complete set of cherry-picks/backports from the upstream kernel.

On Thu, Feb 29, 2024 at 11:43 PM William Tu <witu at nvidia.com> wrote:
>
> BugLink: https://bugs.launchpad.net/bugs/2053155

Please use a new BugLink for this submission.

> Add a script and yaml to verify dpll is supported from kernel. The
> script and yaml file were supported in upstream linux but was deleted
> in this repo. So copy them directly from latest upsteam now.

The description needs to be more detailed as the current one won't
help us in the future .

Do you mean "Add DPLL and syncE support" pull request here by "this
repo"? If so please reference the original BugLink in the description.

Please also give the exact upstream commit number in the description
that these files are copied from.

> William Tu (1):

The patch has the first SoB from Tony suggesting that it is a patch
from him ("From: ..." in the patch seem to be missing).

--
Best regards,
Bartlomiej

>   Add test script and yaml for dpll
> ---
> v2: fix incorrect format
>
>  Documentation/netlink/genetlink.yaml | 330 ++++++++++
>  tools/net/ynl/cli.py                 |  77 +++
>  tools/net/ynl/lib/__init__.py        |   8 +
>  tools/net/ynl/lib/nlspec.py          | 607 +++++++++++++++++++
>  tools/net/ynl/lib/ynl.py             | 873 +++++++++++++++++++++++++++
>  5 files changed, 1895 insertions(+)
>  create mode 100644 Documentation/netlink/genetlink.yaml
>  create mode 100644 tools/net/ynl/cli.py
>  create mode 100644 tools/net/ynl/lib/__init__.py
>  create mode 100644 tools/net/ynl/lib/nlspec.py
>  create mode 100644 tools/net/ynl/lib/ynl.py
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20240306/b96ddd08/attachment.html>