Hi Jiada
Thank you for your patch
> From: Jiada Wang
>
> This patch adds dma request number for busif0 ~ busif7
> to be used by GEN3 series. GEN2 continues to use rxu/txu
> for busif data transfer.
>
> Signed-off-by: Jiada Wang
> ---
> arch/arm64/boot/dts/renesas/r8a7795.dtsi | 164
Hi Jiada
Thank you for your patch
> From: Jiada Wang
>
> This patch adds dma request number for busif0 ~ busif7
> to be used by GEN3 series. GEN2 continues to use rxu/txu
> for busif data transfer.
>
> Signed-off-by: Jiada Wang
> ---
> arch/arm64/boot/dts/renesas/r8a7795.dtsi | 164
Hi Jiada
Thank you for your patch
> ssi0, ssi1, ssi2, ssi3, ssi4 and ssi9 have busif0 to busif7,
> currently only busif0 is used by all ssi.
> But tdm split and tdm ex-split mode, which require busif1 to
> busif7.
>
> This patch adds kctrl in ssi for dai-link, to make it possible
> to
Hi Jiada
Thank you for your patch
> ssi0, ssi1, ssi2, ssi3, ssi4 and ssi9 have busif0 to busif7,
> currently only busif0 is used by all ssi.
> But tdm split and tdm ex-split mode, which require busif1 to
> busif7.
>
> This patch adds kctrl in ssi for dai-link, to make it possible
> to
Hi Jiada
> SSI may use different busif for data transfer, this patch
> adds busif property to each dai stream, to indicate the
> busif used by playback/capture stream.
(snip)
> + io_playback->busif = rsnd_busif_get(priv, dai_i);
> + io_capture->busif = rsnd_busif_get(priv,
Hi Jiada
> SSI may use different busif for data transfer, this patch
> adds busif property to each dai stream, to indicate the
> busif used by playback/capture stream.
(snip)
> + io_playback->busif = rsnd_busif_get(priv, dai_i);
> + io_capture->busif = rsnd_busif_get(priv,
Hi Jiada
Cc: linux-renesas-soc ML
Thank you for your patch
> From: Jiada Wang
>
> Currently ssi driver only request dma channel for SSI_0,
> which is used to transfer data to/from busif0.
>
> But since busif1 to busif7 also maybe used, dedicated dma channels
> are request for data transfer
Hi Jiada
Cc: linux-renesas-soc ML
Thank you for your patch
> From: Jiada Wang
>
> Currently ssi driver only request dma channel for SSI_0,
> which is used to transfer data to/from busif0.
>
> But since busif1 to busif7 also maybe used, dedicated dma channels
> are request for data transfer
From: Kuninori Morimoto
This patch updates license to use SPDX-License-Identifier
instead of verbose license text.
Signed-off-by: Kuninori Morimoto
Reviewed-by: Simon Horman
---
Kishon
2weeks past, resend patch
drivers/phy/renesas/Kconfig | 1 +
drivers/phy/renesas/Makefile | 1 +
2
From: Kuninori Morimoto
This patch updates license to use SPDX-License-Identifier
instead of verbose license text.
Signed-off-by: Kuninori Morimoto
Reviewed-by: Simon Horman
---
Kishon
2weeks past, resend patch
drivers/phy/renesas/Kconfig | 1 +
drivers/phy/renesas/Makefile | 1 +
2
Hi
# I know it is very late response
Acked-by: Kuninori Morimoto
> From: Marcel Ziswiler
>
> Since commit 143b44845d87 ("ASoC: wm9712: replace codec to component")
> "wm9712-codec" got renamed to "wm9712-component", however, this change
>
Hi
# I know it is very late response
Acked-by: Kuninori Morimoto
> From: Marcel Ziswiler
>
> Since commit 143b44845d87 ("ASoC: wm9712: replace codec to component")
> "wm9712-codec" got renamed to "wm9712-component", however, this change
>
form drivers. E.g. on
> Colibri T20 this lead to the following spew upon boot with sound/touch
> being broken:
Oops, my bad...
The platform_driver name is not important,
how about simply rename back it to "wm9712-codec" ?
Best regards
---
Kuninori Morimoto
form drivers. E.g. on
> Colibri T20 this lead to the following spew upon boot with sound/touch
> being broken:
Oops, my bad...
The platform_driver name is not important,
how about simply rename back it to "wm9712-codec" ?
Best regards
---
Kuninori Morimoto
Hi
> From: Andrew Gabbasov
>
> To comply with the style of all kernel messages, add newline
> to the end of every message.
>
> Fixes: 70fb10529f61 ("ASoC: rsnd: add MIX (Mixer) support")
> Signed-off-by: Andrew Gabbasov
> Signed-off-by: Jiada Wang
> --
Hi
> From: Andrew Gabbasov
>
> To comply with the style of all kernel messages, add newline
> to the end of every message.
>
> Fixes: 70fb10529f61 ("ASoC: rsnd: add MIX (Mixer) support")
> Signed-off-by: Andrew Gabbasov
> Signed-off-by: Jiada Wang
> --
Hi Vinod
Thank you for your feedback
> > From: Kuninori Morimoto
> >
> > Current rcar-dmac is using DMAC error interrupt which will handle all
> > channel's error. But in this design, error handling itself will be
> > issue if user want to use virtualization,
Hi Vinod
Thank you for your feedback
> > From: Kuninori Morimoto
> >
> > Current rcar-dmac is using DMAC error interrupt which will handle all
> > channel's error. But in this design, error handling itself will be
> > issue if user want to use virtualization,
Hi Geert
Thank you for your light speed review :)
> > From: Kuninori Morimoto
> >
> > Current rcar-dmac is using DMAC error interrupt which will handle all
> > channel's error. But in this design, error handling itself will be
> > issue if user want to use
Hi Geert
Thank you for your light speed review :)
> > From: Kuninori Morimoto
> >
> > Current rcar-dmac is using DMAC error interrupt which will handle all
> > channel's error. But in this design, error handling itself will be
> > issue if user want to use
Hi
> This patch moves headphone and microphone jack detection gpios from
> simple-card driver. It is preparing for using this feature from other
> drivers.
>
> Signed-off-by: Katsuhiro Suzuki
> ---
For all patches
Acked-by: Kuninori Morimoto
Hi
> This patch moves headphone and microphone jack detection gpios from
> simple-card driver. It is preparing for using this feature from other
> drivers.
>
> Signed-off-by: Katsuhiro Suzuki
> ---
For all patches
Acked-by: Kuninori Morimoto
sorry, it was my English skill error.
Best regards
---
Kuninori Morimoto
sorry, it was my English skill error.
Best regards
---
Kuninori Morimoto
< 0)
> - return ret;
(snip)
> + ret = asoc_simple_card_init_hp(card, >hp_jack, NULL);
> + if (ret < 0)
> + return ret;
> +
> + ret = asoc_simple_card_init_mic(card, >mic_jack, NULL);
> + if (ret < 0)
> + return ret;
I think we want to keep "PREFIX" ?
Best regards
---
Kuninori Morimoto
< 0)
> - return ret;
(snip)
> + ret = asoc_simple_card_init_hp(card, >hp_jack, NULL);
> + if (ret < 0)
> + return ret;
> +
> + ret = asoc_simple_card_init_mic(card, >mic_jack, NULL);
> + if (ret < 0)
> + return ret;
I think we want to keep "PREFIX" ?
Best regards
---
Kuninori Morimoto
imple/audio card are doing samethings in different style.
Thus, I want to synchronized whole simple/audio card.
Best regards
---
Kuninori Morimoto
imple/audio card are doing samethings in different style.
Thus, I want to synchronized whole simple/audio card.
Best regards
---
Kuninori Morimoto
und support?
HDMI sound doesn't use hdmi0_con.
You can do it now, thanks
Best regards
---
Kuninori Morimoto
und support?
HDMI sound doesn't use hdmi0_con.
You can do it now, thanks
Best regards
---
Kuninori Morimoto
Hi Tim, Mark
> >> Could you add some detail to the commit explaining why we need to
> >> replace codec to component? I don't really know what that means.
> >> Please refer to a commit if the ASoC API is changing in some way we
> >> need to catch up with.
> >
> > This is a big transition in the
Hi Tim, Mark
> >> Could you add some detail to the commit explaining why we need to
> >> replace codec to component? I don't really know what that means.
> >> Please refer to a commit if the ASoC API is changing in some way we
> >> need to catch up with.
> >
> > This is a big transition in the
From: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
Now we can replace Codec to Component. Let's do it.
Note:
xxx_codec_xxx() -> xxx_component_xxx()
.idle_bias_off = 0 -> .idle_bias_on = 1
.ignore_pmdown_time = 0 -> .use_
From: Kuninori Morimoto
Now we can replace Codec to Component. Let's do it.
Note:
xxx_codec_xxx() -> xxx_component_xxx()
.idle_bias_off = 0 -> .idle_bias_on = 1
.ignore_pmdown_time = 0 -> .use_pmdown
From: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
We are counting kobject by using kobject_get/put(), but
not have kobject_read() which can be used to check count.
Let's add
Signed-off-by: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
---
include/linux/kobject.h
From: Kuninori Morimoto
We are counting kobject by using kobject_get/put(), but
not have kobject_read() which can be used to check count.
Let's add
Signed-off-by: Kuninori Morimoto
---
include/linux/kobject.h | 1 +
lib/kobject.c | 18 ++
2 files changed, 19
From: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
We are counting of node by using of_node_get/put(), but
not have of_node_read() which can be used to check count.
Let's add it
Signed-off-by: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
---
drivers/of/dy
From: Kuninori Morimoto
We are counting of node by using of_node_get/put(), but
not have of_node_read() which can be used to check count.
Let's add it
Signed-off-by: Kuninori Morimoto
---
drivers/of/dynamic.c | 14 ++
include/linux/of.h | 5 +
2 files changed, 19
Hi Greg, Rob, Frank
We are using {kobject_/of_node_}get/put() for increment/decrement
counter. But, because we don't have counter read() function,
confirming it (mainly for debug purpose) is a little bit difficult.
These patches add read() function for it.
Kuninori Morimoto (2):
kobject: add
Hi Greg, Rob, Frank
We are using {kobject_/of_node_}get/put() for increment/decrement
counter. But, because we don't have counter read() function,
confirming it (mainly for debug purpose) is a little bit difficult.
These patches add read() function for it.
Kuninori Morimoto (2):
kobject: add
From: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
Now we can replace Codec to Component. Let's do it.
Note:
xxx_codec_xxx() -> xxx_component_xxx()
.idle_bias_off = 0 -> .idle_bias_on = 1
.ignore_pmdown_time = 0 -> .use_
From: Kuninori Morimoto
Now we can replace Codec to Component. Let's do it.
Note:
xxx_codec_xxx() -> xxx_component_xxx()
.idle_bias_off = 0 -> .idle_bias_on = 1
.ignore_pmdown_time = 0 -> .use_pmdown
From: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
Now we can replace Codec to Component. Let's do it.
Note:
xxx_codec_xxx() -> xxx_component_xxx()
.idle_bias_off = 0 -> .idle_bias_on = 1
.ignore_pmdown_time = 0 -> .use_
From: Kuninori Morimoto
Now we can replace Codec to Component. Let's do it.
Note:
xxx_codec_xxx() -> xxx_component_xxx()
.idle_bias_off = 0 -> .idle_bias_on = 1
.ignore_pmdown_time = 0 -> .use_pmdown
[cannot apply to v4.16-rc7]
> [if your patch is applied to the wrong git tree, please drop us a note to
> help improve the system]
>
> url:
> https://github.com/0day-ci/linux/commits/Kuninori-Morimoto/media-i2c-wm9090-replace-codec-to-component/20180329-082843
> base: git://lin
[cannot apply to v4.16-rc7]
> [if your patch is applied to the wrong git tree, please drop us a note to
> help improve the system]
>
> url:
> https://github.com/0day-ci/linux/commits/Kuninori-Morimoto/media-i2c-wm9090-replace-codec-to-component/20180329-082843
> base: git://lin
From: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
snd_soc_codec is replaced to snd_soc_component,
and it is not used in this file.
Let's remove it
Signed-off-by: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
---
include/trace/events/asoc.h | 1 -
1 file changed
From: Kuninori Morimoto
snd_soc_codec is replaced to snd_soc_component,
and it is not used in this file.
Let's remove it
Signed-off-by: Kuninori Morimoto
---
include/trace/events/asoc.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/include/trace/events/asoc.h b/include/trace/events
From: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
Now we can replace Codec to Component. Let's do it.
Note:
xxx_codec_xxx() -> xxx_component_xxx()
.idle_bias_off = 0 -> .idle_bias_on = 1
.ignore_pmdown_time = 0 -> .use_
From: Kuninori Morimoto
Now we can replace Codec to Component. Let's do it.
Note:
xxx_codec_xxx() -> xxx_component_xxx()
.idle_bias_off = 0 -> .idle_bias_on = 1
.ignore_pmdown_time = 0 -> .use_pmdown
; Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
> ---
Thank you for your patch
Acked-by: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
Best regards
---
Kuninori Morimoto
; Signed-off-by: Wei Yongjun
> ---
Thank you for your patch
Acked-by: Kuninori Morimoto
Best regards
---
Kuninori Morimoto
Hi Mark,Jernej
> > Ahh.. indeed. Good catch !
> > How about to add such flag ?
> > This is just idea. No tested, No compiled, but can help you ?
>
> I think this makes sense as a patch. We might want to disallow
> allocating components as part of a bigger struct so everything is more
>
Hi Mark,Jernej
> > Ahh.. indeed. Good catch !
> > How about to add such flag ?
> > This is just idea. No tested, No compiled, but can help you ?
>
> I think this makes sense as a patch. We might want to disallow
> allocating components as part of a bigger struct so everything is more
>
Hi Jernej
Thank you for your hard work
> I found the issue. Commit be7ee5f32a9a ("ASoC: soc-generic-dmaengine-pcm:
> replace platform to component") changes struct dmaengine_pcm:
>
> struct dmaengine_pcm {
> struct dma_chan *chan[SNDRV_PCM_STREAM_LAST + 1];
> const struct
Hi Jernej
Thank you for your hard work
> I found the issue. Commit be7ee5f32a9a ("ASoC: soc-generic-dmaengine-pcm:
> replace platform to component") changes struct dmaengine_pcm:
>
> struct dmaengine_pcm {
> struct dma_chan *chan[SNDRV_PCM_STREAM_LAST + 1];
> const struct
evm_extcon_dev_allocate" has relation ?
static int sun4i_usb_phy_probe(struct platform_device *pdev)
{
...
=> data->extcon = devm_extcon_dev_allocate(dev, sun4i_usb_phy0_cable);
...
ret = devm_extcon_dev_register(dev, data->extcon);
...
}
Best regards
---
Kuninori Morimoto
evm_extcon_dev_allocate" has relation ?
static int sun4i_usb_phy_probe(struct platform_device *pdev)
{
...
=> data->extcon = devm_extcon_dev_allocate(dev, sun4i_usb_phy0_cable);
...
ret = devm_extcon_dev_register(dev, data->extcon);
...
}
Best regards
---
Kuninori Morimoto
From: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
Now we can replace Codec to Component. Let's do it.
Note:
xxx_codec_xxx() -> xxx_component_xxx()
.idle_bias_off = 0 -> .idle_bias_on = 1
.ignore_pmdown_time = 0 -> .use_
From: Kuninori Morimoto
Now we can replace Codec to Component. Let's do it.
Note:
xxx_codec_xxx() -> xxx_component_xxx()
.idle_bias_off = 0 -> .idle_bias_on = 1
.ignore_pmdown_time = 0 -> .use_pmdown
From: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
Now we can replace Codec to Component. Let's do it.
Note:
xxx_codec_xxx() -> xxx_component_xxx()
.idle_bias_off = 0 -> .idle_bias_on = 1
.ignore_pmdown_time = 0 -> .use_
From: Kuninori Morimoto
Now we can replace Codec to Component. Let's do it.
Note:
xxx_codec_xxx() -> xxx_component_xxx()
.idle_bias_off = 0 -> .idle_bias_on = 1
.ignore_pmdown_time = 0 -> .use_pmdown
From: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
Now we can replace Codec to Component. Let's do it.
Signed-off-by: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
---
v1 -> v2
- no change
sound/soc/amd/acp-da7219-max98357a.c | 4 ++--
1 file changed, 2 i
From: Kuninori Morimoto
Now we can replace Codec to Component. Let's do it.
Signed-off-by: Kuninori Morimoto
---
v1 -> v2
- no change
sound/soc/amd/acp-da7219-max98357a.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/soc/amd/acp-da7219-max98357a.c
b/so
From: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
Now we can replace Codec to Component. Let's do it.
Signed-off-by: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
---
v1 -> v2
- no change
sound/soc/intel/boards/kbl_da7219_max98357a.c | 12 ++--
1
From: Kuninori Morimoto
Now we can replace Codec to Component. Let's do it.
Signed-off-by: Kuninori Morimoto
---
v1 -> v2
- no change
sound/soc/intel/boards/kbl_da7219_max98357a.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/sound/soc/intel/boa
Hi Mark
These are v2 of replace patch for new added driver on mark/for-next branch.
The diff v1 -> v2 is that compile warning issue was solved on ak driver
Kuninori Morimoto (4):
ASoC: intel: kbl_da7219_max98357: replace codec to component
ASoC: amd: acp-da7219-max98357: replace co
Hi Mark
These are v2 of replace patch for new added driver on mark/for-next branch.
The diff v1 -> v2 is that compile warning issue was solved on ak driver
Kuninori Morimoto (4):
ASoC: intel: kbl_da7219_max98357: replace codec to component
ASoC: amd: acp-da7219-max98357: replace co
Hi Mark
My posted patches had compile issue.
I will post v2 patch.
Please drop these
> These are new added driver on mark/for-next branch,
> but based on old "codec" framework.
> We want to convert it to component. These patches are do it
>
> Kuninori Mori
Hi Mark
My posted patches had compile issue.
I will post v2 patch.
Please drop these
> These are new added driver on mark/for-next branch,
> but based on old "codec" framework.
> We want to convert it to component. These patches are do it
>
> Kuninori Mori
From: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
Now we can replace Codec to Component. Let's do it.
Note:
xxx_codec_xxx() -> xxx_component_xxx()
.idle_bias_off = 0 -> .idle_bias_on = 1
.ignore_pmdown_time = 0 -> .use_
From: Kuninori Morimoto
Now we can replace Codec to Component. Let's do it.
Note:
xxx_codec_xxx() -> xxx_component_xxx()
.idle_bias_off = 0 -> .idle_bias_on = 1
.ignore_pmdown_time = 0 -> .use_pmdown
From: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
Now we can replace Codec to Component. Let's do it.
Note:
xxx_codec_xxx() -> xxx_component_xxx()
.idle_bias_off = 0 -> .idle_bias_on = 1
.ignore_pmdown_time = 0 -> .use_
From: Kuninori Morimoto
Now we can replace Codec to Component. Let's do it.
Note:
xxx_codec_xxx() -> xxx_component_xxx()
.idle_bias_off = 0 -> .idle_bias_on = 1
.ignore_pmdown_time = 0 -> .use_pmdown
From: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
Now we can replace Codec to Component. Let's do it.
Signed-off-by: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
---
sound/soc/amd/acp-da7219-max98357a.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
From: Kuninori Morimoto
Now we can replace Codec to Component. Let's do it.
Signed-off-by: Kuninori Morimoto
---
sound/soc/amd/acp-da7219-max98357a.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/soc/amd/acp-da7219-max98357a.c
b/sound/soc/amd/acp-da7219
From: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
Now we can replace Codec to Component. Let's do it.
Signed-off-by: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
---
sound/soc/intel/boards/kbl_da7219_max98357a.c | 12 ++--
1 file changed, 6 insertions(+),
From: Kuninori Morimoto
Now we can replace Codec to Component. Let's do it.
Signed-off-by: Kuninori Morimoto
---
sound/soc/intel/boards/kbl_da7219_max98357a.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/sound/soc/intel/boards/kbl_da7219_max98357a.c
b
Hi Mark
These are new added driver on mark/for-next branch,
but based on old "codec" framework.
We want to convert it to component. These patches are do it
Kuninori Morimoto (4):
ASoC: intel: kbl_da7219_max98357: replace codec to component
ASoC: amd: acp-da7219-max98357: rep
Hi Mark
These are new added driver on mark/for-next branch,
but based on old "codec" framework.
We want to convert it to component. These patches are do it
Kuninori Morimoto (4):
ASoC: intel: kbl_da7219_max98357: replace codec to component
ASoC: amd: acp-da7219-max98357: rep
Hi Stephen
Thank you for your report.
It seems added new drivers are not based on new framework.
I will post fixup patch for these.
> After merging the sound-asoc tree, today's linux-next build (x86_64
> allmodconfig) failed like this:
>
> sound/soc/amd/acp-da7219-max98357a.c: In function
Hi Stephen
Thank you for your report.
It seems added new drivers are not based on new framework.
I will post fixup patch for these.
> After merging the sound-asoc tree, today's linux-next build (x86_64
> allmodconfig) failed like this:
>
> sound/soc/amd/acp-da7219-max98357a.c: In function
Hi Mark
> > Thank you for reporting.
> > It is my fault, this patch might cause other issue.
> > I'm asking to Mark to remove it from his branch.
>
> Dropped.
Thank you !
Best regards
---
Kuninori Morimoto
Hi Mark
> > Thank you for reporting.
> > It is my fault, this patch might cause other issue.
> > I'm asking to Mark to remove it from his branch.
>
> Dropped.
Thank you !
Best regards
---
Kuninori Morimoto
Hi Stephen
Thank you for reporting.
It is my fault, this patch might cause other issue.
I'm asking to Mark to remove it from his branch.
> After merging the sound-asoc tree, today's linux-next build (arm
> multi_v7_defconfig) failed like this:
>
> /home/sfr/next/next/sound/soc/soc-core.c: In
Hi Stephen
Thank you for reporting.
It is my fault, this patch might cause other issue.
I'm asking to Mark to remove it from his branch.
> After merging the sound-asoc tree, today's linux-next build (arm
> multi_v7_defconfig) failed like this:
>
> /home/sfr/next/next/sound/soc/soc-core.c: In
ng things to use components rather than
> CODEC/platform/whatever driver types that was sent about the same time
> has just gone in - the driver will require the same conversions.
Thank you for pointing it.
I posted conversion patch for this driver.
Best regards
---
Kuninori Morimoto
ng things to use components rather than
> CODEC/platform/whatever driver types that was sent about the same time
> has just gone in - the driver will require the same conversions.
Thank you for pointing it.
I posted conversion patch for this driver.
Best regards
---
Kuninori Morimoto
function [-Wmaybe-uninitialized]
>reg |= WM9081_DAC_MUTE;
>
> Introduced by commit
>
> 48c338764296 ("ASoC: wm9081: replace codec to component")
Thank you for your report.
I posted fixup patch for it.
Best regards
---
Kuninori Morimoto
function [-Wmaybe-uninitialized]
>reg |= WM9081_DAC_MUTE;
>
> Introduced by commit
>
> 48c338764296 ("ASoC: wm9081: replace codec to component")
Thank you for your report.
I posted fixup patch for it.
Best regards
---
Kuninori Morimoto
Hi Laurent
> Hi Morimoto-san,
>
> Thank you for your patch.
>
> On Thursday, 1 February 2018 09:45:36 EET Kuninori Morimoto wrote:
> > From: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
> >
> > panel-lvds.c is for LVDS Panel Driver,
> > n
Hi Laurent
> Hi Morimoto-san,
>
> Thank you for your patch.
>
> On Thursday, 1 February 2018 09:45:36 EET Kuninori Morimoto wrote:
> > From: Kuninori Morimoto
> >
> > panel-lvds.c is for LVDS Panel Driver,
> > not R-Car Display Unit CRTCs
> >
&
From: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
panel-lvds.c is for LVDS Panel Driver,
not R-Car Display Unit CRTCs
Reported-by: Koji Matsuoka <koji.matsuoka...@renesas.com>
Signed-off-by: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
---
drivers/gpu/drm/
From: Kuninori Morimoto
panel-lvds.c is for LVDS Panel Driver,
not R-Car Display Unit CRTCs
Reported-by: Koji Matsuoka
Signed-off-by: Kuninori Morimoto
---
drivers/gpu/drm/panel/panel-lvds.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/panel/panel
soc_write/read on twl4030")
> Fixes: 93a00c467fe9 ("ASoC: don't use snd_soc_write/read on twl6040")
> Cc: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
> Cc: Peter Ujfalusi <peter.ujfal...@ti.com>
> Signed-off-by: Tony Lindgren <t...@atomide.com>
> So to fix the regression, let's just do a partial revert adding back
> the read and write function pointers. Note that other non-regmap
> ASoC drivers may need similar patches.
>
> Fixes: 3bb0f7c31b1a ("ASoC: don't use snd_soc_write/read on twl4030")
> Fixes: 93a00c46
we should
> be screaming loudly about the fact that the I/O we tried to do fails,
> that clearly shouldn't be being ignored.
I'm sorry that my patch breaks your drivers.
It seems removing .read/.write callback was too much aggressive.
I hope your driver will be OK by using regmap.
In worst case, we can back .read/.write, but it will be component driver side.
Best regards
---
Kuninori Morimoto
we should
> be screaming loudly about the fact that the I/O we tried to do fails,
> that clearly shouldn't be being ignored.
I'm sorry that my patch breaks your drivers.
It seems removing .read/.write callback was too much aggressive.
I hope your driver will be OK by using regmap.
In worst case, we can back .read/.write, but it will be component driver side.
Best regards
---
Kuninori Morimoto
ometimes, in some drivers, it seems can be difficult.
Sorry for my noise.
Best regards
---
Kuninori Morimoto
ometimes, in some drivers, it seems can be difficult.
Sorry for my noise.
Best regards
---
Kuninori Morimoto
201 - 300 of 1650 matches
Mail list logo
