Attention is currently required from: Ravi kumar, Martin Roth, Paul Menzel, Ravi Kumar Bokka, mturney mturney.
2 comments:
File src/soc/qualcomm/common/qclib.c:
Something isn't right here... you're adding this to common/qclib.c but I don't see you removing it from sc7180/qclib.c. You still need to keep sc7180 working when you add sc7280 stuff or factor code out into common.
Please do the "factoring out into common" stuff in a separate patch preceding this one, and then the "add sc7280 support" patch should really only contain sc7280 stuff.
File src/soc/qualcomm/sc7280/qclib.c:
Patch Set #36, Line 14: QCLIB_FR_LIMITS_CFG_DATA
Just to double-check, we are sure that the limits_cfg data will never need to be updated throughout the lifetime of the device after it was initially written in the factory, right? Because right now this is defined as an RO flash section. For the DDR training data on Trogdor we also originally started with the assumption that it would never need to be updated and then changed our mind later, so I want to make sure we think about this from the start for this one.
To view, visit change 45205. To unsubscribe, or for help writing mail filters, visit settings.