[{"id":397,"web_url":"https://patchwork.libcamera.org/comment/397/","msgid":"<20190118001254.GH23244@pendragon.ideasonboard.com>","date":"2019-01-18T00:12:54","subject":"Re: [libcamera-devel] [PATCH] libcamera: device_enumerator: extend\n\tdocumentation of DeviceMatch","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/people/2/","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"content":"Hi Niklas,\n\nThank you for the patch.\n\nOn Fri, Jan 18, 2019 at 12:06:36AM +0100, Niklas Söderlund wrote:\n> Extend the document of the intended usage of DeviceMatch need for\n> information to correctly function to find a uniquely identifiable media\n> device.\n> \n> Signed-off-by: Niklas Söderlund \n> ---\n> src/libcamera/device_enumerator.cpp | 9 +++++++++\n> 1 file changed, 9 insertions(+)\n> \n> diff --git a/src/libcamera/device_enumerator.cpp b/src/libcamera/device_enumerator.cpp\n> index 55c510e3b79a415b..e0b19313552f6aa5 100644\n> --- a/src/libcamera/device_enumerator.cpp\n> +++ b/src/libcamera/device_enumerator.cpp\n> @@ -52,6 +52,15 @@ namespace libcamera {\n> *\n> * The description is meant to be filled by pipeline managers and passed to a\n> * device enumerator to find matching media devices.\n> + *\n> + * A DeviceMatch is created with a specific Linux device driver in mind, therefor\n\ns/therefor/therefore/\n\n(The spelling therefor exists but is archaic)\n\n> + * the name of the driver is a required property. One or more Entity names can\n> + * be added the matching with \\a add().\n\nMaybe \"can be added as match criteria\" ?\n\n> + *\n> + * A pipeline handler should add as many entities to a DeviceMatch so that the\n> + * media device can be uniquely identified in case a Linux device driver is\n> + * capable of producing different media graphs as a result of different hardware\n> + * versions or other circumstances.\n\nHow about phrasing that as a recommended usage ?\n\nPipeline handlers are recommended to add entities to DeviceMatch as\nappropriare to ensure that the media device they need can be uniquely\nidentified. This is useful when the corresponding kernel driver can\nproduce different graphs, for instance as a result of different driver\nversions or hardware configurations, and not all those graphs are\nsuitable for a pipeline handler.\ndifferent hardware versions\n\n> */","headers":{"Return-Path":"","Received":["from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 649A560B2D\n\tfor ;\n\tFri, 18 Jan 2019 01:12:54 +0100 (CET)","from pendragon.ideasonboard.com\n\t(dfj612yhrgyx302h3jwwy-3.rev.dnainternet.fi\n\t[IPv6:2001:14ba:21f5:5b00:ce28:277f:58d7:3ca4])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id CB1B253F;\n\tFri, 18 Jan 2019 01:12:53 +0100 (CET)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1547770374;\n\tbh=JflUjvrWanWoVScR2iRHVrp4lHkPCiDXg1tiI7ggKtc=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=NxrzwXucLbcxl8dKiAgqikhDLOLkb8M6mS59TsIaRAaaJb4SUrtLL9qIScbAwut2j\n\tpyiK4a5qzhzEFyLKqqHg7UnGFwsvheaZPAsJffMQ5epkVMZHJi9YUamgfi1K1+zOqI\n\tk2rTiDyCHxHtcY0DYGphB9SqRE72d+eXSFmwu1Fw=","Date":"Fri, 18 Jan 2019 02:12:54 +0200","From":"Laurent Pinchart ","To":"Niklas =?utf-8?q?S=C3=B6derlund?= ","Cc":"libcamera-devel@lists.libcamera.org","Message-ID":"<20190118001254.GH23244@pendragon.ideasonboard.com>","References":"<20190117230636.16176-1-niklas.soderlund@ragnatech.se>","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Disposition":"inline","Content-Transfer-Encoding":"8bit","In-Reply-To":"<20190117230636.16176-1-niklas.soderlund@ragnatech.se>","User-Agent":"Mutt/1.10.1 (2018-07-13)","Subject":"Re: [libcamera-devel] [PATCH] libcamera: device_enumerator: extend\n\tdocumentation of DeviceMatch","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","X-List-Received-Date":"Fri, 18 Jan 2019 00:12:54 -0000"}}]