addressed the comments
9 comments:
File src/soc/qualcomm/sc7180/display/dsi_phy.c:
Patch Set #26, Line 92: DSI_LaneID_Max
No, this is not what I meant. […]
Done
File src/soc/qualcomm/sc7180/display/dsi_phy.c:
Patch Set #28, Line 95: uint32_t decdiv_start;
There are still plenty of fields (like this one) in here that aren't used in the code at all (anymor […]
Done
Patch Set #28, Line 453: lane_cfg.strength_override = 0;
What do you need these 6 fields for? I don't think you need struct dsi_phy_laneconfig_type at all to […]
Done
Patch Set #28, Line 495: wmb();
You don't need this wmb() either.
Done
Patch Set #28, Line 650: phy_cfg->dsi_clksel = 1;
Why do you need this field?
Done
Patch Set #28, Line 760: printk(BIOS_INFO, "Desired bitclock:%u\n", (uint32_t)desired_bclk);
Please add proper spacing and a unit, e.g. […]
Done
Patch Set #28, Line 780: phy_cfg.escape_freq = 19;
Why do you need this field?
Done
File src/soc/qualcomm/sc7180/display/dsi_phy_pll.c:
A lot of unused fields in this file too (e.g. all the ssc_ stuff?). […]
Done
Patch Set #28, Line 59: static inline u64 div_u64(u64 dividend, u32 divisor)
Why do we have these two functions, anyway? I don't really see what they add. Why write […]
Done
To view, visit change 39613. To unsubscribe, or for help writing mail filters, visit settings.