On 7/16/2024 1:02 PM, Serhii Iliushyk wrote: > + /* GTY_RX_BUF_STAT */ > +#ifdef RXBUFSTAT > + p->mp_reg_gty_rx_buf_stat = > + nthw_module_get_register(p->mp_mod_mac_pcs, > MAC_PCS_GTY_RX_BUF_STAT); > + p->mp_field_gty_rx_buf_stat_rx_buf_stat0 = > + nthw_register_get_field(p->mp_reg_gty_rx_buf_stat, > + MAC_PCS_GTY_RX_BUF_STAT_RX_BUF_STAT_0); > + p->mp_field_gty_rx_buf_stat_rx_buf_stat1 = > + nthw_register_get_field(p->mp_reg_gty_rx_buf_stat, > + MAC_PCS_GTY_RX_BUF_STAT_RX_BUF_STAT_1); > + p->mp_field_gty_rx_buf_stat_rx_buf_stat2 = > + nthw_register_get_field(p->mp_reg_gty_rx_buf_stat, > + MAC_PCS_GTY_RX_BUF_STAT_RX_BUF_STAT_2); > + p->mp_field_gty_rx_buf_stat_rx_buf_stat3 = > + nthw_register_get_field(p->mp_reg_gty_rx_buf_stat, > + MAC_PCS_GTY_RX_BUF_STAT_RX_BUF_STAT_3); > + p->mp_field_gty_rx_buf_stat_rx_buf_stat_changed0 = > + nthw_register_get_field(p->mp_reg_gty_rx_buf_stat, > + MAC_PCS_GTY_RX_BUF_STAT_RX_BUF_STAT_CHANGED_0); > + p->mp_field_gty_rx_buf_stat_rx_buf_stat_changed1 = > + nthw_register_get_field(p->mp_reg_gty_rx_buf_stat, > + MAC_PCS_GTY_RX_BUF_STAT_RX_BUF_STAT_CHANGED_1); > + p->mp_field_gty_rx_buf_stat_rx_buf_stat_changed2 = > + nthw_register_get_field(p->mp_reg_gty_rx_buf_stat, > + MAC_PCS_GTY_RX_BUF_STAT_RX_BUF_STAT_CHANGED_2); > + p->mp_field_gty_rx_buf_stat_rx_buf_stat_changed3 = > + nthw_register_get_field(p->mp_reg_gty_rx_buf_stat, > + MAC_PCS_GTY_RX_BUF_STAT_RX_BUF_STAT_CHANGED_3); > +#endif >
These compile time macros require to update source code to change behavior, it is very easy to create test gaps by time. Better to convert it devargs to configure them dynamically if they are required, but as far as I can see these fields not used at, so perhaps above code can be removed, up to you.
