[RFC PATCH v2 1/6] ath79: ensure QCA956x gmac0 mux selects sgmii

Sander Vanheule sander at svanheule.net
Wed Jul 22 12:33:35 EDT 2020


Hi David,

On Wed, 2020-07-22 at 12:28 +0200, David Bauer wrote:
> Hi Sander,
> 
> this looks good, however there are two minor improvements still open.
> 
> > +	int err = 0;
> 
> Remove this, as it is only assigned but never accessed.
> 
> > +	u32 serdes_cal;
> 
> This is never used.
> 

I had copied the bulk of this function from the qca956x serdes init
function (patch 2/6), so I missed these when editing the code. Should
be all cleaned up now. Since the err variable was also unused in patch
2/6, I've removed it there too.

> I'll test this patch on my QCA9563 and QCA9561 devices and report
> back.

I've now tested it on my EAP245 v1 and v3. Both have a QCA9563, but
only the latter has the ethernet interface enabled correctly by the
bootloader (and didn't need this patch).

Best,
Sander




More information about the openwrt-devel mailing list