From: Mark Brown <broonie@kernel.org>
To: Maxime Ripard <maxime@cerno.tech>
Cc: Jaroslav Kysela <perex@perex.cz>,
dri-devel@lists.freedesktop.org,
Daniel Vetter <daniel.vetter@intel.com>,
David Airlie <airlied@linux.ie>,
Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
Thomas Zimmermann <tzimmermann@suse.de>,
Liam Girdwood <lgirdwood@gmail.com>,
Takashi Iwai <tiwai@suse.com>,
devicetree@vger.kernel.org,
Dave Stevenson <dave.stevenson@raspberrypi.com>,
Phil Elwell <phil@raspberrypi.com>,
Tim Gover <tim.gover@raspberrypi.com>,
Dom Cobley <dom@raspberrypi.com>,
linux-doc@vger.kernel.org, Eric Anholt <eric@anholt.net>,
Nicolas Saenz Julienne <nsaenzjulienne@suse.de>,
linux-kernel@vger.kernel.org, alsa-devel@alsa-project.org,
Maxime Ripard <mripard@kernel.org>,
Nicolas Saenz Julienne <nsaenz@kernel.org>,
bcm-kernel-feedback-list@broadcom.com,
linux-arm-kernel@lists.infradead.org,
Daniel Vetter <daniel@ffwll.ch>,
linux-rpi-kernel@lists.infradead.org,
Jonathan Corbet <corbet@lwn.net>,
Rob Herring <robh+dt@kernel.org>
Subject: Re: [PATCH v2 04/12] ASoC: hdmi-codec: Add iec958 controls
Date: Wed, 9 Jun 2021 13:43:04 +0100 [thread overview]
Message-ID: <20210609124304.GB5052@sirena.org.uk> (raw)
In-Reply-To: <20210525132354.297468-5-maxime@cerno.tech>
[-- Attachment #1: Type: text/plain, Size: 703 bytes --]
On Tue, May 25, 2021 at 03:23:46PM +0200, Maxime Ripard wrote:
> The IEC958 status bits can be exposed and modified by the userspace
> through dedicated ALSA controls.
>
> This patch implements those controls for the hdmi-codec driver. It
> relies on a default value being setup at probe time that can later be
> overridden by the control put.
This breaks bisection:
/mnt/kernel/sound/soc/codecs/hdmi-codec.c: In function 'hdmi_codec_hw_params':
/mnt/kernel/sound/soc/codecs/hdmi-codec.c:504:50: error: invalid type argument of '->' (have 'struct hdmi_codec_params')
memcpy(hp.iec.status, hcp->iec_status, sizeof(hp->iec_status));
^~
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Mark Brown <broonie@kernel.org>
To: Maxime Ripard <maxime@cerno.tech>
Cc: alsa-devel@alsa-project.org,
Tim Gover <tim.gover@raspberrypi.com>,
Liam Girdwood <lgirdwood@gmail.com>,
David Airlie <airlied@linux.ie>,
dri-devel@lists.freedesktop.org, Takashi Iwai <tiwai@suse.com>,
Eric Anholt <eric@anholt.net>,
Daniel Vetter <daniel.vetter@intel.com>,
Phil Elwell <phil@raspberrypi.com>,
Dave Stevenson <dave.stevenson@raspberrypi.com>,
linux-doc@vger.kernel.org, bcm-kernel-feedback-list@broadcom.com,
devicetree@vger.kernel.org,
Nicolas Saenz Julienne <nsaenz@kernel.org>,
Daniel Vetter <daniel@ffwll.ch>,
Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
Maxime Ripard <mripard@kernel.org>,
Rob Herring <robh+dt@kernel.org>,
linux-rpi-kernel@lists.infradead.org,
linux-arm-kernel@lists.infradead.org,
Dom Cobley <dom@raspberrypi.com>,
Jonathan Corbet <corbet@lwn.net>,
linux-kernel@vger.kernel.org,
Thomas Zimmermann <tzimmermann@suse.de>,
Nicolas Saenz Julienne <nsaenzjulienne@suse.de>
Subject: Re: [PATCH v2 04/12] ASoC: hdmi-codec: Add iec958 controls
Date: Wed, 9 Jun 2021 13:43:04 +0100 [thread overview]
Message-ID: <20210609124304.GB5052@sirena.org.uk> (raw)
In-Reply-To: <20210525132354.297468-5-maxime@cerno.tech>
[-- Attachment #1: Type: text/plain, Size: 703 bytes --]
On Tue, May 25, 2021 at 03:23:46PM +0200, Maxime Ripard wrote:
> The IEC958 status bits can be exposed and modified by the userspace
> through dedicated ALSA controls.
>
> This patch implements those controls for the hdmi-codec driver. It
> relies on a default value being setup at probe time that can later be
> overridden by the control put.
This breaks bisection:
/mnt/kernel/sound/soc/codecs/hdmi-codec.c: In function 'hdmi_codec_hw_params':
/mnt/kernel/sound/soc/codecs/hdmi-codec.c:504:50: error: invalid type argument of '->' (have 'struct hdmi_codec_params')
memcpy(hp.iec.status, hcp->iec_status, sizeof(hp->iec_status));
^~
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Mark Brown <broonie@kernel.org>
To: Maxime Ripard <maxime@cerno.tech>
Cc: Jaroslav Kysela <perex@perex.cz>,
dri-devel@lists.freedesktop.org,
Daniel Vetter <daniel.vetter@intel.com>,
David Airlie <airlied@linux.ie>,
Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
Thomas Zimmermann <tzimmermann@suse.de>,
Liam Girdwood <lgirdwood@gmail.com>,
Takashi Iwai <tiwai@suse.com>,
devicetree@vger.kernel.org,
Dave Stevenson <dave.stevenson@raspberrypi.com>,
Phil Elwell <phil@raspberrypi.com>,
Tim Gover <tim.gover@raspberrypi.com>,
Dom Cobley <dom@raspberrypi.com>,
linux-doc@vger.kernel.org, Eric Anholt <eric@anholt.net>,
Nicolas Saenz Julienne <nsaenzjulienne@suse.de>,
linux-kernel@vger.kernel.org, alsa-devel@alsa-project.org,
Maxime Ripard <mripard@kernel.org>,
Nicolas Saenz Julienne <nsaenz@kernel.org>,
bcm-kernel-feedback-list@broadcom.com,
linux-arm-kernel@lists.infradead.org,
Daniel Vetter <daniel@ffwll.ch>,
linux-rpi-kernel@lists.infradead.org,
Jonathan Corbet <corbet@lwn.net>,
Rob Herring <robh+dt@kernel.org>
Subject: Re: [PATCH v2 04/12] ASoC: hdmi-codec: Add iec958 controls
Date: Wed, 9 Jun 2021 13:43:04 +0100 [thread overview]
Message-ID: <20210609124304.GB5052@sirena.org.uk> (raw)
In-Reply-To: <20210525132354.297468-5-maxime@cerno.tech>
[-- Attachment #1.1: Type: text/plain, Size: 703 bytes --]
On Tue, May 25, 2021 at 03:23:46PM +0200, Maxime Ripard wrote:
> The IEC958 status bits can be exposed and modified by the userspace
> through dedicated ALSA controls.
>
> This patch implements those controls for the hdmi-codec driver. It
> relies on a default value being setup at probe time that can later be
> overridden by the control put.
This breaks bisection:
/mnt/kernel/sound/soc/codecs/hdmi-codec.c: In function 'hdmi_codec_hw_params':
/mnt/kernel/sound/soc/codecs/hdmi-codec.c:504:50: error: invalid type argument of '->' (have 'struct hdmi_codec_params')
memcpy(hp.iec.status, hcp->iec_status, sizeof(hp->iec_status));
^~
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
[-- Attachment #2: Type: text/plain, Size: 176 bytes --]
_______________________________________________
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: Mark Brown <broonie@kernel.org>
To: Maxime Ripard <maxime@cerno.tech>
Cc: alsa-devel@alsa-project.org,
Tim Gover <tim.gover@raspberrypi.com>,
linux-doc@vger.kernel.org, David Airlie <airlied@linux.ie>,
dri-devel@lists.freedesktop.org, Takashi Iwai <tiwai@suse.com>,
Daniel Vetter <daniel.vetter@intel.com>,
Phil Elwell <phil@raspberrypi.com>,
Dave Stevenson <dave.stevenson@raspberrypi.com>,
bcm-kernel-feedback-list@broadcom.com,
devicetree@vger.kernel.org,
Nicolas Saenz Julienne <nsaenz@kernel.org>,
Liam Girdwood <lgirdwood@gmail.com>,
Rob Herring <robh+dt@kernel.org>,
linux-rpi-kernel@lists.infradead.org,
Jaroslav Kysela <perex@perex.cz>,
linux-arm-kernel@lists.infradead.org,
Dom Cobley <dom@raspberrypi.com>,
Jonathan Corbet <corbet@lwn.net>,
linux-kernel@vger.kernel.org,
Thomas Zimmermann <tzimmermann@suse.de>,
Nicolas Saenz Julienne <nsaenzjulienne@suse.de>
Subject: Re: [PATCH v2 04/12] ASoC: hdmi-codec: Add iec958 controls
Date: Wed, 9 Jun 2021 13:43:04 +0100 [thread overview]
Message-ID: <20210609124304.GB5052@sirena.org.uk> (raw)
In-Reply-To: <20210525132354.297468-5-maxime@cerno.tech>
[-- Attachment #1: Type: text/plain, Size: 703 bytes --]
On Tue, May 25, 2021 at 03:23:46PM +0200, Maxime Ripard wrote:
> The IEC958 status bits can be exposed and modified by the userspace
> through dedicated ALSA controls.
>
> This patch implements those controls for the hdmi-codec driver. It
> relies on a default value being setup at probe time that can later be
> overridden by the control put.
This breaks bisection:
/mnt/kernel/sound/soc/codecs/hdmi-codec.c: In function 'hdmi_codec_hw_params':
/mnt/kernel/sound/soc/codecs/hdmi-codec.c:504:50: error: invalid type argument of '->' (have 'struct hdmi_codec_params')
memcpy(hp.iec.status, hcp->iec_status, sizeof(hp->iec_status));
^~
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
next prev parent reply other threads:[~2021-06-09 12:43 UTC|newest]
Thread overview: 130+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-25 13:23 [PATCH v2 00/12] drm/vc4: hdmi: Enable Channel Mapping, IEC958, HBR Passthrough using hdmi-codec Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` [PATCH v2 01/12] ALSA: doc: Clarify IEC958 controls iface Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 15:58 ` Takashi Iwai
2021-05-25 15:58 ` Takashi Iwai
2021-05-25 15:58 ` Takashi Iwai
2021-05-25 15:58 ` Takashi Iwai
2021-05-25 13:23 ` [PATCH v2 02/12] ALSA: iec958: Split status creation and fill Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 16:01 ` Takashi Iwai
2021-05-25 16:01 ` Takashi Iwai
2021-05-25 16:01 ` Takashi Iwai
2021-05-25 16:01 ` Takashi Iwai
2021-05-25 13:23 ` [PATCH v2 03/12] ASoC: hdmi-codec: Rework to support more controls Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-26 10:38 ` Mark Brown
2021-05-26 10:38 ` Mark Brown
2021-05-26 10:38 ` Mark Brown
2021-05-26 10:38 ` Mark Brown
2021-05-25 13:23 ` [PATCH v2 04/12] ASoC: hdmi-codec: Add iec958 controls Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-26 10:38 ` Mark Brown
2021-05-26 10:38 ` Mark Brown
2021-05-26 10:38 ` Mark Brown
2021-05-26 10:38 ` Mark Brown
2021-06-09 12:43 ` Mark Brown [this message]
2021-06-09 12:43 ` Mark Brown
2021-06-09 12:43 ` Mark Brown
2021-06-09 12:43 ` Mark Brown
2021-06-10 12:26 ` Maxime Ripard
2021-06-10 12:26 ` Maxime Ripard
2021-06-10 12:26 ` Maxime Ripard
2021-06-10 12:26 ` Maxime Ripard
2021-05-25 13:23 ` [PATCH v2 05/12] ASoC: hdmi-codec: Add a prepare hook Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-26 10:39 ` Mark Brown
2021-05-26 10:39 ` Mark Brown
2021-05-26 10:39 ` Mark Brown
2021-05-26 10:39 ` Mark Brown
2021-05-31 9:42 ` Maxime Ripard
2021-05-31 9:42 ` Maxime Ripard
2021-05-31 9:42 ` Maxime Ripard
2021-05-31 9:42 ` Maxime Ripard
2021-05-31 11:12 ` Takashi Iwai
2021-05-31 11:12 ` Takashi Iwai
2021-05-31 11:12 ` Takashi Iwai
2021-05-31 11:12 ` Takashi Iwai
2021-06-01 12:36 ` Mark Brown
2021-06-01 12:36 ` Mark Brown
2021-06-01 12:36 ` Mark Brown
2021-06-01 12:36 ` Mark Brown
2021-06-03 8:18 ` Maxime Ripard
2021-06-03 8:18 ` Maxime Ripard
2021-06-03 8:18 ` Maxime Ripard
2021-06-03 8:18 ` Maxime Ripard
2021-05-25 13:23 ` [PATCH v2 06/12] drm/vc4: hdmi: Set HD_CTL_WHOLSMP and HD_CTL_CHALIGN_SET Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-06-01 8:45 ` nicolas saenz julienne
2021-06-01 8:45 ` nicolas saenz julienne
2021-06-01 8:45 ` nicolas saenz julienne
2021-06-01 8:45 ` nicolas saenz julienne
2021-05-25 13:23 ` [PATCH v2 07/12] drm/vc4: hdmi: Set HDMI_MAI_FMT Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-06-01 8:47 ` nicolas saenz julienne
2021-06-01 8:47 ` nicolas saenz julienne
2021-06-01 8:47 ` nicolas saenz julienne
2021-06-01 8:47 ` nicolas saenz julienne
2021-05-25 13:23 ` [PATCH v2 08/12] drm/vc4: hdmi: Set VC4_HDMI_MAI_CONFIG_FORMAT_REVERSE Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-06-01 8:49 ` nicolas saenz julienne
2021-06-01 8:49 ` nicolas saenz julienne
2021-06-01 8:49 ` nicolas saenz julienne
2021-06-01 8:49 ` nicolas saenz julienne
2021-05-25 13:23 ` [PATCH v2 09/12] drm/vc4: hdmi: Remove firmware logic for MAI threshold setting Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-06-01 8:52 ` nicolas saenz julienne
2021-06-01 8:52 ` nicolas saenz julienne
2021-06-01 8:52 ` nicolas saenz julienne
2021-06-01 8:52 ` nicolas saenz julienne
2021-06-01 10:01 ` Dom Cobley
2021-06-01 10:01 ` Dom Cobley
2021-05-25 13:23 ` [PATCH v2 10/12] drm/vc4: hdmi: Register HDMI codec Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-06-01 9:26 ` nicolas saenz julienne
2021-06-01 9:26 ` nicolas saenz julienne
2021-06-01 9:26 ` nicolas saenz julienne
2021-06-01 9:26 ` nicolas saenz julienne
2021-06-23 12:29 ` Maxime Ripard
2021-06-23 12:29 ` Maxime Ripard
2021-06-23 12:29 ` Maxime Ripard
2021-06-23 12:29 ` Maxime Ripard
2021-05-25 13:23 ` [PATCH v2 11/12] drm/vc4: hdmi: Remove redundant variables Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-06-01 9:30 ` nicolas saenz julienne
2021-06-01 9:30 ` nicolas saenz julienne
2021-06-01 9:30 ` nicolas saenz julienne
2021-06-01 9:30 ` nicolas saenz julienne
2021-05-25 13:23 ` [PATCH v2 12/12] ARM: dts: bcm2711: Tune DMA parameters for HDMI audio Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 13:23 ` Maxime Ripard
2021-05-25 15:25 ` Florian Fainelli
2021-05-25 15:25 ` Florian Fainelli
2021-05-25 15:25 ` Florian Fainelli
2021-05-25 15:25 ` Florian Fainelli
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=20210609124304.GB5052@sirena.org.uk \
--to=broonie@kernel.org \
--cc=airlied@linux.ie \
--cc=alsa-devel@alsa-project.org \
--cc=bcm-kernel-feedback-list@broadcom.com \
--cc=corbet@lwn.net \
--cc=daniel.vetter@intel.com \
--cc=daniel@ffwll.ch \
--cc=dave.stevenson@raspberrypi.com \
--cc=devicetree@vger.kernel.org \
--cc=dom@raspberrypi.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=eric@anholt.net \
--cc=lgirdwood@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rpi-kernel@lists.infradead.org \
--cc=maarten.lankhorst@linux.intel.com \
--cc=maxime@cerno.tech \
--cc=mripard@kernel.org \
--cc=nsaenz@kernel.org \
--cc=nsaenzjulienne@suse.de \
--cc=perex@perex.cz \
--cc=phil@raspberrypi.com \
--cc=robh+dt@kernel.org \
--cc=tim.gover@raspberrypi.com \
--cc=tiwai@suse.com \
--cc=tzimmermann@suse.de \
/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.