Koen Kooi wrote, on 12/19/2010 08:56 AM:

Op 19 dec 2010, om 15:47 heeft Nishanth Menon het volgende geschreven:

Koen Kooi wrote, on 12/19/2010 08:42 AM:
$subject - should'nt you be doing this in separate patches? easier to track 
with git bisect at a later date than a whole sale change?

Is that really needed?
it is usually recommended to keep each separate functionality introduced/fixed independent of the other.


btw, why are we enabling camera here? should'nt that be done in camera's probe 
function? no?

On P7/P8 it's the camera, but on A2 and above it's the LDO that powers the 
TFP410, so if you want to have output on DVI you need to turn it on.
>I'm not sure if it should be part of the DVI init since it has different functions on different board revisions. P7/P8 were not production Beagles if my memory serves me right.. DVI init makes more sense to me.

--
Regards,
Nishanth Menon
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to