On Fri, 13 Nov 2009, Kuninori Morimoto wrote: > > Dear Guennadi > > Thank you for checking patch > > > + ret = i2c_smbus_write_byte_data(client, OUTCTR1, > > + VSP_LO | VSSL_VVALID | > > + HSP_LO | HSSL_DVALID); > > > > now you dropped VSP_LO | HSP_LO, could you, please, explain, why? Also, > > sorry for not explaining properly. Yesterday I wrote > > Because VSP_LO = HSP_LO = 0. Oh, indeed. Ok, but can you add proper support for both high and low polarities? > And when I use xVALID, xSP_LO mean ACTIVE HI. > > So, I drop these explain because it is just un-understandable. Thanks Guennadi --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/ -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
