All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kevin Hilman <khilman@baylibre.com>
To: Neil Armstrong <narmstrong@baylibre.com>,
	daniel@ffwll.ch, dri-devel@lists.freedesktop.org
Cc: mjourdan@baylibre.com, Neil Armstrong <narmstrong@baylibre.com>,
	linux-kernel@vger.kernel.org, ppaalanen@gmail.com,
	linux-amlogic@lists.infradead.org, brian.starkey@arm.com,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v3 0/7] drm/meson: add support for Amlogic Video FBC
Date: Tue, 24 Mar 2020 15:07:17 -0700	[thread overview]
Message-ID: <7hsghx2yqy.fsf@baylibre.com> (raw)
In-Reply-To: <20200324142016.31824-1-narmstrong@baylibre.com>

Neil Armstrong <narmstrong@baylibre.com> writes:

> Amlogic uses a proprietary lossless image compression protocol and format
> for their hardware video codec accelerators, either video decoders or
> video input encoders.
>
> It considerably reduces memory bandwidth while writing and reading
> frames in memory.
>
> The underlying storage is considered to be 3 components, 8bit or 10-bit
> per component, YCbCr 420, single plane :
> - DRM_FORMAT_YUV420_8BIT
> - DRM_FORMAT_YUV420_10BIT
>
> This modifier will be notably added to DMA-BUF frames imported from the V4L2
> Amlogic VDEC decoder.
>
> At least two layout are supported :
> - Basic: composed of a body and a header
> - Scatter: the buffer is filled with a IOMMU scatter table referring
>   to the encoder current memory layout. This mode if more efficient in terms
>   of memory allocation but frames are not dumpable and only valid during until
>   the buffer is freed and back in control of the encoder
>
> At least two options are supported :
> - Memory saving: when the pixel bpp is 8b, the size of the superblock can
>   be reduced, thus saving memory.
>
> This serie adds the missing register, updated the FBC decoder registers
> content to be committed by the crtc code.
>
> The Amlogic FBC has been tested with compressed content from the Amlogic
> HW VP9 decoder on S905X (GXL), S905D2 (G12A) and S905X3 (SM1) in 8bit
> (Scatter+Mem Saving on G12A/SM1, Mem Saving on GXL) and 10bit
> (Scatter on G12A/SM1, default on GXL).

Tested on meson-sm1-sei610 (VP9 60fps content).

Tested-by: Kevin Hilman <khilman@baylibre.com>

Kevin

_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic

WARNING: multiple messages have this Message-ID (diff)
From: Kevin Hilman <khilman@baylibre.com>
To: Neil Armstrong <narmstrong@baylibre.com>,
	daniel@ffwll.ch, dri-devel@lists.freedesktop.org
Cc: mjourdan@baylibre.com, Neil Armstrong <narmstrong@baylibre.com>,
	linux-kernel@vger.kernel.org, ppaalanen@gmail.com,
	linux-amlogic@lists.infradead.org, brian.starkey@arm.com,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v3 0/7] drm/meson: add support for Amlogic Video FBC
Date: Tue, 24 Mar 2020 15:07:17 -0700	[thread overview]
Message-ID: <7hsghx2yqy.fsf@baylibre.com> (raw)
In-Reply-To: <20200324142016.31824-1-narmstrong@baylibre.com>

Neil Armstrong <narmstrong@baylibre.com> writes:

> Amlogic uses a proprietary lossless image compression protocol and format
> for their hardware video codec accelerators, either video decoders or
> video input encoders.
>
> It considerably reduces memory bandwidth while writing and reading
> frames in memory.
>
> The underlying storage is considered to be 3 components, 8bit or 10-bit
> per component, YCbCr 420, single plane :
> - DRM_FORMAT_YUV420_8BIT
> - DRM_FORMAT_YUV420_10BIT
>
> This modifier will be notably added to DMA-BUF frames imported from the V4L2
> Amlogic VDEC decoder.
>
> At least two layout are supported :
> - Basic: composed of a body and a header
> - Scatter: the buffer is filled with a IOMMU scatter table referring
>   to the encoder current memory layout. This mode if more efficient in terms
>   of memory allocation but frames are not dumpable and only valid during until
>   the buffer is freed and back in control of the encoder
>
> At least two options are supported :
> - Memory saving: when the pixel bpp is 8b, the size of the superblock can
>   be reduced, thus saving memory.
>
> This serie adds the missing register, updated the FBC decoder registers
> content to be committed by the crtc code.
>
> The Amlogic FBC has been tested with compressed content from the Amlogic
> HW VP9 decoder on S905X (GXL), S905D2 (G12A) and S905X3 (SM1) in 8bit
> (Scatter+Mem Saving on G12A/SM1, Mem Saving on GXL) and 10bit
> (Scatter on G12A/SM1, default on GXL).

Tested on meson-sm1-sei610 (VP9 60fps content).

Tested-by: Kevin Hilman <khilman@baylibre.com>

Kevin

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

WARNING: multiple messages have this Message-ID (diff)
From: Kevin Hilman <khilman@baylibre.com>
To: Neil Armstrong <narmstrong@baylibre.com>,
	daniel@ffwll.ch, dri-devel@lists.freedesktop.org
Cc: mjourdan@baylibre.com, Neil Armstrong <narmstrong@baylibre.com>,
	linux-kernel@vger.kernel.org, linux-amlogic@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v3 0/7] drm/meson: add support for Amlogic Video FBC
Date: Tue, 24 Mar 2020 15:07:17 -0700	[thread overview]
Message-ID: <7hsghx2yqy.fsf@baylibre.com> (raw)
In-Reply-To: <20200324142016.31824-1-narmstrong@baylibre.com>

Neil Armstrong <narmstrong@baylibre.com> writes:

> Amlogic uses a proprietary lossless image compression protocol and format
> for their hardware video codec accelerators, either video decoders or
> video input encoders.
>
> It considerably reduces memory bandwidth while writing and reading
> frames in memory.
>
> The underlying storage is considered to be 3 components, 8bit or 10-bit
> per component, YCbCr 420, single plane :
> - DRM_FORMAT_YUV420_8BIT
> - DRM_FORMAT_YUV420_10BIT
>
> This modifier will be notably added to DMA-BUF frames imported from the V4L2
> Amlogic VDEC decoder.
>
> At least two layout are supported :
> - Basic: composed of a body and a header
> - Scatter: the buffer is filled with a IOMMU scatter table referring
>   to the encoder current memory layout. This mode if more efficient in terms
>   of memory allocation but frames are not dumpable and only valid during until
>   the buffer is freed and back in control of the encoder
>
> At least two options are supported :
> - Memory saving: when the pixel bpp is 8b, the size of the superblock can
>   be reduced, thus saving memory.
>
> This serie adds the missing register, updated the FBC decoder registers
> content to be committed by the crtc code.
>
> The Amlogic FBC has been tested with compressed content from the Amlogic
> HW VP9 decoder on S905X (GXL), S905D2 (G12A) and S905X3 (SM1) in 8bit
> (Scatter+Mem Saving on G12A/SM1, Mem Saving on GXL) and 10bit
> (Scatter on G12A/SM1, default on GXL).

Tested on meson-sm1-sei610 (VP9 60fps content).

Tested-by: Kevin Hilman <khilman@baylibre.com>

Kevin
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

WARNING: multiple messages have this Message-ID (diff)
From: Kevin Hilman <khilman@baylibre.com>
To: Neil Armstrong <narmstrong@baylibre.com>,
	daniel@ffwll.ch, dri-devel@lists.freedesktop.org
Cc: ppaalanen@gmail.com, mjourdan@baylibre.com,
	brian.starkey@arm.com, Neil Armstrong <narmstrong@baylibre.com>,
	linux-amlogic@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 0/7] drm/meson: add support for Amlogic Video FBC
Date: Tue, 24 Mar 2020 15:07:17 -0700	[thread overview]
Message-ID: <7hsghx2yqy.fsf@baylibre.com> (raw)
In-Reply-To: <20200324142016.31824-1-narmstrong@baylibre.com>

Neil Armstrong <narmstrong@baylibre.com> writes:

> Amlogic uses a proprietary lossless image compression protocol and format
> for their hardware video codec accelerators, either video decoders or
> video input encoders.
>
> It considerably reduces memory bandwidth while writing and reading
> frames in memory.
>
> The underlying storage is considered to be 3 components, 8bit or 10-bit
> per component, YCbCr 420, single plane :
> - DRM_FORMAT_YUV420_8BIT
> - DRM_FORMAT_YUV420_10BIT
>
> This modifier will be notably added to DMA-BUF frames imported from the V4L2
> Amlogic VDEC decoder.
>
> At least two layout are supported :
> - Basic: composed of a body and a header
> - Scatter: the buffer is filled with a IOMMU scatter table referring
>   to the encoder current memory layout. This mode if more efficient in terms
>   of memory allocation but frames are not dumpable and only valid during until
>   the buffer is freed and back in control of the encoder
>
> At least two options are supported :
> - Memory saving: when the pixel bpp is 8b, the size of the superblock can
>   be reduced, thus saving memory.
>
> This serie adds the missing register, updated the FBC decoder registers
> content to be committed by the crtc code.
>
> The Amlogic FBC has been tested with compressed content from the Amlogic
> HW VP9 decoder on S905X (GXL), S905D2 (G12A) and S905X3 (SM1) in 8bit
> (Scatter+Mem Saving on G12A/SM1, Mem Saving on GXL) and 10bit
> (Scatter on G12A/SM1, default on GXL).

Tested on meson-sm1-sei610 (VP9 60fps content).

Tested-by: Kevin Hilman <khilman@baylibre.com>

Kevin

  parent reply	other threads:[~2020-03-24 22:07 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-03-24 14:20 [PATCH v3 0/7] drm/meson: add support for Amlogic Video FBC Neil Armstrong
2020-03-24 14:20 ` Neil Armstrong
2020-03-24 14:20 ` Neil Armstrong
2020-03-24 14:20 ` Neil Armstrong
2020-03-24 14:20 ` [PATCH v3 1/7] drm/fourcc: Add modifier definitions for describing Amlogic Video Framebuffer Compression Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:20 ` [PATCH v3 2/7] drm/meson: add Amlogic Video FBC registers Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:20 ` [PATCH v3 3/7] drm/meson: overlay: setup overlay for Amlogic FBC Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:20 ` [PATCH v3 4/7] drm/meson: crtc: handle commit of Amlogic FBC frames Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:20 ` [PATCH v3 5/7] drm/fourcc: amlogic: Add modifier definitions for Memory Saving option Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:20 ` [PATCH v3 6/7] drm/meson: overlay: setup overlay for Amlogic FBC Memory Saving mode Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:24   ` Neil Armstrong
2020-03-24 14:24     ` Neil Armstrong
2020-03-24 14:24     ` Neil Armstrong
2020-03-24 14:24     ` Neil Armstrong
2020-03-24 14:20 ` [PATCH v3 7/7] drm/meson: overlay: setup overlay for Amlogic FBC Scatter Memory layout Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 14:20   ` Neil Armstrong
2020-03-24 22:07 ` Kevin Hilman [this message]
2020-03-24 22:07   ` [PATCH v3 0/7] drm/meson: add support for Amlogic Video FBC Kevin Hilman
2020-03-24 22:07   ` Kevin Hilman
2020-03-24 22:07   ` Kevin Hilman
2020-03-25  8:50 ` Neil Armstrong
2020-03-25  8:50   ` Neil Armstrong
2020-03-25  8:50   ` Neil Armstrong
2020-03-25  8:50   ` Neil Armstrong

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=7hsghx2yqy.fsf@baylibre.com \
    --to=khilman@baylibre.com \
    --cc=brian.starkey@arm.com \
    --cc=daniel@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=linux-amlogic@lists.infradead.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mjourdan@baylibre.com \
    --cc=narmstrong@baylibre.com \
    --cc=ppaalanen@gmail.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.