[PATCH net-next v3 0/2] Add support for partial store and forward

From: Pranavi Somisetty
Date: Tue May 30 2023 - 05:52:21 EST


Add support for partial store and forward mode in Cadence MACB.

Link for v1:
https://lore.kernel.org/all/20221213121245.13981-1-pranavi.somisetty@xxxxxxx/

Changes v2:
1. Removed all the changes related to validating FCS when Rx checksum
offload is disabled.
2. Instead of using a platform dependent number (0xFFF) for the reset
value of rx watermark, derive it from designcfg_debug2 register.
3. Added a check to see if partial s/f is supported, by reading the
designcfg_debug6 register.
4. Added devicetree bindings for "rx-watermark" property.
Link for v2:
https://lore.kernel.org/all/20230511071214.18611-1-pranavi.somisetty@xxxxxxx/

Changes v3:
1. Fixed DT schema error: "scalar properties shouldn't have array keywords"
2. Modified description of rx-watermark in to include units of the watermark value
3. Modified the DT property name corresponding to rx_watermark in pbuf_rxcutthru to
"cdns,rx-watermark".
4. Followed reverse christmas tree pattern in declaring variables.
5. Return -EINVAL when an invalid watermark value is set.
6. Removed netdev_info when partial store and forward is not enabled.
7. Validating the rx-watermark value in probe itself and only write to the register
in init.
8. Writing a reset value to the pbuf_cuthru register before disabing partial store
and forward is redundant. So removing it.
9. Removed the platform caps flag.
10. Instead of reading rx-watermark from DT in macb_configure_caps,
reading it in probe.
11. Changed Signed-Off-By and author names on the macb driver patch.

Maulik Jodhani (1):
net: macb: Add support for partial store and forward

Pranavi Somisetty (1):
dt-bindings: net: cdns,macb: Add rx-watermark property

.../devicetree/bindings/net/cdns,macb.yaml | 9 ++++++
drivers/net/ethernet/cadence/macb.h | 14 +++++++++
drivers/net/ethernet/cadence/macb_main.c | 29 +++++++++++++++++++
3 files changed, 52 insertions(+)

--
2.36.1