1 comment:
File src/soc/mediatek/mt8183/mt6358.c:
Patch Set #3, Line 782: VSIM2_CALI_POS_60_MV
It's weird to me that you define a lot of enum for different values and perform arithmetic computation on them...
What about rename 'vsim2_cali' to 'vsim2_cali_10mv' (or any better name) and remove those enums?
And line 781-784 looks equivalent to 'vsim2_cali = MAX(0, vsim2_cali - 6);'
To view, visit change 32057. To unsubscribe, or for help writing mail filters, visit settings.