From: Vincent Jardin <vjardin@free.fr>
To: Vladimir Oltean <vladimir.oltean@nxp.com>
Cc: u-boot@lists.u-boot-project.org,
Priyanka Jain <priyanka.jain@nxp.com>,
Wasim Khan <wasim.khan@nxp.com>, Tom Rini <trini@konsulko.com>,
Peng Fan <peng.fan@nxp.com>,
Ioana Ciornei <ioana.ciornei@nxp.com>,
Hemant Agrawal <hemant.agrawal@nxp.com>,
Meenakshi Aggarwal <meenakshi.aggarwal@nxp.com>
Subject: Re: [PATCH v6 12/13] doc: lx2160a: document the sd1 hwconfig option
Date: Fri, 9 Oct 2026 00:36:26 +0200 [thread overview]
Message-ID: <asga6m8Voff3NFtf@L30177.local> (raw)
In-Reply-To: <20261007104737.siyopctt5alyeehz@skbuf>
Hi Vladimir,
> 2x40g should have the "SRDS_PRTCL_S1" field as "13 or 20" as well, no?
> Because protocol 20 is literally 2x40g.
right -> v7
> Why "of protocol 13"? You could simply say "Active DPMACs for each mode:"
ok
> For clarity, I would recommend one more column
thanks, I'll take your recommendation with the template from your former
email.
> Also, I think a clarification is in order, that your SerDes1 protocol
> naming convention needs to be read from right to left, compared to the
> SRDS_PRTCL_S1 table from the hardware RM.
Right, and it is good to mention it since I had face that issue.
I'll comment it into the v7.
Best regards,
Vincent
next prev parent reply other threads:[~2026-10-08 22:36 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-01 20:47 [PATCH v6 00/13] board: lx2160a: runtime SD 1 lane configuration for LX2160A boards Vincent Jardin
2026-10-01 20:47 ` [PATCH v6 01/13] board: lx2160a: serdes: drop the split subcommand Vincent Jardin
2026-10-01 20:47 ` [PATCH v6 02/13] board: lx2160a: serdes: rename serdes1:split to serdes1:100g4x25g Vincent Jardin
2026-10-01 20:47 ` [PATCH v6 03/13] board: lx2160a: serdes: rename SPLIT_POLL_US to SRDS_POLL_US Vincent Jardin
2026-10-01 20:47 ` [PATCH v6 04/13] board: lx2160a: serdes: move the SD1 halt and reset into helpers Vincent Jardin
2026-10-01 20:47 ` [PATCH v6 05/13] board: lx2160a: serdes: set SD1 with a table Vincent Jardin
2026-10-01 20:47 ` [PATCH v6 06/13] board: lx2160a: serdes: report unknown SD1 cases Vincent Jardin
2026-10-01 20:47 ` [PATCH v6 07/13] board: lx2160a: serdes: no SD1 config once MC runs Vincent Jardin
2026-10-01 20:47 ` [PATCH v6 08/13] board: lx2160a: serdes: add serdes1:2x40g Vincent Jardin
2026-10-02 14:07 ` Vladimir Oltean
2026-10-06 14:10 ` Vincent Jardin
2026-10-01 20:47 ` [PATCH v6 09/13] board: lx2160a: serdes: enter a SD1 mode through the mode it starts from Vincent Jardin
2026-10-02 14:08 ` Vladimir Oltean
2026-10-01 20:47 ` [PATCH v6 10/13] board: lx2160a: serdes: add serdes1:40g4x10g Vincent Jardin
2026-10-07 11:27 ` Vladimir Oltean
2026-10-08 22:32 ` Vincent Jardin
2026-10-01 20:47 ` [PATCH v6 11/13] board: lx2160a: serdes: add serdes1:100g4x10g Vincent Jardin
2026-10-01 20:47 ` [PATCH v6 12/13] doc: lx2160a: document the sd1 hwconfig option Vincent Jardin
2026-10-07 10:47 ` Vladimir Oltean
2026-10-08 22:36 ` Vincent Jardin [this message]
2026-10-01 20:47 ` [PATCH v6 13/13] board: nbxv3: apply sd1 hwconfig before MC starts Vincent Jardin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=asga6m8Voff3NFtf@L30177.local \
--to=vjardin@free.fr \
--cc=hemant.agrawal@nxp.com \
--cc=ioana.ciornei@nxp.com \
--cc=meenakshi.aggarwal@nxp.com \
--cc=peng.fan@nxp.com \
--cc=priyanka.jain@nxp.com \
--cc=trini@konsulko.com \
--cc=u-boot@lists.u-boot-project.org \
--cc=vladimir.oltean@nxp.com \
--cc=wasim.khan@nxp.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox