cron job: media_tree daily build: ERRORS

2015-08-31 Thread Hans Verkuil
This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date: Tue Sep 1 04:00:24 CEST 2015 git branch: test git hash: d071c833a0d30e7aae0ea565d92ef83c79106d6f gcc

Re: [PATCH v8 16/55] [media] media: Don't accept early-created links

2015-08-31 Thread Mauro Carvalho Chehab
Em Mon, 31 Aug 2015 16:39:34 +0200 Javier Martinez Canillas escreveu: > Hello Hans, > > On 08/31/2015 01:01 PM, Hans Verkuil wrote: > > On 08/31/2015 12:54 PM, Mauro Carvalho Chehab wrote: > >> Em Mon, 31 Aug 2015 12:30:16 +0200 > >> Hans Verkuil

Re: [PATCH v3 4/6] [media] c8sectpfe: Update binding to reset-gpios

2015-08-31 Thread Rob Herring
On Fri, Aug 28, 2015 at 12:52 PM, Peter Griffin wrote: > gpio.txt documents that GPIO properties should be named > "[-]gpios", with being the purpose of this > GPIO for the device. > > This change has been done as one atomic commit. dtb and kernel updates are not

Re: New Terratec Cinergy S2 Box usb-id

2015-08-31 Thread Johann Klammer
On 08/31/2015 01:19 PM, Maximilian Imgrund wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA256 > > Dear all, > > I am currently figuring out how to get the Terratec Cinergy S2 USB Box > up and running. I already modified a patch to previous version (see > attachment) to include the new

Re: [PATCH] media: dvb-core: Don't force CAN_INVERSION_AUTO in oneshot mode.

2015-08-31 Thread Johann Klammer
Why not just remove the line? info->caps |= FE_CAN_INVERSION_AUTO; The capabilities call interacting with the oneshot setting is rather weird and maybe unexpected. -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to

Re: [PATCH] media: dvb-core: Don't force CAN_INVERSION_AUTO in oneshot mode.

2015-08-31 Thread Malcolm Priestley
On 31/08/15 18:03, Johann Klammer wrote: Why not just remove the line? info->caps |= FE_CAN_INVERSION_AUTO; The capabilities call interacting with the oneshot setting is rather weird and maybe unexpected. No, because in normal mode it can do auto inversion. -- To unsubscribe

[PATCH TRY 2] Support for EVOLVEO XtraTV stick

2015-08-31 Thread CIJOML CIJOMLovic
Hello guys, please find out down this email patch to support EVOLVEO XtraTV stick. This tuner is for android phones with microusb connecter, however with reduction it works perfectly with linux kernel: The device identify itself at USB bus as Bus 002 Device 004: ID 1f4d:a115 G-Tek Electronics

[linux-next:master 2531/11785] videobuf2-core.c:undefined reference to `__tracepoint_vb2_buf_queue'

2015-08-31 Thread kbuild test robot
tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: 4da7932545f1474564af3b2558b738c7a15ed853 commit: 2091f5181c66b3617a977e79843aba10e087be6c [2531/11785] [media] videobuf2: add trace events config: i386-randconfig-sb0-08312345 (attached as .config) reproduce:

Re: [PATCH] media: dvb-core: Don't force CAN_INVERSION_AUTO in oneshot mode.

2015-08-31 Thread Devin Heitmueller
Hi Malcolm, >> The capabilities call interacting with the oneshot setting is rather weird >> and maybe unexpected. >> >> > > No, because in normal mode it can do auto inversion. This isn't my area of expertise, but I suspect this is going to cause some pretty confusing behavior. Generally

Re: [PATCH TRY 2] Support for EVOLVEO XtraTV stick

2015-08-31 Thread Antti Palosaari
On 08/31/2015 09:04 PM, CIJOML CIJOMLovic wrote: Hello guys, please find out down this email patch to support EVOLVEO XtraTV stick. This tuner is for android phones with microusb connecter, however with reduction it works perfectly with linux kernel: The device identify itself at USB bus as Bus

Re: [PATCH] media: dvb-core: Don't force CAN_INVERSION_AUTO in oneshot mode.

2015-08-31 Thread Malcolm Priestley
Hi Devin On 31/08/15 19:07, Devin Heitmueller wrote: Hi Malcolm, The capabilities call interacting with the oneshot setting is rather weird and maybe unexpected. No, because in normal mode it can do auto inversion. ... If the goal was for the software-emulated auto inversion to be

Re: Do you need a loan?

2015-08-31 Thread
We offer private, commercial and personal loans with very low annualinterest rates as low as 2% in one year to 50 years repayment period anywhere in the world. We offer loans ranging from $5000 to $100 million. Our loans are well insured for maximum security is our priority. Are you losing

tvtime cannot set PAL-DK for cx88 device

2015-08-31 Thread Paweł Kiełkowski
Hello, Last week I made a big jump from Fedora 16 to Fedora 22 and tvtime is no longer able to set the PAL-DK norm. What I am getting is: "videoinput: Driver refuses to set norm: Device or resource busy". There is no such message when running tvtime on Fedora 16. I presume that following change

Re: [PATCH] rcar_vin: propagate querystd() error upstream

2015-08-31 Thread Sergei Shtylyov
Hello. On 08/25/2015 05:25 PM, Hans Verkuil wrote: rcar_vin_set_fmt() defaults to PAL when the subdevice's querystd() method call fails (e.g. due to I2C error). This doesn't work very well when a camera being used outputs NTSC which has different order of fields and resolution. Let us

Linux 4.2 ALSA snd-usb-audio inconsistent lock state warn in PCM nonatomic mode

2015-08-31 Thread Shuah Khan
Hi Takashi, I am seeing the following inconsistent lock state warning when PCM is run in nonatomic mode. This is on 4.2.0 and with the following change to force PCM on nonatomic mode: diff --git a/sound/usb/stream.c b/sound/usb/stream.c index 310a382..16bbb71 100644 --- a/sound/usb/stream.c +++

Re: [PATCH v8 25/55] [media] dvbdev: add support for interfaces

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > Now that the infrastruct for that is set, add support for > interfaces. > > Please notice that we're missing two links: > DVB FE intf-> tuner > DVB demux intf -> dvr > > Those should be added latter, after having the entire

Re: [PATCH v8 16/55] [media] media: Don't accept early-created links

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > Links are graph objects that represent the links of two already > existing objects in the graph. > > While with the current implementation, it is possible to create > the links earlier, It doesn't make any sense to allow linking > two objects

Re: [PATCH v8 42/55] [media] dvb: modify core to implement interfaces/entities at MC new gen

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > The Media Controller New Generation redefines the types for both > interfaces and entities to be used on DVB. Make the needed > changes at the DVB core for all interfaces, entities and > data and interface links to appear in the graph. > >

Re: [RFC PATCH v3 2/5] media: videobuf2: Restructure vb2_buffer

2015-08-31 Thread Hans Verkuil
On 08/31/2015 09:56 AM, Junghak Sung wrote: > > > On 08/31/2015 11:01 AM, Junghak Sung wrote: >> Hello Hans, >> >> Thank you for your review. >> I leave some comments in the body for reply. >> >> Regards, >> Junghak >> >> >> >> On 08/28/2015 10:31 PM, Hans Verkuil wrote: >>> Hi Junghak, >>> >>>

[PATCH 2/2] [media] media-controller: enable all interface links at init

2015-08-31 Thread Mauro Carvalho Chehab
Interface links are normally enabled, meaning that the interfaces are bound to the entities. So, any ioctl send to the interface are reflected at the entities managed by the interface. However, when a device is usage, other interfaces for the same hardware could be decoupled from the entities

[PATCH 1/2] [media] media-device: supress backlinks at G_TOPOLOGY ioctl

2015-08-31 Thread Mauro Carvalho Chehab
Due to the graph traversal algorithm currently in usage, we need a copy of all data links. Those backlinks should not be send to userspace, as otherwise, all links there will be duplicated. Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/media-device.c

Re: [PATCH v8 03/55] [media] omap3isp: get entity ID using media_entity_id()

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > From: Javier Martinez Canillas > > Assessing media_entity ID should now use media_entity_id() macro to > obtain the entity ID, as a next patch will remove the .id field from > struct media_entity . > > So, get rid of

Re: [PATCH v8 14/55] [media] media: add functions to allow creating interfaces

2015-08-31 Thread Mauro Carvalho Chehab
Em Mon, 31 Aug 2015 12:20:43 +0200 Hans Verkuil escreveu: > On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > > Interfaces are different than entities: they represent a > > Kernel<->userspace interaction, while entities represent a > > piece of hardware/firmware/software

Re: [PATCH v8 41/55] [media] DocBook: update descriptions for the media controller entities

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > Cleanup the media controller entities description: > - remove MEDIA_ENT_T_DEVNODE and MEDIA_ENT_T_V4L2_SUBDEV entity > types, as they don't mean anything; Shouldn't this add MEDIA_ENT_T_V4L2_VBI and _SWRADIO descriptions? Regards,

New Terratec Cinergy S2 Box usb-id

2015-08-31 Thread Maximilian Imgrund
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Dear all, I am currently figuring out how to get the Terratec Cinergy S2 USB Box up and running. I already modified a patch to previous version (see attachment) to include the new ID in the device driver, module is also loading with the ds3000

Re: [PATCH v8 43/55] [media] media: report if a pad is sink or source at debug msg

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > Sometimes, it is important to see if the created pad is > sink or source. Add info to track that. > > Signed-off-by: Mauro Carvalho Chehab > > diff --git a/drivers/media/dvb-core/dvbdev.c

Re: [PATCH v8 26/55] [media] media: add a linked list to track interfaces by mdev

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > The media device should list the interface objects, so add a linked list > for those interfaces in struct media_device. > > Signed-off-by: Mauro Carvalho Chehab Acked-by: Hans Verkuil > >

Re: [PATCH 1/2] media: atmel-isi: setup the ISI_CFG2 register directly

2015-08-31 Thread Josh Wu
Hi, Guennadi Thanks for the review. On 8/30/2015 4:48 PM, Guennadi Liakhovetski wrote: Hi Josh, On Wed, 17 Jun 2015, Josh Wu wrote: In the function configure_geometry(), we will setup the ISI CFG2 according to the sensor output format. It make no sense to just read back the CFG2 register

Re: [PATCH v8 16/55] [media] media: Don't accept early-created links

2015-08-31 Thread Mauro Carvalho Chehab
Em Mon, 31 Aug 2015 12:30:16 +0200 Hans Verkuil escreveu: > On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > > Links are graph objects that represent the links of two already > > existing objects in the graph. > > > > While with the current implementation, it is

Re: [PATCH v8 16/55] [media] media: Don't accept early-created links

2015-08-31 Thread Hans Verkuil
On 08/31/2015 12:54 PM, Mauro Carvalho Chehab wrote: > Em Mon, 31 Aug 2015 12:30:16 +0200 > Hans Verkuil escreveu: > >> On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: >>> Links are graph objects that represent the links of two already >>> existing objects in the graph.

Re: [PATCH v8 27/55] [media] dvbdev: add support for indirect interface links

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > Some interfaces indirectly control multiple entities. > Add support for those. > > Signed-off-by: Mauro Carvalho Chehab > > diff --git a/drivers/media/dvb-core/dvbdev.c b/drivers/media/dvb-core/dvbdev.c > index

Re: [PATCH v8 31/55] [media] media: add macros to check if subdev or V4L2 DMA

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > As we'll be removing entity subtypes from the Kernel, we need > to provide a way for drivers and core to check if a given > entity is represented by a V4L2 subdev or if it is an V4L2 > I/O entity (typically with DMA). This needs more

Re: [PATCH v8 38/55] [media] v4l2-subdev: use MEDIA_ENT_T_UNKNOWN for new subdevs

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > Instead of abusing MEDIA_ENT_T_V4L2_SUBDEV, initialize > new subdev entities as MEDIA_ENT_T_UNKNOWN. > > Signed-off-by: Mauro Carvalho Chehab Acked-by: Hans Verkuil > > diff --git

Re: [PATCH v8 39/55] [media] media controller: get rid of entity subtype on Kernel

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > Don't use anymore the type/subtype entity data/macros > inside the Kernel. > > Signed-off-by: Mauro Carvalho Chehab > > diff --git a/include/media/media-entity.h b/include/media/media-entity.h > index

Re: [PATCH v8 21/55] [media] media: make media_link more generic to handle interace links

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > By adding an union at media_link, we get for free a way to > represent interface->entity links. > > No need to change anything at the code, just at the internal > header file. > > Signed-off-by: Mauro Carvalho Chehab

Re: [PATCH v8 20/55] [media] media: make add link more generic

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > The media_entity_add_link() function takes an entity > as an argument just to get the list head. > > Make it more generic by changing the function argument > to list_head. > > No functional changes. > > Signed-off-by: Mauro Carvalho Chehab

Re: [RFC PATCH v3 2/5] media: videobuf2: Restructure vb2_buffer

2015-08-31 Thread Junghak Sung
On 08/31/2015 11:01 AM, Junghak Sung wrote: Hello Hans, Thank you for your review. I leave some comments in the body for reply. Regards, Junghak On 08/28/2015 10:31 PM, Hans Verkuil wrote: Hi Junghak, Thanks for this patch, it looks much better. I do have a number of comments, though...

Re: [PATCH v8 40/55] [media] media.h: don't use legacy entity macros at Kernel

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > Put the legacy MEDIA_ENT_* macros under a #ifndef __KERNEL__, > in order to be sure that none of those old symbols are used > inside the Kernel. > > Signed-off-by: Mauro Carvalho Chehab Acked-by: Hans Verkuil

[PATCH] media: dvb-core: Don't force CAN_INVERSION_AUTO in oneshot mode.

2015-08-31 Thread Malcolm Priestley
When in FE_TUNE_MODE_ONESHOT the frontend must report the actual capabilities so user can take appropriate action. With frontends that can't do auto inversion this is done by dvb-core automatically so CAN_INVERSION_AUTO is valid. However, when in FE_TUNE_MODE_ONESHOT this is not true. So only

Re: [PATCH v8 23/55] [media] media: add support to link interfaces and entities

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > Now that we have a new graph object called "interfaces", we > need to be able to link them to the entities. > > Add a linked list to the interfaces to allow them to be > linked to the entities. > > Signed-off-by: Mauro Carvalho Chehab

Re: [PATCH v8 24/55] [media] media-entity: add a helper function to create interface

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > As we'll be adding other interface types in the future, put the > common interface create code on a separate function. > > Suggested-by: Hans Verkuil > Signed-off-by: Mauro Carvalho Chehab

Re: [PATCH v8 30/55] [media] replace all occurrences of MEDIA_ENT_T_DEVNODE_DVB

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > Now that interfaces and entities are distinct, it makes no sense > of keeping something named as MEDIA_ENT_T_DEVNODE_DVB_foo. > > Made via this script: > for i in $(git grep -l MEDIA_ENT_T|grep -v uapi/linux/media.h); do sed >

Re: [PATCH v8 29/55] [media] replace all occurrences of MEDIA_ENT_T_DEVNODE_V4L

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > Now that interfaces and entities are distinct, it makes no sense > of keeping something named as MEDIA_ENT_T_DEVNODE. > > This change was done with this script: > > for i in $(git grep -l MEDIA_ENT_T|grep -v uapi/linux/media.h); do sed

Re: [PATCH v8 31/55] [media] media: add macros to check if subdev or V4L2 DMA

2015-08-31 Thread Hans Verkuil
On 08/31/2015 01:31 PM, Hans Verkuil wrote: > On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: >> As we'll be removing entity subtypes from the Kernel, we need >> to provide a way for drivers and core to check if a given >> entity is represented by a V4L2 subdev or if it is an V4L2 >> I/O

Re: [PATCH v8 14/55] [media] media: add functions to allow creating interfaces

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > Interfaces are different than entities: they represent a > Kernel<->userspace interaction, while entities represent a > piece of hardware/firmware/software that executes a function. > > Let's distinguish them by creating a separate structure

Re: [PATCH v8 28/55] [media] uapi/media.h: Fix entity namespace

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > Now that interfaces got created, we need to fix the entity > namespace. > > So, let's create a consistent new namespace and add backward > compatibility macros to keep the old namespace preserved. > > Signed-off-by: Mauro Carvalho Chehab

Re: Linux 4.2 ALSA snd-usb-audio inconsistent lock state warn in PCM nonatomic mode

2015-08-31 Thread Takashi Iwai
On Tue, 01 Sep 2015 00:48:30 +0200, Shuah Khan wrote: > > Hi Takashi, > > I am seeing the following inconsistent lock state warning when PCM > is run in nonatomic mode. This is on 4.2.0 and with the following > change to force PCM on nonatomic mode: > > diff --git a/sound/usb/stream.c

Re: [PATCH v8 44/55] [media] uapi/media.h: Add MEDIA_IOC_G_TOPOLOGY ioctl

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > Add a new ioctl that will report the entire topology on > one go. > > Signed-off-by: Mauro Carvalho Chehab > > diff --git a/include/media/media-entity.h b/include/media/media-entity.h > index

[PATCH] v4l2-compat-ioctl32: fix alignment for ARM64

2015-08-31 Thread Andrzej Hajda
Alignment/padding rules on AMD64 and ARM64 differs. To allow properly match compatible ioctls on ARM64 kernels without breaking AMD64 some fields should be aligned using compat_s64 type and in one case struct should be unpacked. Signed-off-by: Andrzej Hajda --- Hi Hans, I

Re: [PATCH v8 47/55] [media] media-device: add pads and links to media_device

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > The MC next gen API sends objects to userspace grouped by > their types. > > In the case of pads and links, in order to improve performance > and have a simpler code, the best is to store them also on > separate linked lists at MC. > > If we

muxing ES to mpeg-ts

2015-08-31 Thread Ran Shalit
Hello, I would please like to ask what is a good choice for muxing ES to mpeg transport stream. It is required to do this in application (muxing the encoder output into mpeg-ts which is transffered in ethernet udp). I know that both ffmpeg and opencaster can support this. What do you think will

Re: [PATCH v8 31/55] [media] media: add macros to check if subdev or V4L2 DMA

2015-08-31 Thread Mauro Carvalho Chehab
Em Mon, 31 Aug 2015 13:31:32 +0200 Hans Verkuil escreveu: > On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > > As we'll be removing entity subtypes from the Kernel, we need > > to provide a way for drivers and core to check if a given > > entity is represented by a V4L2

Re: [PATCH v8 53/55] [media] v4l2-core: create MC interfaces for devnodes

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:07 AM, Mauro Carvalho Chehab wrote: > V4L2 device (and subdevice) nodes should create an > interface, if the Media Controller support is enabled. > > Please notice that radio devices should not create an > entity, as radio input/output is either via wires or > via ALSA. A

Re: [PATCH v8 31/55] [media] media: add macros to check if subdev or V4L2 DMA

2015-08-31 Thread Hans Verkuil
On 08/31/2015 03:08 PM, Mauro Carvalho Chehab wrote: > Em Mon, 31 Aug 2015 13:31:32 +0200 > Hans Verkuil escreveu: > >> On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: >>> As we'll be removing entity subtypes from the Kernel, we need >>> to provide a way for drivers and

Re: [PATCH v8 16/55] [media] media: Don't accept early-created links

2015-08-31 Thread Javier Martinez Canillas
Hello Hans, On 08/31/2015 01:01 PM, Hans Verkuil wrote: > On 08/31/2015 12:54 PM, Mauro Carvalho Chehab wrote: >> Em Mon, 31 Aug 2015 12:30:16 +0200 >> Hans Verkuil escreveu: >> >>> On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: Links are graph objects that

Re: [PATCH v8 52/55] [media] media-device: remove interfaces and interface links

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:07 AM, Mauro Carvalho Chehab wrote: > Just like what's done with entities, when the media controller is > unregistered, release and interface and interface links that s/release and/release any/ > might still be there. > > Signed-off-by: Mauro Carvalho Chehab

Re: [PATCH v8 51/55] [media] remove interface links at media_entity_unregister()

2015-08-31 Thread Mauro Carvalho Chehab
Em Mon, 31 Aug 2015 14:53:45 +0200 Hans Verkuil escreveu: > On 08/30/2015 05:07 AM, Mauro Carvalho Chehab wrote: > > Interface links connected to an entity should be removed > > before being able of removing the entity. > > > > Signed-off-by: Mauro Carvalho Chehab

Re: [PATCH v8 48/55] [media] media_device: add a topology version field

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > Every time a graph object is added or removed, the version > of the topology changes. That's a requirement for the new > MEDIA_IOC_G_TOPOLOGY, in order to allow userspace to know > that the topology has changed after a previous call to it. >

Re: [PATCH v8 54/55] [media] au0828: unregister MC at the end

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:07 AM, Mauro Carvalho Chehab wrote: > au0828_analog_unregister() calls video_unregister_device(), > with, in turn, calls media_devnode_remove() in order to drop > the media interfaces. > > We can't release the media controller before that, or an > OOPS will occur: So this patch

Re: [PATCH v8 28/55] [media] uapi/media.h: Fix entity namespace

2015-08-31 Thread Mauro Carvalho Chehab
Em Mon, 31 Aug 2015 13:17:08 +0200 Hans Verkuil escreveu: > On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > > Now that interfaces got created, we need to fix the entity > > namespace. > > > > So, let's create a consistent new namespace and add backward > >

Re: [PATCH v8 48/55] [media] media_device: add a topology version field

2015-08-31 Thread Mauro Carvalho Chehab
Em Mon, 31 Aug 2015 14:29:28 +0200 Hans Verkuil escreveu: > On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > > Every time a graph object is added or removed, the version > > of the topology changes. That's a requirement for the new > > MEDIA_IOC_G_TOPOLOGY, in order to

Re: [PATCH v8 48/55] [media] media_device: add a topology version field

2015-08-31 Thread Hans Verkuil
On 08/31/2015 02:52 PM, Mauro Carvalho Chehab wrote: > Em Mon, 31 Aug 2015 14:29:28 +0200 > Hans Verkuil escreveu: > >> On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: >>> Every time a graph object is added or removed, the version >>> of the topology changes. That's a

Re: [PATCH v8 46/55] [media] media: move mdev list init to gobj

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > Let's control the topology changes inside the graph_object. > So, move the removal of interfaces/entities from the mdev > lists to media_gobj_init() and media_gobj_remove(). s/removal/addition and removal/ > > The main reason is that mdev

Re: [PATCH v8 50/55] [media] media-entity: unregister entity links

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:07 AM, Mauro Carvalho Chehab wrote: > Add functions to explicitly unregister all entity links. > This function is called automatically when an entity > link is destroyed. > > Signed-off-by: Mauro Carvalho Chehab Acked-by: Hans Verkuil

Re: [PATCH v8 44/55] [media] uapi/media.h: Add MEDIA_IOC_G_TOPOLOGY ioctl

2015-08-31 Thread Mauro Carvalho Chehab
Em Mon, 31 Aug 2015 14:00:42 +0200 Hans Verkuil escreveu: > On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > > Add a new ioctl that will report the entire topology on > > one go. > > > > Signed-off-by: Mauro Carvalho Chehab > > > > diff --git

Re: [PATCH v8 49/55] [media] media-device: add support for MEDIA_IOC_G_TOPOLOGY ioctl

2015-08-31 Thread Mauro Carvalho Chehab
Em Mon, 31 Aug 2015 14:47:03 +0200 Hans Verkuil escreveu: > On 08/30/2015 05:07 AM, Mauro Carvalho Chehab wrote: > > Add support for the new MEDIA_IOC_G_TOPOLOGY ioctl, according > > with the RFC for the MC next generation. > > > > Signed-off-by: Mauro Carvalho Chehab

Re: [PATCH v8 45/55] [media] media: Use a macro to interate between all interfaces

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:06 AM, Mauro Carvalho Chehab wrote: > Just like we do with entities, use a similar macro for the > interfaces loop. > > Signed-off-by: Mauro Carvalho Chehab Acked-by: Hans Verkuil > > diff --git

Re: [PATCH v8 49/55] [media] media-device: add support for MEDIA_IOC_G_TOPOLOGY ioctl

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:07 AM, Mauro Carvalho Chehab wrote: > Add support for the new MEDIA_IOC_G_TOPOLOGY ioctl, according > with the RFC for the MC next generation. > > Signed-off-by: Mauro Carvalho Chehab > > diff --git a/drivers/media/media-device.c

Re: [PATCH v8 51/55] [media] remove interface links at media_entity_unregister()

2015-08-31 Thread Hans Verkuil
On 08/30/2015 05:07 AM, Mauro Carvalho Chehab wrote: > Interface links connected to an entity should be removed > before being able of removing the entity. > > Signed-off-by: Mauro Carvalho Chehab > > diff --git a/drivers/media/media-device.c