Adrian Bunk wrote:

The Coverity checker noticed that in the function setclkMHz in 
drivers/video/imsttfb.c, the variable clk_p is never assigned a value 
other than 0.

Could someone understanding this code decide whether this is a real bug 
or whether clk_p can be replaced by 0?

It does look like to be an oversight.  The same is present in Xorg code
too.

Tony