Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1460662 > unrolled thread

[PATCH 0/2] [media] tvp5150: use .registered callback to register entity and links

Back to article view | Back to linux.kernel

Contents

  [PATCH 0/2] [media] tvp5150: use .registered callback to register entity and links Javier Martinez Canillas <javier@osg.samsung.com> - 2016-08-11 18:30 +0200
    Re: [PATCH 0/2] [media] tvp5150: use .registered callback to register  entity and links Javier Martinez Canillas <javier@osg.samsung.com> - 2016-08-11 21:40 +0200
    Re: [PATCH 0/2] [media] tvp5150: use .registered callback to register  entity and links Sakari Ailus <sakari.ailus@linux.intel.com> - 2016-08-11 21:40 +0200

#1460662 — [PATCH 0/2] [media] tvp5150: use .registered callback to register entity and links

Hello,

Sakari pointed out in "[PATCH 2/8] [media] v4l2-async: call registered_async
after subdev registration" [0] that the added .registered_async callback isn't
needed since the v4l2 core already has an internal_ops .registered callback.

I missed that there was already this when added the .registered_async callback,
sorry about that.

This small series convert the tvp5150 driver to use the proper .registered and
remove .registered_async since isn't needed.

[0]: https://lkml.org/lkml/2016/8/11/254

Best regards,
Javier


Javier Martinez Canillas (2):
  [media] tvp5150: use sd internal ops .registered instead
    .registered_async
  [media] v4l2-async: remove unneeded .registered_async callback

 drivers/media/i2c/tvp5150.c          | 8 ++++++--
 drivers/media/v4l2-core/v4l2-async.c | 7 -------
 include/media/v4l2-subdev.h          | 3 ---
 3 files changed, 6 insertions(+), 12 deletions(-)

-- 
2.5.5

[toc] | [next] | [standalone]

#1460751 — Re: [PATCH 0/2] [media] tvp5150: use .registered callback to register entity and links

Hello Sakari,

On 08/11/2016 03:31 PM, Sakari Ailus wrote:
> Javier Martinez Canillas wrote:
>> Hello,
>>
>> Sakari pointed out in "[PATCH 2/8] [media] v4l2-async: call registered_async
>> after subdev registration" [0] that the added .registered_async callback isn't
>> needed since the v4l2 core already has an internal_ops .registered callback.
>>
>> I missed that there was already this when added the .registered_async callback,
>> sorry about that.
>>
>> This small series convert the tvp5150 driver to use the proper .registered and
>> remove .registered_async since isn't needed.
> 
> Thanks!
> 
> For both:
> 
> Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> 

Thanks to you for pointing out my silly mistake.

Best regards,
-- 
Javier Martinez Canillas
Open Source Group
Samsung Research America

[toc] | [prev] | [next] | [standalone]

#1460752 — Re: [PATCH 0/2] [media] tvp5150: use .registered callback to register entity and links

Javier Martinez Canillas wrote:
> Hello,
>
> Sakari pointed out in "[PATCH 2/8] [media] v4l2-async: call registered_async
> after subdev registration" [0] that the added .registered_async callback isn't
> needed since the v4l2 core already has an internal_ops .registered callback.
>
> I missed that there was already this when added the .registered_async callback,
> sorry about that.
>
> This small series convert the tvp5150 driver to use the proper .registered and
> remove .registered_async since isn't needed.

Thanks!

For both:

Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>

-- 
Kind regards,

Sakari Ailus
sakari.ailus@linux.intel.com

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web