On 13 November 2013 09:17, Tushar Behera <tushar.behera@xxxxxxxxxx> wrote: > On 12 November 2013 15:47, Kukjin Kim <kgene@xxxxxxxxxx> wrote: >> Sachin Kamat wrote: >>> >> + dt ml. >> >>> As per the timing information for supported panel, the value should >>> be between 47.2 MHz to 47.9 MHz for 60Hz refresh rate. >>> >>> Total horizontal pixels = 1024 (x-res) + 80 (margin) + 48 (hsync) = 1152 >>> Total vertical pixels = 600 (y-res) + 80 (margin) + 3 (vsync) = 653 >> >> Should be 683? >> >>> >>> Target pixel clock rate for refresh rate @60 Hz >>> = 1152 * 653 * 60 = 47208960 Hz ~ 47.5 MHz >>> >> Same here. >> > > Yes, typo in both the cases, however the calculation is right. The > same typo in patch 1/2 also. You may please fix them while applying. > Ping Kukjin. -- With warm regards, Sachin -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html