7 comments:
File src/soc/qualcomm/sc7180/clock.c:
Patch Set #15, Line 353: uint32_t mdss_clk_type)
It looks like this should fit into 96 characters.
The implementation is updated as per the latest comments from Julius.
Patch Set #15, Line 369: reg_addr = (void *)mdss_extclk_config[i].cbcr;
Is it common that this typo is not 0 or 1? Maybe add an else branch with a debug message?
The implementation is updated as per the latest comments from Julius.
Please use CB_SUCCESS and friends.
We would like to keep such cleanups for the next project as we have passed CS.
Patch Set #15, Line 410: CLK_CTL_RCG_MND_BMSK);
This should fit into 96 characters.
Done
Please add enums for this.
Implementation is updated as per latest comments from Julius.
File src/soc/qualcomm/sc7180/include/soc/addressmap.h:
Patch Set #15, Line 16: #define DISP_CC_BASE 0x0AF00000
Please align using tabs.
Done
File src/soc/qualcomm/sc7180/include/soc/clock.h:
Patch Set #15, Line 35: ((offsetof(type, mem)) + base)
This fits into one line.
Implementation updated as per latest comments from Julius.
To view, visit change 39612. To unsubscribe, or for help writing mail filters, visit settings.