All of lore.kernel.org
 help / color / mirror / Atom feed
From: AngeloGioacchino Del Regno  <angelogioacchino.delregno@collabora.com>
To: Bo-Chen Chen <rex-bc.chen@mediatek.com>,
	chunkuang.hu@kernel.org, p.zabel@pengutronix.de, daniel@ffwll.ch,
	robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
	mripard@kernel.org, tzimmermann@suse.de, matthias.bgg@gmail.com,
	deller@gmx.de, airlied@linux.ie
Cc: msp@baylibre.com, granquet@baylibre.com, jitao.shi@mediatek.com,
	wenst@chromium.org, ck.hu@mediatek.com, liangxu.xu@mediatek.com,
	dri-devel@lists.freedesktop.org,
	linux-mediatek@lists.infradead.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-fbdev@vger.kernel.org,
	Project_Global_Chrome_Upstream_Group@mediatek.com
Subject: Re: [PATCH v15 04/11] video/hdmi: Add audio_infoframe packing for DP
Date: Wed, 27 Jul 2022 11:35:23 +0200	[thread overview]
Message-ID: <4990e4c4-1158-feff-089c-918c890d35d3@collabora.com> (raw)
In-Reply-To: <20220727045035.32225-5-rex-bc.chen@mediatek.com>

Il 27/07/22 06:50, Bo-Chen Chen ha scritto:
> From: Markus Schneider-Pargmann <msp@baylibre.com>
> 
> Similar to HDMI, DP uses audio infoframes as well which are structured
> very similar to the HDMI ones.
> 
> This patch adds a helper function to pack the HDMI audio infoframe for
> DP, called hdmi_audio_infoframe_pack_for_dp().
> hdmi_audio_infoframe_pack_only() is split into two parts. One of them
> packs the payload only and can be used for HDMI and DP.
> 
> Also constify the frame parameter in hdmi_audio_infoframe_check() as
> it is passed to hdmi_audio_infoframe_check_only() which expects a const.
> 
> Signed-off-by: Markus Schneider-Pargmann <msp@baylibre.com>
> Signed-off-by: Guillaume Ranquet <granquet@baylibre.com>
> Signed-off-by: Bo-Chen Chen <rex-bc.chen@mediatek.com>

Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>


WARNING: multiple messages have this Message-ID (diff)
From: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
To: Bo-Chen Chen <rex-bc.chen@mediatek.com>,
	chunkuang.hu@kernel.org, p.zabel@pengutronix.de, daniel@ffwll.ch,
	robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
	mripard@kernel.org, tzimmermann@suse.de, matthias.bgg@gmail.com,
	deller@gmx.de, airlied@linux.ie
Cc: devicetree@vger.kernel.org, linux-fbdev@vger.kernel.org,
	granquet@baylibre.com, jitao.shi@mediatek.com,
	liangxu.xu@mediatek.com, linux-kernel@vger.kernel.org,
	dri-devel@lists.freedesktop.org, msp@baylibre.com,
	Project_Global_Chrome_Upstream_Group@mediatek.com,
	linux-mediatek@lists.infradead.org, wenst@chromium.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v15 04/11] video/hdmi: Add audio_infoframe packing for DP
Date: Wed, 27 Jul 2022 11:35:23 +0200	[thread overview]
Message-ID: <4990e4c4-1158-feff-089c-918c890d35d3@collabora.com> (raw)
In-Reply-To: <20220727045035.32225-5-rex-bc.chen@mediatek.com>

Il 27/07/22 06:50, Bo-Chen Chen ha scritto:
> From: Markus Schneider-Pargmann <msp@baylibre.com>
> 
> Similar to HDMI, DP uses audio infoframes as well which are structured
> very similar to the HDMI ones.
> 
> This patch adds a helper function to pack the HDMI audio infoframe for
> DP, called hdmi_audio_infoframe_pack_for_dp().
> hdmi_audio_infoframe_pack_only() is split into two parts. One of them
> packs the payload only and can be used for HDMI and DP.
> 
> Also constify the frame parameter in hdmi_audio_infoframe_check() as
> it is passed to hdmi_audio_infoframe_check_only() which expects a const.
> 
> Signed-off-by: Markus Schneider-Pargmann <msp@baylibre.com>
> Signed-off-by: Guillaume Ranquet <granquet@baylibre.com>
> Signed-off-by: Bo-Chen Chen <rex-bc.chen@mediatek.com>

Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>


WARNING: multiple messages have this Message-ID (diff)
From: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
To: Bo-Chen Chen <rex-bc.chen@mediatek.com>,
	chunkuang.hu@kernel.org, p.zabel@pengutronix.de, daniel@ffwll.ch,
	robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
	mripard@kernel.org, tzimmermann@suse.de, matthias.bgg@gmail.com,
	deller@gmx.de, airlied@linux.ie
Cc: msp@baylibre.com, granquet@baylibre.com, jitao.shi@mediatek.com,
	wenst@chromium.org, ck.hu@mediatek.com, liangxu.xu@mediatek.com,
	dri-devel@lists.freedesktop.org,
	linux-mediatek@lists.infradead.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-fbdev@vger.kernel.org,
	Project_Global_Chrome_Upstream_Group@mediatek.com
Subject: Re: [PATCH v15 04/11] video/hdmi: Add audio_infoframe packing for DP
Date: Wed, 27 Jul 2022 11:35:23 +0200	[thread overview]
Message-ID: <4990e4c4-1158-feff-089c-918c890d35d3@collabora.com> (raw)
In-Reply-To: <20220727045035.32225-5-rex-bc.chen@mediatek.com>

Il 27/07/22 06:50, Bo-Chen Chen ha scritto:
> From: Markus Schneider-Pargmann <msp@baylibre.com>
> 
> Similar to HDMI, DP uses audio infoframes as well which are structured
> very similar to the HDMI ones.
> 
> This patch adds a helper function to pack the HDMI audio infoframe for
> DP, called hdmi_audio_infoframe_pack_for_dp().
> hdmi_audio_infoframe_pack_only() is split into two parts. One of them
> packs the payload only and can be used for HDMI and DP.
> 
> Also constify the frame parameter in hdmi_audio_infoframe_check() as
> it is passed to hdmi_audio_infoframe_check_only() which expects a const.
> 
> Signed-off-by: Markus Schneider-Pargmann <msp@baylibre.com>
> Signed-off-by: Guillaume Ranquet <granquet@baylibre.com>
> Signed-off-by: Bo-Chen Chen <rex-bc.chen@mediatek.com>

Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>


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

  reply	other threads:[~2022-07-27  9:35 UTC|newest]

Thread overview: 89+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-07-27  4:50 [PATCH v15 00/11] drm/mediatek: Add MT8195 DisplayPort driver Bo-Chen Chen
2022-07-27  4:50 ` Bo-Chen Chen
2022-07-27  4:50 ` Bo-Chen Chen
2022-07-27  4:50 ` [PATCH v15 01/11] dt-bindings: mediatek,dp: Add Display Port binding Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  7:23   ` Krzysztof Kozlowski
2022-07-27  7:23     ` Krzysztof Kozlowski
2022-07-27  7:23     ` Krzysztof Kozlowski
2022-07-28  9:39     ` Rex-BC Chen
2022-07-28  9:39       ` Rex-BC Chen
2022-07-28  9:39       ` Rex-BC Chen
2022-07-27  4:50 ` [PATCH v15 02/11] drm/edid: Convert cea_sad helper struct to kernelDoc Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  4:50 ` [PATCH v15 03/11] drm/edid: Add cea_sad helpers for freq/length Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  9:34   ` AngeloGioacchino Del Regno
2022-07-27  9:34     ` AngeloGioacchino Del Regno
2022-07-27  9:34     ` AngeloGioacchino Del Regno
2022-08-02 14:11   ` Jani Nikula
2022-08-02 14:11     ` Jani Nikula
2022-08-02 14:11     ` Jani Nikula
2022-08-05  6:54     ` Rex-BC Chen
2022-08-05  6:54       ` Rex-BC Chen
2022-08-05  6:54       ` Rex-BC Chen
2022-08-05  7:54       ` Jani Nikula
2022-08-05  7:54         ` Jani Nikula
2022-07-27  4:50 ` [PATCH v15 04/11] video/hdmi: Add audio_infoframe packing for DP Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  9:35   ` AngeloGioacchino Del Regno [this message]
2022-07-27  9:35     ` AngeloGioacchino Del Regno
2022-07-27  9:35     ` AngeloGioacchino Del Regno
2022-07-27  4:50 ` [PATCH v15 05/11] drm/mediatek: Add MT8195 Embedded DisplayPort driver Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  9:36   ` AngeloGioacchino Del Regno
2022-07-27  9:36     ` AngeloGioacchino Del Regno
2022-07-27  9:36     ` AngeloGioacchino Del Regno
2022-08-02  2:56   ` CK Hu
2022-08-02  2:56     ` CK Hu
2022-08-02  2:56     ` CK Hu
2022-08-02  5:09   ` CK Hu
2022-08-02  5:09     ` CK Hu
2022-08-02  5:09     ` CK Hu
2022-08-05  8:22     ` Rex-BC Chen
2022-08-05  8:22       ` Rex-BC Chen
2022-08-05  8:22       ` Rex-BC Chen
2022-08-02  7:57   ` CK Hu
2022-08-02  7:57     ` CK Hu
2022-08-02  7:57     ` CK Hu
2022-07-27  4:50 ` [PATCH v15 06/11] drm/mediatek: Add MT8195 External DisplayPort support Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  9:38   ` AngeloGioacchino Del Regno
2022-07-27  9:38     ` AngeloGioacchino Del Regno
2022-07-27  9:38     ` AngeloGioacchino Del Regno
2022-08-02  2:37   ` CK Hu
2022-08-02  2:37     ` CK Hu
2022-08-02  2:37     ` CK Hu
2022-08-03  5:47     ` Rex-BC Chen
2022-08-03  5:47       ` Rex-BC Chen
2022-08-03  5:47       ` Rex-BC Chen
2022-07-27  4:50 ` [PATCH v15 07/11] drm/mediatek: Add retry to prevent misjudgment for sink devices Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  9:40   ` AngeloGioacchino Del Regno
2022-07-27  9:40     ` AngeloGioacchino Del Regno
2022-07-27  9:40     ` AngeloGioacchino Del Regno
2022-07-28  9:40     ` Rex-BC Chen
2022-07-28  9:40       ` Rex-BC Chen
2022-07-28  9:40       ` Rex-BC Chen
2022-08-05  7:10       ` Rex-BC Chen
2022-08-05  7:10         ` Rex-BC Chen
2022-08-05  7:10         ` Rex-BC Chen
2022-07-27  4:50 ` [PATCH v15 08/11] drm/mediatek: add hpd debounce Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  4:50 ` [PATCH v15 09/11] drm/mediatek: set monitor to DP_SET_POWER_D3 to avoid garbage Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  4:50 ` [PATCH v15 10/11] drm/mediatek: DP audio support for MT8195 Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  4:50 ` [PATCH v15 11/11] drm/mediatek: Use cached audio config when changing resolution Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen
2022-07-27  4:50   ` Bo-Chen Chen

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=4990e4c4-1158-feff-089c-918c890d35d3@collabora.com \
    --to=angelogioacchino.delregno@collabora.com \
    --cc=Project_Global_Chrome_Upstream_Group@mediatek.com \
    --cc=airlied@linux.ie \
    --cc=chunkuang.hu@kernel.org \
    --cc=ck.hu@mediatek.com \
    --cc=daniel@ffwll.ch \
    --cc=deller@gmx.de \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=granquet@baylibre.com \
    --cc=jitao.shi@mediatek.com \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=liangxu.xu@mediatek.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-fbdev@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=matthias.bgg@gmail.com \
    --cc=mripard@kernel.org \
    --cc=msp@baylibre.com \
    --cc=p.zabel@pengutronix.de \
    --cc=rex-bc.chen@mediatek.com \
    --cc=robh+dt@kernel.org \
    --cc=tzimmermann@suse.de \
    --cc=wenst@chromium.org \
    /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.