Hi Jose
Cc: Mark, Thierry, Daniel
> > From: Kuninori Morimoto
> >
> > Current dw-hdmi is supporting sound via AHB bus, but it has
> > I2S audio feature too. This patch adds I2S audio support to dw-hdmi.
> > This HDMI I2S is supported by using ALSA SoC
Hi Mark
> > Mark, Thierry, Daniel
> > I wonder who can be maintainer for this patch ??
>
> It's a DRM patch so I'd expect someone in the DRM subsystem.
OK, I see.
But, I will keep Cc to you for this patch-set.
Hi Russell
> > +static int snd_dw_hdmi_probe(struct platform_device *pdev)
> > +{
> > + struct dw_hdmi_i2s_audio_data *audio = pdev->dev.platform_data;
> > + struct platform_device_info pdevinfo;
> > + struct hdmi_codec_pdata pdata;
> > +
> > + pdata.ops = &dw_hdmi_i2s_ops;
Hi
> These are DesignWare HDMI I2S support patches.
> It will use ALSA SoC hdmi-codec driver, but we can't use it as-is.
> So, 2), 3) patches modify hdmi-codec style.
>
> Kuninori Morimoto (3):
> 1) drm: bridge: add DesignWare HDMI I2S audio support
> 2) ASoC: hdm
under different maintainer.
Kuninori Morimoto (2):
1) ASoC: hdmi-codec: enable multi probe for same device
2) drm: bridge: add DesignWare HDMI I2S audio support
drivers/gpu/drm/bridge/Kconfig | 8
drivers/gpu/drm/bridge/Makefile| 1 +
drivers/gp
From: Kuninori Morimoto
hdmi-codec driver is common HDMI sound driver,
but it doesn't care about multi sound ports.
For example, hdmi-codec driver is supporting 1 I2S and 1 SPDIF ports,
so, we can't use this driver if HDMI has 2 or more I2S ports.
And we would like to use multi dete
From: Kuninori Morimoto
Current dw-hdmi is supporting sound via AHB bus, but it has
I2S audio feature too. This patch adds I2S audio support to dw-hdmi.
This HDMI I2S is supported by using ALSA SoC common HDMI encoder
driver.
Signed-off-by: Kuninori Morimoto
---
v1 -> v2
- tidyup ret
rt.
2) patch is based on it.
Difficult is that these 2 patches are under different maintainer.
1) Mark ?
2) Archit ?
Kuninori Morimoto (2):
1) ASoC: hdmi-codec: enable multi probe for same device
2) drm: bridge: add DesignWare HDMI I2S audio support
drivers/gpu/drm/b
Hi Daniel
> > > > Mark, Thierry, Daniel
> > > > I wonder who can be maintainer for this patch ??
> > >
> > > It's a DRM patch so I'd expect someone in the DRM subsystem.
> >
> > OK, I see.
> > But, I will keep Cc to you for this patch-set.
>
> Archit Tajena is the maintainer of last resort for
From: Kuninori Morimoto
hdmi-codec driver is common HDMI sound driver,
but it doesn't care about multi sound ports.
For example, hdmi-codec driver is supporting 1 I2S and 1 SPDIF ports,
so, we can't use this driver if HDMI has 2 or more I2S ports.
And we would like to use multi dete
From: Kuninori Morimoto
Current dw-hdmi is supporting sound via AHB bus, but it has
I2S audio feature too. This patch adds I2S audio support to dw-hdmi.
This HDMI I2S is supported by using ALSA SoC common HDMI encoder
driver.
Tested-by: Jose Abreu
Signed-off-by: Kuninori Morimoto
---
v1
From: Kuninori Morimoto
This patch updates license to use SPDX-License-Identifier
instead of verbose license text.
Signed-off-by: Kuninori Morimoto
---
few weeks passed, nothing happen. I re-post this patch again.
I added Andrew on Cc
drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c | 5
(I added these address again on this mail).
> > ... what's happening ??
>
> I've noticed that several mailing lists sometimes drop addresses from Cc: in
> the e-mail content (but still forwards mails to those addresses). I haven't
> investigated why.
Hmm...
I hope this mail is still reaching to Maintainer
Best regards
---
Kuninori Morimoto
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
Hi
> > >> This patch updates license to use SPDX-License-Identifier
> > >> instead of verbose license text.
> > >>
> > >> Signed-off-by: Kuninori Morimoto
> > >
> > > Reviewed-by: Laurent Pinchart
> > >
> > >
Best regards
---
Kuninori Morimoto
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
my repository in the
> drm/du/base branch.
OK, Thanks.
Nice to know
Best regards
---
Kuninori Morimoto
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
Hi Sven
Thank you for your patch
> In some situations, codec configuration will depend on the
> bclk_ratio generated by the cpu dai. The tda998x hdmi transmitter
> is an example of this.
>
> Allow simple-card to set the bclk_ratio via the 'bclk-slot-ratio'
> devicetree property, which describes
MMON_CLK
You can remove sound/soc/sh/fsi.c
It is for SH SoC.
Thank you for your help !!
Best regards
---
Kuninori Morimoto
From: Kuninori Morimoto
Current dw-hdmi is supporting sound via AHB bus, but it has
I2S audio feature too. This patch adds I2S audio support to dw-hdmi.
This HDMI I2S is supported by using ALSA SoC common HDMI encoder
driver.
Tested-by: Jose Abreu
Signed-off-by: Kuninori Morimoto
---
v1
Hi Laurent, John,
> > Here's what I'm using to get it working:
> > https://git.linaro.org/people/john.stultz/android-dev.git/commitdiff/1024cb4
> > 85b6f00a7e355ce60425f04a584481148
>
> Thank you.
>
> We need to standardize DT bindings for HDMI sound output. Morimoto-san, could
> you have a lo
Hi Russell again
> > > +static struct platform_driver snd_dw_hdmi_driver = {
> > > + .probe = snd_dw_hdmi_probe,
> >
> > The driver must have a .remove function, because the platform device it
> > is binding against can appear and disappear.
>
> Thank you for your feedback
> OK, I will add it
Hi Russell
> > +static struct platform_driver snd_dw_hdmi_driver = {
> > + .probe = snd_dw_hdmi_probe,
>
> The driver must have a .remove function, because the platform device it
> is binding against can appear and disappear.
Thank you for your feedback
OK, I will add it
Hi John
> > Above patch is using normal simple-card style for HDMI sound, but as Laurent
> > said we want to use same DT style for HDMI video and sound (= OF graph
> > style).
> > Fortunately, I posted patch-set for OF graph support on sound card
> > yesterday.
> > Can you check this ?
> > http
From: Kuninori Morimoto
Current dw-hdmi is supporting sound via AHB bus, but it has
I2S audio feature too. This patch adds I2S audio support to dw-hdmi.
This HDMI I2S is supported by using ALSA SoC common HDMI encoder
driver.
Tested-by: Jose Abreu
Signed-off-by: Kuninori Morimoto
---
v2
Hi Russell
> > + platform = platform_device_register_full(&pdevinfo);
> > + if (IS_ERR_OR_NULL(platform))
> > + return PTR_ERR(platform);
>
> This is wrong. If platform is NULL, PTR_ERR() will return zero, which
> will be interpreted as success. Please, avoid using IS_ERR_OR_NULL
full(&pdevinfo);
> > if (IS_ERR(platform))
> > return PTR_ERR(platform);
>
> Yes, that's exactly correct.
Thanks !
I will send v4 patch
Best regards
---
Kuninori Morimoto
From: Kuninori Morimoto
Current dw-hdmi is supporting sound via AHB bus, but it has
I2S audio feature too. This patch adds I2S audio support to dw-hdmi.
This HDMI I2S is supported by using ALSA SoC common HDMI encoder
driver.
Tested-by: Jose Abreu
Signed-off-by: Kuninori Morimoto
---
v3
Hi Russell
> > @@ -11,4 +11,11 @@ struct dw_hdmi_audio_data {
> > u8 *eld;
> > };
> >
> > +struct dw_hdmi_i2s_audio_data {
> > + struct dw_hdmi *hdmi;
> > +
> > + void (*write)(struct dw_hdmi *hdmi, u8 val, int offset);
> > + u8 (*read)(struct dw_hdmi *hdmi, int offset);
> > +};
>
>
Hi Archit, David, and DRM ML
I had heared that Archit is the maintainer of dw-hdmi driver, but am I wrong ??
I'm posting this patch series since half year ago, but no response
from him, and nothing happen (I got review from Russell though).
Is Archit really maintainer ??
OTOH, get_maintainer.pl i
Hi Archit
> > I had heared that Archit is the maintainer of dw-hdmi driver, but am I
> > wrong ??
> > I'm posting this patch series since half year ago, but no response
> > from him, and nothing happen (I got review from Russell though).
> > Is Archit really maintainer ??
> > OTOH, get_maintaine
Fixes: 2761ba6c0925 ("drm: bridge: add DesignWare HDMI I2S audio support")
> Signed-off-by: Arnd Bergmann
> ---
Acked-by: Kuninori Morimoto
> drivers/gpu/drm/bridge/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/gpu/drm/bridge/Kconfig b/dr
From: Kuninori Morimoto
ALSA SoC needs to know connected DAI ID for probing.
It is not a big problem if device/driver was only for sound,
but getting DAI ID will be difficult if device includes both
Video/Sound, like HDMI.
To solve this issue, this patch adds new .get_dai_id callback
on
From: Kuninori Morimoto
This driver's Copyright is under Renesas Solutions Corp
Signed-off-by: Kuninori Morimoto
---
drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-au
From: Kuninori Morimoto
Now, we can use of_graph_get_remote_endpoint(). Let's use it.
Signed-off-by: Kuninori Morimoto
---
based on 4c9c3d595f1bad021cc126d20879df4016801736
("of_graph: add of_graph_get_remote_endpoint()")
drivers/gpu/drm/rcar-du/rcar_du_kms.c | 2 +-
1
each ODD, EVEN timing.
Before this patch, for example 1080i@60Hz, print complete indication
happen in 30Hz.
After this patch, in interlace case, indication coming 60Hz
Best regards
---
Kuninori Morimoto
___
dri-devel mailing list
dri-devel@lists.freed
From: Kuninori Morimoto
DesignWare HDMI driver (= dw-hdmi) is supporting HDMI sound, and its
probe function was calling sound binding function multiple times as
same HDMI device different port.
Because of this behavior, commit 9731f82d601 ("ASoC: hdmi-codec: enable
multi probe for ...&
From: Kuninori Morimoto
ALSA SoC needs to know connected DAI ID for detecting.
It is not a big problem if device/driver was only for sound,
but getting DAI ID will be difficult if device includes both
Video/Sound, like HDMI.
To solve this issue, this patch adds new snd_soc_get_dai_id() and
its
From: Kuninori Morimoto
ALSA SoC needs to know connected DAI ID for probing.
It is not a big problem if device/driver was only for sound,
but getting DAI ID will be difficult if device includes both
Video/Sound, like HDMI.
To solve this issue, this patch adds new .get_dai_id callback
on
ID <-> Sound DAI
matching. 1) - 4) are for these purpose.
1) - 4) : ASoC side patch for hdmi-codec
5) : V4L2 side patch for SYNOPSYS dw-hdmi
Kuninori Morimoto (5):
1) ASoC: add snd_soc_get_dai_id() function
2) ASoC: simple-card-utils: support snd_soc_get_dai_id()
3) ASoC: hdmi
From: Kuninori Morimoto
ALSA SoC needs to know connected DAI ID for detecting.
It is not a big problem if device/driver was only for sound,
but getting DAI ID will be difficult if device includes both
Video/Sound, like HDMI.
To solve this issue, this patch adds new snd_soc_get_dai_id() and
its
From: Kuninori Morimoto
ALSA SoC needs to know connected DAI ID for probing.
It is not a big problem if device/driver was only for sound,
but getting DAI ID will be difficult if device includes both
Video/Sound, like HDMI.
To solve this issue, this patch adds new .get_dai_id callback
on
stand what's going on with the graph code this seems to
> make sense to me. How do we want to go about handling the patch?
This is comment to me ? or DRM maintainer ?
If to me, any case (pickup by Mark, or by DRM maintainer) is OK for me
Best regards
---
Kuninori Morimoto
_
From: Kuninori Morimoto
Current dw-hdmi is supporting sound via AHB bus, but it has
I2S audio feature too. This patch adds I2S audio support to dw-hdmi.
This HDMI I2S is supported by using ALSA SoC common HDMI encoder
driver.
Signed-off-by: Kuninori Morimoto
---
drivers/gpu/drm/bridge
Hi Mark
Can I have feedback about this patch ?
> From: Kuninori Morimoto
>
> Current dw-hdmi is supporting sound via AHB bus, but it has
> I2S audio feature too. This patch adds I2S audio support to dw-hdmi.
> This HDMI I2S is supported by using ALSA SoC common HDMI e
From: Kuninori Morimoto
EPROBE_DEFER is not error, thus, error message on kernel log on this
case is confusable for user. Prints it only error cases.
Signed-off-by: Kuninori Morimoto
---
drivers/gpu/drm/rcar-du/rcar_du_drv.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff
Hi David, Laurent
These removes unneeded error message from Renesas DU driver.
Current this unneeded error message makes user confuse.
Kuninori Morimoto (2):
drm: rcar-du: error message is not needed for drm_vblank_init()
drm: rcar-du: error message is not needed for EPROBE_DEFER
Hi Laurent
> > From: Kuninori Morimoto
> >
> > EPROBE_DEFER is not error, thus, error message on kernel log on this
> > case is confusable for user. Prints it only error cases.
> >
> > Signed-off-by: Kuninori Morimoto
> > ---
> > drivers/gpu/
From: Kuninori Morimoto
The only reason drm_vblank_init() could return an error at the
moment is a kcalloc() failure.
So we can remove current error message completely.
Signed-off-by: Kuninori Morimoto
---
v1 -> v2
- just removed error message
drivers/gpu/drm/rcar-du/rcar_du_drv.c
From: Kuninori Morimoto
EPROBE_DEFER is not error, thus, error message on kernel log on this
case is confusable. Print it only error cases
Signed-off-by: Kuninori Morimoto
---
v1 -> v2
- only for rcar_du_modeset_init()
drivers/gpu/drm/rcar-du/rcar_du_drv.c | 3 ++-
1 file changed
From: Kuninori Morimoto
This patch adds FCP device nodes for R-Car M3-W+ (r8a77961) SoC.
This patch is test on R-Car M3-W+ Salvator-XS board.
Signed-off-by: Kuninori Morimoto
---
arch/arm64/boot/dts/renesas/r8a77961.dtsi | 52 +++
1 file changed, 52 insertions(+)
diff
From: Kuninori Morimoto
Document the R-Car M3-W+ (R8A77961) SoC in the R-Car DU bindings.
Signed-off-by: Kuninori Morimoto
---
Documentation/devicetree/bindings/display/renesas,du.txt | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/display/renesas,du.txt
From: Kuninori Morimoto
This patch adds VSP device nodes for R-Car M3-W+ (r8a77961) SoC.
This patch is test on R-Car M3-W+ Salvator-XS board.
Signed-off-by: Kuninori Morimoto
---
arch/arm64/boot/dts/renesas/r8a77961.dtsi | 55 +++
1 file changed, 55 insertions(+)
diff
From: Kuninori Morimoto
This patch adds R-Car M3-W+ (R8A77961) SoC bindings.
Signed-off-by: Kuninori Morimoto
---
.../devicetree/bindings/display/bridge/renesas,dw-hdmi.txt | 1 +
1 file changed, 1 insertion(+)
diff --git
a/Documentation/devicetree/bindings/display/bridge/renesas,dw
Hi Geert, Laurent
These patches enables HDMI Display/Sound for R8A77961.
These are tested on R-Car M3-W+ Salvator-XS board.
Kuninori Morimoto (9):
dt-bindings: display: renesas: du: Document the r8a77961 bindings
dt-bindings: display: renesas: dw-hdmi: Add R8A77961 support
drm: rcar-du
From: Kuninori Morimoto
This patch adds R-Car M3-W+ (R8A77961) support which has
compatible to R-Car M3-W (R8A77960).
Signed-off-by: Kuninori Morimoto
---
drivers/gpu/drm/rcar-du/rcar_du_drv.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/rcar-du/rcar_du_drv.c
b
From: Kuninori Morimoto
This patch adds DU device nodes for R-Car M3-W+ (r8a77961) SoC.
This patch is test on R-Car M3-W+ Salvator-XS board.
Signed-off-by: Kuninori Morimoto
---
arch/arm64/boot/dts/renesas/r8a77961.dtsi | 13 -
1 file changed, 12 insertions(+), 1 deletion
From: Kuninori Morimoto
This patch enables HDMI Sound on R-Car M3-W+ Salvator-XS board.
This patch is test on R-Car M3-W+ Salvator-XS board.
This reverts commit b997613fad58a03588f0f64a3d86db6c5bd76dd2.
Signed-off-by: Kuninori Morimoto
---
.../boot/dts/renesas/r8a77961-salvator-xs.dts | 29
From: Kuninori Morimoto
This patch enables HDMI Display on R-Car M3-W+ Salvator-XS board.
This patch is test on R-Car M3-W+ Salvator-XS board.
Signed-off-by: Kuninori Morimoto
---
.../boot/dts/renesas/r8a77961-salvator-xs.dts | 28 +++
1 file changed, 28 insertions(+)
diff
From: Kuninori Morimoto
This patch adds HDMI device nodes for R-Car M3-W+ (r8a77961) SoC.
This patch is test on R-Car M3-W+ Salvator-XS board.
Signed-off-by: Kuninori Morimoto
---
arch/arm64/boot/dts/renesas/r8a77961.dtsi | 12 +++-
1 file changed, 11 insertions(+), 1 deletion
w
> describing the port mappings.
Oops indeed.
Thank you for pointing it, I didn't noticed about it.
Will do in v2
Thank you for your help !!
Best regards
---
Kuninori Morimoto
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
Hi Kieran
> > From: Kuninori Morimoto
> >
> > This patch adds VSP device nodes for R-Car M3-W+ (r8a77961) SoC.
> > This patch is test on R-Car M3-W+ Salvator-XS board.
> >
> > Signed-off-by: Kuninori Morimoto
> > ---
(snip)
> Do we keep the items
new patch (= [02/10])
- add Geert's Reviewed-by (= [03/10])
- address sorting for VSP node (= [06/10])
Kuninori Morimoto (10):
dt-bindings: display: renesas: du: Document the r8a77961 bindings
dt-bindings: display: renesas: dw-hdmi: tidyup example compat
From: Kuninori Morimoto
This patch adds FCP device nodes for R-Car M3-W+ (r8a77961) SoC.
This patch was tested on R-Car M3-W+ Salvator-XS board.
Signed-off-by: Kuninori Morimoto
---
arch/arm64/boot/dts/renesas/r8a77961.dtsi | 52 +++
1 file changed, 52 insertions
From: Kuninori Morimoto
This patch adds R-Car M3-W+ (R8A77961) support which has
compatible to R-Car M3-W (R8A77960).
Signed-off-by: Kuninori Morimoto
---
drivers/gpu/drm/rcar-du/rcar_du_drv.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/rcar-du/rcar_du_drv.c
b
From: Kuninori Morimoto
This patch adds VSP device nodes for R-Car M3-W+ (r8a77961) SoC.
This patch was tested on R-Car M3-W+ Salvator-XS board.
Signed-off-by: Kuninori Morimoto
---
arch/arm64/boot/dts/renesas/r8a77961.dtsi | 55 +++
1 file changed, 55 insertions
From: Kuninori Morimoto
Document the R-Car M3-W+ (R8A77961) SoC in the R-Car DU bindings.
Signed-off-by: Kuninori Morimoto
---
Documentation/devicetree/bindings/display/renesas,du.txt | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Documentation/devicetree/bindings/display/renesas
From: Kuninori Morimoto
This patch adds R-Car M3-W+ (R8A77961) SoC bindings.
Signed-off-by: Kuninori Morimoto
Reviewed-by: Geert Uytterhoeven
---
.../devicetree/bindings/display/bridge/renesas,dw-hdmi.txt | 1 +
1 file changed, 1 insertion(+)
diff --git
a/Documentation/devicetree
From: Kuninori Morimoto
required is "renesas,r8a7795-hdmi", instead of "renesas,r8a7795-dw-hdmi"
Signed-off-by: Kuninori Morimoto
---
.../devicetree/bindings/display/bridge/renesas,dw-hdmi.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/Docu
From: Kuninori Morimoto
This patch enables HDMI Display on R-Car M3-W+ Salvator-XS board.
Signed-off-by: Kuninori Morimoto
---
.../boot/dts/renesas/r8a77961-salvator-xs.dts | 28 +++
1 file changed, 28 insertions(+)
diff --git a/arch/arm64/boot/dts/renesas/r8a77961-salvator
From: Kuninori Morimoto
This patch adds DU device nodes for R-Car M3-W+ (r8a77961) SoC.
This patch was tested on R-Car M3-W+ Salvator-XS board.
Signed-off-by: Kuninori Morimoto
---
arch/arm64/boot/dts/renesas/r8a77961.dtsi | 13 -
1 file changed, 12 insertions(+), 1 deletion
From: Kuninori Morimoto
This patch adds HDMI device nodes for R-Car M3-W+ (r8a77961) SoC.
This patch was tested on R-Car M3-W+ Salvator-XS board.
Signed-off-by: Kuninori Morimoto
---
arch/arm64/boot/dts/renesas/r8a77961.dtsi | 12 +++-
1 file changed, 11 insertions(+), 1 deletion
From: Kuninori Morimoto
This patch enables HDMI Sound on R-Car M3-W+ Salvator-XS board.
This reverts commit b997613fad58a03588f0f64a3d86db6c5bd76dd2.
Signed-off-by: Kuninori Morimoto
---
.../boot/dts/renesas/r8a77961-salvator-xs.dts | 29 +++
1 file changed, 29 insertions
Hi Laurent
Thank you for your review
> > From: Kuninori Morimoto
> >
> > This patch enables HDMI Sound on R-Car M3-W+ Salvator-XS board.
> >
> > This reverts commit b997613fad58a03588f0f64a3d86db6c5bd76dd2.
>
> Which tree can this commit be found in ?
Hi Laurent
> > > > From: Kuninori Morimoto
> > > >
> > > > This patch enables HDMI Sound on R-Car M3-W+ Salvator-XS board.
> > > >
> > > > This reverts commit b997613fad58a03588f0f64a3d86db6c5bd76dd2.
> > >
> > >
Hi Laurent,
> > Author: Kuninori Morimoto
> > Date: Tue Sep 8 09:34:11 2020 +0900
> >
> > dt-bindings: display: renesas: dw-hdmi: Tidyup example compatible
> >
> > The DT example erronously uses the "renesas,r8a7795-dw-hdmi", when the
Hi Geert
> > From: Kuninori Morimoto
> >
> > This patch adds DU device nodes for R-Car M3-W+ (r8a77961) SoC.
> > This patch was tested on R-Car M3-W+ Salvator-XS board.
> >
> > Signed-off-by: Kuninori Morimoto
> > ---
(snip
Hi Geert
> > From: Kuninori Morimoto
> >
> > This patch adds FCP device nodes for R-Car M3-W+ (r8a77961) SoC.
> > This patch was tested on R-Car M3-W+ Salvator-XS board.
> >
> > Signed-off-by: Kuninori Morimoto
(snip)
> Missing "iommus
Hi David
ping ?
> These removes unneeded error message from Renesas DU driver.
> Current this unneeded error message makes user confuse.
>
> Kuninori Morimoto (2):
> drm: rcar-du: error message is not needed for drm_vblank_init()
> drm: rcar-du: error message
From: Kuninori Morimoto
Current dw-hdmi is supporting sound via AHB bus, but it has
I2S audio feature too. This patch adds I2S audio support to dw-hdmi.
This HDMI I2S is supported by using ALSA SoC common HDMI encoder
driver.
Signed-off-by: Kuninori Morimoto
---
drivers/gpu/drm/bridge
From: Kuninori Morimoto
Current hdmi-codec driver is assuming that it will be registered
from HDMI driver. Because of this assumption, each callback function
has struct device pointer which is parent device (= HDMI).
Then, it can use dev_get_drvdata() to get private data.
OTOH, on some SoC/HDMI
From: Kuninori Morimoto
hdmi-codec driver is common HDMI sound driver,
but it doesn't care about multi sound ports.
For example, hdmi-codec driver is supporting 1 I2S and 1 SPDIF ports,
so, we can't use this driver if HDMI has 2 or more I2S ports.
And we would like to use multi dete
Hi Mark, Thierry, Russell
These are DesignWare HDMI I2S support patches.
It will use ALSA SoC hdmi-codec driver, but we can't use it as-is.
So, 2), 3) patches modify hdmi-codec style.
Kuninori Morimoto (3):
1) drm: bridge: add DesignWare HDMI I2S audio support
2) ASoC: hdmi-codec: cal
gt; received HDMI sound support?
HDMI sound doesn't use hdmi0_con.
You can do it now, thanks
Best regards
---
Kuninori Morimoto
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
Hi Geert
> > HDMI sound doesn't use hdmi0_con.
> > You can do it now, thanks
>
> Thanks, so hdmi0_con can be moved now.
>
> But HDMI sound does use hdmi0/ports/port@2. right?
Yes
Best regards
---
Kuninori Morimoto
_
From: Kuninori Morimoto
This patch updates license to use SPDX-License-Identifier
instead of verbose license text.
Signed-off-by: Kuninori Morimoto
---
drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/gpu/drm
, and fixed for
DU_DOTCLKIN0 -> DU0
DU_DOTCLKIN1 -> DU1
But on E3 (Ebisu) board, it has only DU_DOTCLKIN0.
We might use like this
DU_DOTCLKIN0 -> DU0
DU_DOTCLKIN0 -> DU1
It is possible to adjust to this situation ?
DIDSR :: PDCSn allows only 0
Best
From: Kuninori Morimoto
This patch updates license to use SPDX-License-Identifier
instead of verbose license text.
Signed-off-by: Kuninori Morimoto
---
2weeks passed. resend
drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff
From: Kuninori Morimoto
This patch updates license to use SPDX-License-Identifier
instead of verbose license text.
Signed-off-by: Kuninori Morimoto
Acked-by: Laurent Pinchart
---
drivers/gpu/drm/shmobile/Kconfig | 1 +
drivers/gpu/drm/shmobile/shmob_drm_backlight.c | 6
From: Kuninori Morimoto
This patch updates license to use SPDX-License-Identifier
instead of verbose license text.
Kconfig doesn't have license line, thus, it is GPL-2.0 as default.
Signed-off-by: Kuninori Morimoto
Acked-by: Laurent Pinchart
---
drivers/gpu/drm/rcar-du/Kc
Hi Laurent
These are not yet appear on Linus tree.
I will re-post it.
Kuninori Morimoto (2):
drm: rcar-du: convert to SPDX identifiers
drm: shmobile: convert to SPDX identifiers
drivers/gpu/drm/rcar-du/Kconfig| 1 +
drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 6
Hi Laurent
> > These are not yet appear on Linus tree.
> > I will re-post it.
>
> Thank you, but there was no need to, they're stored safely in my tree :-)
> I'll
> send a pull request for v4.20.
Thanks. Nice to know.
just to this situation ?
> > DIDSR :: PDCSn allows only 0
>
> I think this would make sense. I'm not sure how to implement that, but I'll
> give it a try. What is the priority ?
Normal priority is very OK, so far.
Thank you
Best regards
---
Kuninori Morimoto
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
xx, NULL);
+ of_graph_get_endpoint_by_regs(xxx, 0, -1);
Link: https://lore.kernel.org/r/20240202174941.ga310089-r...@kernel.org
Signed-off-by: Kuninori Morimoto
Reviewed-by: Laurent Pinchart
---
v1 -> v2
- based on latest linux-next/master
drivers/gpu/drm/drm_of.c | 4 +++-
driv
; v3
- return NULL if it it doesn't have ports / port
- add visible comment on of_graph_get_next_ports()
v1 -> v2
- add each Reviewed-by / Acked-by
- tidyup/update Kernel Docs
- use prev as parameter
- update git-log explanation
- remo
ot;, like below
node {
port { };
};
node {
ports {
port { };
};
};
Signed-off-by: Kuninori Morimoto
---
drivers/of/property.c| 112 +++
include/linu
x27;t check node name (= "endpoint").
To handle endpoint more intuitive, create of_graph_get_next_port_endpoint()
of_graph_get_next_port_endpoint(port1, NULL); // P10
of_graph_get_next_port_endpoint(port1, P10); // P11
of_graph_get_next_port_endpoint(port1, P11); // N
_each function from "endpoint" base to "port" base.
Signed-off-by: Kuninori Morimoto
Acked-by: Mark Brown
---
sound/soc/generic/test-component.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/soc/generic/test-component.c
b/sound/soc/generic/tes
Now we can use new port related functions for port parsing. Use it.
Signed-off-by: Kuninori Morimoto
Acked-by: Mark Brown
---
sound/soc/sh/rcar/core.c | 12 +++-
1 file changed, 3 insertions(+), 9 deletions(-)
diff --git a/sound/soc/sh/rcar/core.c b/sound/soc/sh/rcar/core.c
index
Now we can use new port related functions for port parsing. Use it.
Signed-off-by: Kuninori Morimoto
Acked-by: Mark Brown
---
sound/soc/generic/audio-graph-card.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/sound/soc/generic/audio-graph-card.c
b/sound/soc/generic
Now we can use new port related functions for port parsing. Use it.
Signed-off-by: Kuninori Morimoto
Acked-by: Mark Brown
---
sound/soc/generic/audio-graph-card2.c | 111 --
1 file changed, 49 insertions(+), 62 deletions(-)
diff --git a/sound/soc/generic/audio-graph
1 - 100 of 363 matches
Mail list logo
