5 comments:
File src/soc/qualcomm/sc7180/clock.c:
Patch Set #22, Line 13: #define HALF_DIVIDER(div2x) (div2x ? (div2x - 1) : 0)
I'm unclear what this macro is doing, it might be better to do this operation explicitly in mdss_clo […]
Done
Patch Set #22, Line 109: struct mdss_clock_config mdss_extclk_config[] = {
Please do not bind information with string comparisons. Just do something like this: […]
Done
While we're starting to use these functions for non-boot-critical clocks, it would be a good idea to […]
We would like to take such cleanups in the next project as it does not break any functionality.
File src/soc/qualcomm/sc7180/include/soc/clock.h:
Patch Set #22, Line 38: u32 cbcr;
Please fit what you're doing into the existing framework rather than rebuilding a copy next to it. […]
Done
Patch Set #22, Line 235: struct mdss_clock_config {
Please remove this too, I don't know how it got in here.
Done
To view, visit change 39612. To unsubscribe, or for help writing mail filters, visit settings.