Re: [PATCH net-next v7 02/12] net: dsa: microchip: add IPV information support

From: Oleksij Rempel
Date: Tue May 07 2024 - 00:37:54 EST


Hi Woojung,

On Mon, May 06, 2024 at 08:43:48PM +0000, Woojung.Huh@xxxxxxxxxxxxx wrote:
> Hi Oleksij,
>
> Thanks for the patch and sorry about late comment on this.
>
> I have a comment on the name of IPV (Internal Priority Value)
> IPV is added and used term in 802.1Qci PSFP
> (https://ieeexplore.ieee.org/document/8064221) and, merged into 802.1Q (from 802.1Q-2018)
> for another functions.
>
> Even it does similar operation holding temporal priority value internally (as it is named),
> because KSZ datasheet doesn't use the term of IPV (Internal Priority Value) and
> avoiding any confusion later when PSFP is in the Linux world,
> I would like to recommend a different name such as IPM (Internal Priority Mapping) than IPV.
>
> How do you think?

Ok.

Do IPV in LAN9372 datasheet means, IPV 802.1Qci PSFP or IPM?

Regards,
Oleksij
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |