Re: [PATCH v3] phy: cadence: Sierra: Add single link SGMII register configuration

From: Vinod Koul
Date: Wed Jun 21 2023 - 13:53:30 EST


On 21-06-23, 23:09, Vinod Koul wrote:
> On 22-05-23, 17:24, Marcin Wierzbicki wrote:
> > Add single link SGMII register configuration for no SSC for
> > cdns,sierra-phy-t0 compatibility string.
> > The configuration is based on Sierra Programmer's Guide and
> > validated in Cisco CrayAR SoC.
>
> Applied, thanks

This breaks build for me, so dropped. This is with arm64 allmodconfig
and gcc-aarch4 toolchain

drivers/phy/cadence/phy-cadence-sierra.c:2556:32: error: initialized field overwritten [-Werror=override-init]
2556 | [TYPE_SGMII] = {
| ^
drivers/phy/cadence/phy-cadence-sierra.c:2556:32: note: (near initialization for ‘cdns_map_sierra.pma_cmn_vals[3]’)
drivers/phy/cadence/phy-cadence-sierra.c:2599:32: error: initialized field overwritten [-Werror=override-init]
2599 | [TYPE_SGMII] = {
| ^
drivers/phy/cadence/phy-cadence-sierra.c:2599:32: note: (near initialization for ‘cdns_map_sierra.pma_ln_vals[3]’)
cc1: all warnings being treated as errors

--
~Vinod