From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 05E4EC352A1 for ; Wed, 30 Nov 2022 11:31:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=UWT6FssADKzJWQM/gerX7nJhcxRLfiC7R+/vE7xrwcE=; b=3stLbjmunFhJ0n 84fummtR+WhqtzNYdg9SMAuiwOOSMryn7sQrha8CJg59QYWzVDOIaJeWcCp+MY/3ZGyuVgBQg7iap Oyx60faId/bsJ6lxIGmYXAfW/wsAp8Fj3jrvbhBdizrxZqoVN1L5m1SK+H+pc8VHIrJiMlWig1Yip HiBnd5f/07y+lHyn/pQyWRo6hX5IllwoZQZPHzTLxaqhpjAWU6+EAhiSIpayWdNt16/6WX8RpwJr9 LTjJnsehzwvWzxRr7vRxCy4MK8xo7hoGChlJNdYeCXit94LrU4whKS5EpigMH5b5GYjeP2SQec8o/ +ciRZVhcREG/0LYolz+w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1p0LIk-00Fp1w-LP; Wed, 30 Nov 2022 11:30:54 +0000 Received: from mail-wm1-x333.google.com ([2a00:1450:4864:20::333]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1p0LIh-00Fozf-NG for linux-arm-kernel@lists.infradead.org; Wed, 30 Nov 2022 11:30:53 +0000 Received: by mail-wm1-x333.google.com with SMTP id 2-20020a1c0202000000b003d0760654d3so580283wmc.4 for ; Wed, 30 Nov 2022 03:30:49 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ffwll.ch; s=google; h=in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date:from:to:cc :subject:date:message-id:reply-to; bh=4abusOcnTWcYiz6maXQH1YPT40DZgw56TQZW9VHybd4=; b=WKj0j4jysik5wdQCAkoDKJVQYX0buM/n4Na0EISlg4QBBP16I4eeLkFYpQg+lGdZMS oavTPa1L42D3PnFuKD2jGCHHXdcKJKSolowcVXn9G/7QNV+a8N/CCOLF+F5xRPFytN9F rPBIFfp+nOphNHBnGg/ONF7IsIU8XicSU6x6A= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=4abusOcnTWcYiz6maXQH1YPT40DZgw56TQZW9VHybd4=; b=ft5NLac1sXsp/79INOZ4iCUNpKRwi6NETszCtUI1S3NN0WpCDQRRA9r4E49CY+3bN4 CWC6/3EqMdTx3eXTX+Ml0Hu+cJ9xNTIv853BC9F4oTV8thl4TQbAO1l+2f+OCvtMDQyW ypAVNehVDZpTYJypNYshidexluPiRPVvfaJfELyfErM+lujseq0tIMy10yjQuD2EB1bN XN6RQPWHiez5Mu0Cl7XN/H/fbvcM3yx0YEHChg9SU36gZhbJNKRvGOo7khRB9S/v6PYD y97a1TvTQ7yt4zqPFX9JUpdpJgyzCg3D09kquNMoCm0UfAuOcVoFJArIVTqdqrriArJx D/WQ== X-Gm-Message-State: ANoB5pnpGjOgJGtdJKE63UXEUX5rlnHQveE+PUVz00qmPhSw/3Hddspq 6r54dSl+7OPjbUi/+tHgeSzkyQ== X-Google-Smtp-Source: AA0mqf5/6pFEpEVZ8cuAHWpWi5MJb0amF7qqtIA6StIT+vCHu8WQLwRuihsmdXUccFqoI5K6zStKsw== X-Received: by 2002:a05:600c:4a9a:b0:3c6:d811:6cff with SMTP id b26-20020a05600c4a9a00b003c6d8116cffmr48907043wmp.43.1669807847926; Wed, 30 Nov 2022 03:30:47 -0800 (PST) Received: from phenom.ffwll.local (212-51-149-33.fiber7.init7.net. [212.51.149.33]) by smtp.gmail.com with ESMTPSA id u16-20020adff890000000b0024165454262sm1323268wrp.11.2022.11.30.03.30.46 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 30 Nov 2022 03:30:47 -0800 (PST) Date: Wed, 30 Nov 2022 12:30:45 +0100 From: Daniel Vetter To: Hsia-Jun Li Cc: dri-devel@lists.freedesktop.org, nicolas@ndufresne.ca, sakari.ailus@linux.intel.com, airlied@linux.ie, ayaka@soulik.info, linux-kernel@vger.kernel.org, tfiga@chromium.org, helen.koike@collabora.com, linux-media@vger.kernel.org, ezequiel@vanguardiasur.com.ar, tzimmermann@suse.de, ribalda@chromium.org, sebastian.hesselbarth@gmail.com, mchehab@kernel.org, linux-arm-kernel@lists.infradead.org, laurent.pinchart@ideasonboard.com Subject: Re: [PATCH v5 1/2] drm/fourcc: Add Synaptics VideoSmart tiled modifiers Message-ID: Mail-Followup-To: Hsia-Jun Li , dri-devel@lists.freedesktop.org, nicolas@ndufresne.ca, sakari.ailus@linux.intel.com, airlied@linux.ie, ayaka@soulik.info, linux-kernel@vger.kernel.org, tfiga@chromium.org, helen.koike@collabora.com, linux-media@vger.kernel.org, ezequiel@vanguardiasur.com.ar, tzimmermann@suse.de, ribalda@chromium.org, sebastian.hesselbarth@gmail.com, mchehab@kernel.org, linux-arm-kernel@lists.infradead.org, laurent.pinchart@ideasonboard.com References: <20221130092149.102788-1-randy.li@synaptics.com> <20221130092149.102788-2-randy.li@synaptics.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20221130092149.102788-2-randy.li@synaptics.com> X-Operating-System: Linux phenom 5.19.0-2-amd64 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221130_033051_782533_F5CEAE8B X-CRM114-Status: GOOD ( 30.80 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, Nov 30, 2022 at 05:21:48PM +0800, Hsia-Jun Li wrote: > From: "Hsia-Jun(Randy) Li" > > Those modifiers only record the parameters would effort pixel > layout or memory layout. Whether physical memory page mapping > is used is not a part of format. > > Signed-off-by: Hsia-Jun(Randy) Li > --- > include/uapi/drm/drm_fourcc.h | 76 +++++++++++++++++++++++++++++++++++ > 1 file changed, 76 insertions(+) > > diff --git a/include/uapi/drm/drm_fourcc.h b/include/uapi/drm/drm_fourcc.h > index bc056f2d537d..e0905f573f43 100644 > --- a/include/uapi/drm/drm_fourcc.h > +++ b/include/uapi/drm/drm_fourcc.h > @@ -407,6 +407,7 @@ extern "C" { > #define DRM_FORMAT_MOD_VENDOR_ARM 0x08 > #define DRM_FORMAT_MOD_VENDOR_ALLWINNER 0x09 > #define DRM_FORMAT_MOD_VENDOR_AMLOGIC 0x0a > +#define DRM_FORMAT_MOD_VENDOR_SYNAPTICS 0x0b > > /* add more to the end as needed */ > > @@ -1507,6 +1508,81 @@ drm_fourcc_canonicalize_nvidia_format_mod(__u64 modifier) > #define AMD_FMT_MOD_CLEAR(field) \ > (~((__u64)AMD_FMT_MOD_##field##_MASK << AMD_FMT_MOD_##field##_SHIFT)) > > +/* > + * Synaptics VideoSmart modifiers > + * > + * Tiles could be arranged in Groups of Tiles (GOTs), it is a small tile > + * within a tile. GOT size and layout varies based on platform and > + * performance concern. > + * > + * Besides, an 8 length 4 bytes arrary (32 bytes) would be need to store > + * some compression parameters for a compression metadata plane. > + * > + * Further information can be found in > + * Documentation/gpu/synaptics.rst > + * > + * Macro > + * Bits Param Description > + * ---- ----- ----------------------------------------------------------------- > + * > + * 7:0 f Scan direction description. > + * > + * 0 = Invalid > + * 1 = V4, the scan would always start from vertical for 4 pixel > + * then move back to the start pixel of the next horizontal > + * direction. > + * 2 = Reserved for future use. > + * > + * 15:8 m The times of pattern repeat in the right angle direction from > + * the first scan direction. > + * > + * 19:16 p The padding bits after the whole scan, could be zero. > + * > + * 20:20 g GOT packing flag. > + * > + * 23:21 - Reserved for future use. Must be zero. Can you pls fold all the future use reservations into the top end? Also I think it'd be good to at least reserve maybe the top 8 bits or so for a synaptics specific format indicator, so that it's easier to extend this in the future ... -Daniel > + * > + * 27:24 h log2(horizontal) of pixels, in GOTs. > + * > + * 31:28 v log2(vertical) of pixels, in GOTs. > + * > + * 35:32 - Reserved for future use. Must be zero. > + * > + * 36:36 c Compression flag. > + * > + * 55:37 - Reserved for future use. Must be zero. > + * > + */ > + > +#define DRM_FORMAT_MOD_SYNA_V4_TILED fourcc_mod_code(SYNAPTICS, 1) > + > +#define DRM_FORMAT_MOD_SYNA_MTR_LINEAR_2D(f, m, p, g, h, v, c) \ > + fourcc_mod_code(SYNAPTICS, ((__u64)((f) & 0xff) | \ > + ((__u64)((m) & 0xff) << 8) | \ > + ((__u64)((p) & 0xf) << 16) | \ > + ((__u64)((g) & 0x1) << 20) | \ > + ((__u64)((h) & 0xf) << 24) | \ > + ((__u64)((v) & 0xf) << 28) | \ > + ((__u64)((c) & 0x1) << 36))) > + > +#define DRM_FORMAT_MOD_SYNA_V4H1 \ > + DRM_FORMAT_MOD_SYNA_MTR_LINEAR_2D(1, 1, 0, 0, 0, 0, 0) > + > +#define DRM_FORMAT_MOD_SYNA_V4H3P8 \ > + DRM_FORMAT_MOD_SYNA_MTR_LINEAR_2D(1, 3, 8, 0, 0, 0, 0) > + > +#define DRM_FORMAT_MOD_SYNA_V4H1_64L4_COMPRESSED \ > + DRM_FORMAT_MOD_SYNA_MTR_LINEAR_2D(1, 1, 0, 1, 6, 2, 1) > + > +#define DRM_FORMAT_MOD_SYNA_V4H3P8_64L4_COMPRESSED \ > + DRM_FORMAT_MOD_SYNA_MTR_LINEAR_2D(1, 3, 8, 1, 6, 2, 1) > + > +#define DRM_FORMAT_MOD_SYNA_V4H1_128L128_COMPRESSED \ > + DRM_FORMAT_MOD_SYNA_MTR_LINEAR_2D(1, 1, 0, 1, 7, 7, 1) > + > +#define DRM_FORMAT_MOD_SYNA_V4H3P8_128L128_COMPRESSED \ > + DRM_FORMAT_MOD_SYNA_MTR_LINEAR_2D(1, 3, 8, 1, 7, 7, 1) > + > #if defined(__cplusplus) > } > #endif > -- > 2.37.3 > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel