linux-api.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
To: Mauro Carvalho Chehab <mchehab-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>
Cc: Hans Verkuil <hverkuil-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>,
	Linux Media Mailing List
	<linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Mauro Carvalho Chehab
	<mchehab-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>,
	linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Javier Martinez Canillas
	<javier-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>
Subject: Re: [PATCH] [media] uapi/media.h: Use u32 for the number of graph objects
Date: Thu, 17 Dec 2015 16:20:34 +0100	[thread overview]
Message-ID: <2294897.lftU8mbJHZ@wuerfel> (raw)
In-Reply-To: <20151217125806.3f4f879e-+RedX5hVuTR+urZeOPWqwQ@public.gmane.org>

On Thursday 17 December 2015 12:58:06 Mauro Carvalho Chehab wrote:
> > Can you clarify how the 'topology_version' is used here? Is that
> > the version of the structure layout that decides how we interpret the
> > rest, or is it a number that is runtime dependent?
> 
> No, topology_version is just a mononotonic counter that starts on 0
> and it is incremented every time a graph object is added or removed. 
> 
> It is meant to be used to track if the topology changes after a previous
> call to this ioctl.
> 
> On existing media controller embedded device hardware, it should
> always be zero, but on devices that allow dynamic hardware changes
> (some embedded DTV hardware allows that - also on devices with FPGA,
> with RISC CPUs or hot-pluggable devices) should use it to know if the
> hardware got modified. 
> 
> This is also needed on multi-function devices where different drivers 
> are used for each function. That's the case of au0828, with uses a
> media driver for video, and the standard USB Audio Class driver for
> audio. As the drivers are independent, the topology_version will
> be zero when the first driver is loaded, but it will change during
> at probe time on second driver. It will also be increased if one
> of the drivers got unbind.

Ok, got it. Thanks for the explanation.

	Arnd

      parent reply	other threads:[~2015-12-17 15:20 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-17 11:09 [PATCH] [media] uapi/media.h: Use u32 for the number of graph objects Mauro Carvalho Chehab
     [not found] ` <5672A69F.7020505@xs4all.nl>
2015-12-17 12:45   ` Mauro Carvalho Chehab
     [not found]     ` <20151217104556.70d4f0f8-+RedX5hVuTR+urZeOPWqwQ@public.gmane.org>
2015-12-17 13:55       ` Arnd Bergmann
2015-12-17 14:58         ` Mauro Carvalho Chehab
     [not found]           ` <20151217125806.3f4f879e-+RedX5hVuTR+urZeOPWqwQ@public.gmane.org>
2015-12-17 15:20             ` Arnd Bergmann [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=2294897.lftU8mbJHZ@wuerfel \
    --to=arnd-r2ngtmty4d4@public.gmane.org \
    --cc=hverkuil-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org \
    --cc=javier-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org \
    --cc=linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=mchehab-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org \
    --cc=mchehab-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).