From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [213.167.242.64]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A9613192580 for ; Wed, 19 Feb 2025 17:49:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=213.167.242.64 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1739987394; cv=none; b=E3QlEMoX2hz7m609DaT825mmhEQt9gvhEx3leQoI1mB3WPk2rtYKmxj6qrjBRJ5lFruZ+nRY+vFLnUGqfRnuTD0G5SqUv0dfVAtXUl47YgCxXGA4W4hVxD0GYENorkfWHJ/S8AAynFlAWhL/E1jFkWRkSsdBCMyF7560BwP53Fg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1739987394; c=relaxed/simple; bh=ehH8TK1UXsd8MxE8OrYZ4KkZHnWrbgI+iBbISw88D7k=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=Hmiehvr/OEVHewl1PnrpKx24hdZSQtxgAoubwDFiBWZuW/mZBhDdaY6A/BHko1cjIpIhJmRC0gepyBrouZLNCVw7vn65THGxOz8hEMkBN4yS9aU9QvsQoTFrJ461yBetyfymVe6VcuI3pfUwcQeZc/07cscCgBH2yuwGZHtgXkw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=ideasonboard.com; spf=pass smtp.mailfrom=ideasonboard.com; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b=qxjpHJ4t; arc=none smtp.client-ip=213.167.242.64 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=ideasonboard.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ideasonboard.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="qxjpHJ4t" Received: from pendragon.ideasonboard.com (81-175-209-231.bb.dnainternet.fi [81.175.209.231]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 2F8503E; Wed, 19 Feb 2025 18:48:27 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1739987307; bh=ehH8TK1UXsd8MxE8OrYZ4KkZHnWrbgI+iBbISw88D7k=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=qxjpHJ4twUM+bakLBtJEWS9NKWqtOJRbDhVokKIkLnCc+djLVtTuNKfeFHfek2QIY +FbpMh91pxfr91KoQ+pWSRfxuMnwoxq6n30drFS6mJOI4Fz5AnJ3pjn5DUyYIELb3E 7rPsubtTQg07KBH46csT7LagRALlwuBWko7UlXPQ= Date: Wed, 19 Feb 2025 19:49:34 +0200 From: Laurent Pinchart To: Dmitry Baryshkov Cc: Tomi Valkeinen , Vishal Sagar , Anatoliy Klymenko , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Michal Simek , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Geert Uytterhoeven Subject: Re: [PATCH v3 06/11] drm/fourcc: Add DRM_FORMAT_XVUY2101010 Message-ID: <20250219174934.GA22240@pendragon.ideasonboard.com> References: <20250212-xilinx-formats-v3-0-90d0fe106995@ideasonboard.com> <20250212-xilinx-formats-v3-6-90d0fe106995@ideasonboard.com> <7674314f-d95a-433a-81d2-ca78bc199359@ideasonboard.com> <23e71045-1ae2-4e02-9780-aa11a338d481@ideasonboard.com> <20250219150840.GA31825@pendragon.ideasonboard.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: On Wed, Feb 19, 2025 at 07:28:44PM +0200, Dmitry Baryshkov wrote: > On Wed, Feb 19, 2025 at 05:08:40PM +0200, Laurent Pinchart wrote: > > On Wed, Feb 19, 2025 at 04:47:26PM +0200, Tomi Valkeinen wrote: > > > Hi, > > > > > > On 18/02/2025 05:26, Dmitry Baryshkov wrote: > > > > On Mon, Feb 17, 2025 at 10:27:56PM +0200, Tomi Valkeinen wrote: > > > >> Hi, > > > >> > > > >> On 17/02/2025 22:15, Dmitry Baryshkov wrote: > > > >>> On Wed, Feb 12, 2025 at 04:56:10PM +0200, Tomi Valkeinen wrote: > > > >>>> Add XVUY2101010, a 10 bits per component YCbCr format in a 32 bit > > > >>>> container. > > > >>> > > > >>> Is there a more common name for this format? Otherwise googling for it > > > >>> reveals only your series. > > > >> > > > >> In the cover letter I mention the gstreamer names where available (this > > > >> particular format is not in gstreamer). AMD has these in their zynqmp > > > >> documentation (https://docs.amd.com/r/en-US/ug1085-zynq-ultrascale-trm/Video-Packer-Format). > > > >> > > > >> XVUY2101010 is YUV444_10BPC in AMD docs. > > > >> > > > >> X403 is Y444_10LE32 in gstreamer, and YV24_10BPC in AMD docs. > > > >> > > > >> I'm not sure you'll have much more luck googling with those names, though > > > >> =). > > > > > > > > I'm asking, because include/uapi/drm/drm_fourcc.h has a pretty explicit > > > > waiver: GL, Vulkan or other open standards. Otherwise normal > > > > requirements apply and it's required to have an open-source usespace > > > > implementation, etc. > > > > > > I can drop DRM_FORMAT_XVUY2101010 until we get it to gstreamer. I just > > > had it ready, so I thought it's better to include it than leave out. > > > > > > Is the current gstreamer support enough for the other formats to fulfill > > > the userspace requirement? > > > > We've received a green light in the past to add formats to drm_fourcc.h > > that would be used by cameras only. There's no open formal standard there, but > > we have libcamera as a de-facto standard. I would assume GStreamer to be > > enough for a scanout format. > > Thanks for the clarification! To make it extra clear, this format is not currently (as in the mainline kernel, with Tomi's work) used for cameras, but only for display scanout. My comment related to cameras was meant to indicate I expect drm_fourcc.h to not strictly require all formats to match open formal standards. -- Regards, Laurent Pinchart