[PATCH 17/42] drm/omap: Add dispc_mgr_get_supported_outputs()

[Tomi Valkeinen]

On 07/03/16 10:47, Laurent Pinchart wrote:
> Hi Tomi,
> 
> Thank you for the patch.
> 
> On Monday 22 February 2016 19:10:23 Tomi Valkeinen wrote:
>> We are removing the use of the 'struct omap_overlay_manager' from
>> omapdrm, and one part of that is removing the use of
>> mgr->supported_outputs field.
>>
>> This patch adds dispc_mgr_get_supported_outputs() function which can be
>> used instead of mgr->supported_outputs. omap_crtc.c is changed to use
>> the new function.
>>
>> Signed-off-by: Tomi Valkeinen <tomi.valkeinen at ti.com>
>> ---
>>  drivers/gpu/drm/omapdrm/dss/dispc.c   | 6 ++++++
>>  drivers/gpu/drm/omapdrm/dss/omapdss.h | 2 ++
>>  drivers/gpu/drm/omapdrm/omap_crtc.c   | 2 +-
>>  3 files changed, 9 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/omapdrm/dss/dispc.c
>> b/drivers/gpu/drm/omapdrm/dss/dispc.c index a4274dca384a..a5940892a788
>> 100644
>> --- a/drivers/gpu/drm/omapdrm/dss/dispc.c
>> +++ b/drivers/gpu/drm/omapdrm/dss/dispc.c
>> @@ -2910,6 +2910,12 @@ bool dispc_ovl_enabled(enum omap_plane plane)
>>  }
>>  EXPORT_SYMBOL(dispc_ovl_enabled);
>>
>> +enum omap_dss_output_id dispc_mgr_get_supported_outputs(enum omap_channel
>> channel)
>> +{
>> +	return dss_feat_get_supported_outputs(channel);
>> +}
> 
> Wouldn't it make sense to turn this into a static inline function, as it's a 
> one-liner ?

I plan to get rid of the dss_features.c and move the code into
respective driver files. So this function will probably get a bit larger
with that.

And hmm... Would it even work, as dss_feat_get_supported_outputs() is
not visible to the callers?

 Tomi

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160307/72b8e1e0/attachment.sig>