On 06/09/16 11:19, Jyri Sarha wrote: > Add tilcdc_crtc_set_clk() and cleanup cpufreq_transition(). The new > tilcdc_crtc_set_clk() is used in tilcdc_crtc_mode_set_nofb() instead > tilcdc_crtc_update_clk(). New tilcdc_crtc_update_clk() is implemented > using tilcdc_crtc_set_clk() for cpufreq_transition() alone. I think the patch looks ok, but the description doesn't help too much in understanding what this does. It gives no hint on what tilcdc_crtc_set_clk() is, what's its relation to tilcdc_crtc_update_clk(), and what's the overall purpose here. If I have to read the code to understand the reason for the patch, the description has failed =). Tomi
Attachment:
signature.asc
Description: OpenPGP digital signature
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel