Re: [PATCH] drm: encoder_slave: respect of_node on i2c encoder init

2013-06-11 Thread Sebastian Hesselbarth
On 06/11/2013 05:10 PM, Francisco Jerez wrote: Sebastian Hesselbarth writes: - I think we could also drop the call to ->set_config since presumably an of-enabled driver grabbed any required info already from the dt. [...] I think this way we could still share encoder slaves across tons of

Re: [PATCH] drm: encoder_slave: respect of_node on i2c encoder init

2013-06-11 Thread Francisco Jerez
Hi, Sebastian Hesselbarth writes: >> - I think we could also drop the call to ->set_config since presumably an >>of-enabled driver grabbed any required info already from the dt. >[...] >> I think this way we could still share encoder slaves across tons of >> platforms, only the init sequence

Re: [PATCH] drm: encoder_slave: respect of_node on i2c encoder init

2013-06-11 Thread Sebastian Hesselbarth
On 06/11/13 09:24, Daniel Vetter wrote: On Mon, Jun 10, 2013 at 11:23:42PM +0200, Sebastian Hesselbarth wrote: Current DRM slave encoder API conflicts with auto-registration of i2c client when using DT probed clients. To allow DRM slave encoders passed by DT, this patch adds a check to drm_i2c_e

Re: [PATCH] drm: encoder_slave: respect of_node on i2c encoder init

2013-06-11 Thread Daniel Vetter
On Mon, Jun 10, 2013 at 11:23:42PM +0200, Sebastian Hesselbarth wrote: > Current DRM slave encoder API conflicts with auto-registration of i2c client > when using DT probed clients. To allow DRM slave encoders passed by DT, this > patch adds a check to drm_i2c_encoder_init for a non-NULL .of_node o

[PATCH] drm: encoder_slave: respect of_node on i2c encoder init

2013-06-10 Thread Sebastian Hesselbarth
Current DRM slave encoder API conflicts with auto-registration of i2c client when using DT probed clients. To allow DRM slave encoders passed by DT, this patch adds a check to drm_i2c_encoder_init for a non-NULL .of_node on i2c_board_info and calls an of_i2c helper to get the i2c client device inst