Skip to content

RFC Unrestricted media entity ID range support

Sakari Ailus linux-media 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 RFC Latest9 patches 5 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/9media: Rename MEDIA_ENTITY_ENUM_MAX_ID as MEDIA_ENTITY_MAX_LOW_IDdrivers/media/media-entity.c include/media/media-entity.h+5 -50 replies
2/9media: Introduce low_id for media entitiesdrivers/media/media-device.c include/media/media-device.h include/media/media-entity.h+19 -00 replies
3/9media: Add an API to manage entity enumerationsinclude/media/media-entity.h+44 -10 replies
4/9media: Use media entity enum API for managing low IDsdrivers/media/media-device.c include/media/media-device.h include/media/media-entity.h+20 -90 replies
5/9media: Use entity enums in graph walkdrivers/media/media-entity.c include/media/media-entity.h+6 -60 replies
6/9media: entity: Remove useless WARN_ON()'sdrivers/media/media-entity.c+0 -50 replies
7/9omap3isp: Use media entity enumeration APIdrivers/media/platform/omap3isp/isp.c drivers/media/platform/omap3isp/isp.h drivers/media/platform/omap3isp/ispccdc.c +2 more+18 -190 replies
8/9vsp1: Use media entity enumeration APIdrivers/media/platform/vsp1/vsp1_video.c+3 -40 replies
9/9staging: omap4iss: Use media entity enum APIdrivers/staging/media/omap4iss/iss.c drivers/staging/media/omap4iss/iss.h drivers/staging/media/omap4iss/iss_video.c +1 more+10 -90 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [RFC 0/9] Unrestricted media entity ID range support

Hi all,

This patchset adds an API for managing entity enumerations, i.e. storing a bit of information per entity. The entity ID is no longer limited to small integers (e.g. 31 or 63), but MEDIA_ENTITY_MAX_LOW_ID. The drivers are also converted to use that instead of a fixed number.

If the number of entities in a real use case grows beyond the capabilities of the approach, the algorithm may be changed. But most importantly, the API is used to perform the same operation everywhere it's done.

I'm sending this for review only, the code itself is untested.

I haven't entirely made up my mind on the fourth patch. It could be dropped, as it uses the API for a somewhat different purpose.

Signature · 2 lines
Kind regards,
Sakari