Dear Guennadi Thank your for checking patch. > > Guys, are you both sure this should be SLAVE, not MASTER? Have you tested > > it? Both tw9910 and ov772x register themselves as MASTER and from the Of course I tested it. > Ok, sorry, you, probably, did test it and it worked, but just because the > SLAVE / MASTER flag is not tested in soc_camera_bus_param_compatible(), > which I should fix with the next pull, but this does look wrong. Please, > fix. Hmm. I should have asked you what is MASTER/SLAVE before sending patch. I suspect it it about who generates the clock signal either the camera or the host. Our CEU does not support any clock generation so it is always SLAVE. Therefore I didn't support MASTER for CEU. But it seems wrong understanding... I would like ask you What MASTER/SLAVE means ? Best regards -- Kuninori Morimoto -- 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