Hi Ken Lin,

On Tuesday 17 January 2017 03:44 AM, Ken.Lin wrote:

Hi Joe and Mugunthan,

We encountered the voltage peak issue while doing the IEEE PHY conformance test, which has to do with the AR8033 register (SetDes Test and System Mode Control) setting in u-boot. In your commit change info, you tried to enable tx clock delay by setting bit 8 to 1 (filling in 0x100, setting the reserved bits to 0) for solving the auto negotiation failure issue. https://patchwork.ozlabs.org/patch/681801/

After we checked with Qualcomm (Atheros) and they responded that on their platform, the register should be set to 0x2C47 (for the reserved bits) and this would solve the voltage peak issue we experienced. Could you please help check if it's appropriate to set the reserved bits according to Qualcomm's setting since your commit breaks the original setting (see https://community.nxp.com/message/868898 for more details info)? Please let me know if you have any suggestions/comments on this.

I guess the right thing to do would be to readback this registers, modify only bit 8 and write the value back.

Is that something you can try? You can check that the value you read back for the reserved bits is indeed what Qualcomm recommends you to set.

Thanks, Sekhar