On Thu, Jan 24, 2013 at 7:01 AM, Daniel Vetter daniel@ffwll.ch wrote:
On Tue, Jan 22, 2013 at 04:36:24PM -0600, Rob Clark wrote:
Add output panel driver for i2c encoder slaves.
Signed-off-by: Rob Clark robdclark@gmail.com
Found some more stuff ...
[cut]
+static const struct drm_encoder_helper_funcs slave_encoder_helper_funcs = {
.dpms = drm_i2c_encoder_dpms,.mode_fixup = drm_i2c_encoder_mode_fixup,.prepare = slave_encoder_prepare,.commit = drm_i2c_encoder_commit,.mode_set = drm_i2c_encoder_mode_set,.save = drm_i2c_encoder_save,.restore = drm_i2c_encoder_restore,+};
I couldn't find these wrappers anywhere ...
this is in one of the dependent patches (also on my tilcdc-next branch):
https://patchwork.kernel.org/patch/1950971/
I also cleaned up nouveau to use 'em:
https://patchwork.kernel.org/patch/1951051/
+static const struct i2c_board_info info = {
I2C_BOARD_INFO("tda998x", 0x70)+};
Shouldn't there be some of/devicetree thing to tell us which one to load?
There are limited options for the i2c addresses, and all the boards I've seen which have this part are at 0x70 and 0x34 for CEC. Probably the CEC address needs to come from a config struct or maybe the i2c encoder just gets passed a 'struct device_node *'. I'm not really sure the best way to handle that to share the encoder between DT and non-DT platforms, and since I didn't really need the configurability yet, I just left that for now and decided we could figure out how to handle that as the need arose.
BR, -R
-Daniel
Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch