4 comments:
Patch Set #2, Line 7: sc7280
Please start to put the vendor in the prefix too. […]
Is there a rule about that? Considering that patch subject space is often too short to really convey a good summary of a patch anyway, I don't think we should force people to waste too much of it on prefix. SoC names don't really clash between vendors.
File src/soc/qualcomm/sc7280/include/soc/symbols.h:
Patch Set #6, Line 15: DECLARE_REGION(aop)
Should probably just dump these into <soc/symbols_common.h> rather than duplicate between chips. Extra symbols don't hurt even if a chip isn't using them.
File src/soc/qualcomm/sc7280/qclib.c:
Patch Set #6, Line 9: int qclib_soc_blob_load(void)
Please factor this out in qualcomm/common if it's needed by more than one SoC. Since all the SoC's we're using for now need it anyway, I'd just throw it into qclib_load_and_run(). If you really want to keep it separate you could guard it with a Kconfig, or make a separate qclib_add_pmic_stuff() and call that from here or something.
File src/soc/qualcomm/sc7280/timer.c:
Patch Set #6, Line 7: void init_timer(void)
Should consider putting this in qualcomm/common and just using SRC_XO_HZ from <soc/clock.h> for the frequency (or define a separate ARCH_TIMER_HZ in each <soc/clock.h> that can then be defined to SRC_XO_HZ or whatever else it might be for each individual SoC).
To view, visit change 45205. To unsubscribe, or for help writing mail filters, visit settings.