On 12/13/2016 04:02 PM, Florian Fainelli wrote:
No strong feelings either, it just seems easier and safer to move the check down in the function and make it return success rather than potentially affecting the error path within the caller of emac_clks_phase{1,2}_init here.
I suppose that makes sense. I'll post a V2. -- Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm Technologies, Inc. Qualcomm Technologies, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project.