Re: (subset) [PATCH v7 00/16] Add audio support for the MediaTek Genio 350-evk board
On Thu, 12 Sep 2024, Mark Brown wrote: > On Thu, Sep 12, 2024 at 03:51:00PM +0100, Lee Jones wrote: > > On Wed, 04 Sep 2024, Mark Brown wrote: > > > > [03/16] dt-bindings: mfd: mediatek: Add codec property for MT6357 PMIC > > > commit: 761cab667898d86c04867948f1b7aec1090be796 > > > Did you mean to hoover this up? > > It seemed to go along with the series and had a DT review so it looked > like you'd just left it to the DT people to review, there wasn't any > other MFD content in the series. I applied it from this set 6 weeks ago! :) -- Lee Jones [李琼斯]
Re: (subset) [PATCH v7 00/16] Add audio support for the MediaTek Genio 350-evk board
On Thu, Sep 12, 2024 at 03:51:00PM +0100, Lee Jones wrote: > On Wed, 04 Sep 2024, Mark Brown wrote: > > [03/16] dt-bindings: mfd: mediatek: Add codec property for MT6357 PMIC > > commit: 761cab667898d86c04867948f1b7aec1090be796 > Did you mean to hoover this up? It seemed to go along with the series and had a DT review so it looked like you'd just left it to the DT people to review, there wasn't any other MFD content in the series. signature.asc Description: PGP signature
Re: (subset) [PATCH v7 00/16] Add audio support for the MediaTek Genio 350-evk board
On Wed, 04 Sep 2024, Mark Brown wrote: > On Mon, 22 Jul 2024 08:53:29 +0200, Alexandre Mergnat wrote: > > This serie aim to add the following audio support for the Genio 350-evk: > > - Playback > > - 2ch Headset Jack (Earphone) > > - 1ch Line-out Jack (Speaker) > > - 8ch HDMI Tx > > - Capture > > - 1ch DMIC (On-board Digital Microphone) > > - 1ch AMIC (On-board Analogic Microphone) > > - 1ch Headset Jack (External Analogic Microphone) > > > > [...] > > Applied to > >https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next > > Thanks! > > [01/16] ASoC: dt-bindings: mediatek,mt8365-afe: Add audio afe document > commit: ceb3ca2876243e3ea02f78b3d488b1f2d734de49 > [02/16] ASoC: dt-bindings: mediatek,mt8365-mt6357: Add audio sound card > document > commit: 76d80dcdd55f70b28930edb97b96ee375e1cce5a > [03/16] dt-bindings: mfd: mediatek: Add codec property for MT6357 PMIC > commit: 761cab667898d86c04867948f1b7aec1090be796 Did you mean to hoover this up? > [04/16] ASoC: mediatek: mt8365: Add common header > commit: 38c7c9ddc74033406461d64e541bbc8268e77f73 > [05/16] ASoC: mediatek: mt8365: Add audio clock control support > commit: ef307b40b7f0042d54f020bccb3e728ced292282 > [06/16] ASoC: mediatek: mt8365: Add I2S DAI support > commit: 402bbb13a195caa83b3279ebecdabfb11ddee084 > [07/16] ASoC: mediatek: mt8365: Add ADDA DAI support > commit: 7c58c88e524180e8439acdfc44872325e7f6d33d > [08/16] ASoC: mediatek: mt8365: Add DMIC DAI support > commit: 1c50ec75ce6c0c6b5736499393e522f73e19d0cf > [09/16] ASoC: mediatek: mt8365: Add PCM DAI support > commit: 5097c0c8634d703e3c59cfb89831b7db9dc46339 > [10/16] ASoc: mediatek: mt8365: Add a specific soundcard for EVK > commit: 1bf6dbd75f7603dd026660bebf324f812200dc1b > [11/16] ASoC: mediatek: mt8365: Add the AFE driver support > commit: e1991d102bc2abb32331c462f8f3e77059c69578 > [12/16] ASoC: codecs: add MT6357 support > (no commit info) > [13/16] ASoC: mediatek: Add MT8365 support > (no commit info) > > All being well this means that it will be integrated into the linux-next > tree (usually sometime in the next 24 hours) and sent to Linus during > the next merge window (or sooner if it is a bug fix), however if > problems are discovered then the patch may be dropped or reverted. > > You may get further e-mails resulting from automated or manual testing > and review of the tree, please engage with people reporting problems and > send followup patches addressing any issues that are reported if needed. > > If any updates are required or you are submitting further changes they > should be sent as incremental updates against current git, existing > patches will not be replaced. > > Please add any relevant lists and maintainers to the CCs when replying > to this mail. > > Thanks, > Mark > -- Lee Jones [李琼斯]
Re: (subset) [PATCH v7 00/16] Add audio support for the MediaTek Genio 350-evk board
On Fri, Sep 06, 2024 at 12:23:23PM -0700, Nathan Chancellor wrote: > On Fri, Sep 06, 2024 at 07:27:01PM +0100, Mark Brown wrote: > > Are these just warnings introduced by recent versions of the toolchains? > > These commits passed an x86 allmodconfig with GCC 12 at each step (I did > > catch one warning there with another patch in the series that didn't get > > applied) and 0day didn't say anything at any point. > Not sure, I did not look too hard. At cursory glance, I am not sure x86 > allmodconfig would catch these, as this code depends on ARCH_MEDIATEK > (not COMPILE_TEST), which only exists for arm and arm64. Ah, I hadn't seen that (the other bits were building on x86). > > > Clang 19: > > That's relatively modern, though some of the warnings don't look > > particularly new and exciting. > Fair although I still see some of them on old versions too: Yeah, like I say some of the warnings didn't look like they were new. > https://github.com/ClangBuiltLinux/continuous-integration2/actions/runs/10738441894 > > > > sound/soc/mediatek/mt8365/mt8365-dai-adda.c:93:8: error: implicit > > > conversion from 'unsigned long' to 'unsigned int' changes value from > > > 18446744073709551614 to 4294967294 [-Werror,-Wconstant-conversion] > > > 91 | regmap_update_bits(afe->regmap, > > > AFE_ADDA_UL_DL_CON0, > > > | ~~ > > > 92 |AFE_ADDA_UL_DL_ADDA_AFE_ON, > > > 93 |~AFE_ADDA_UL_DL_ADDA_AFE_ON); > > > |^~~ > > > 1 error generated. > > > > That's a bit surprising, regmap_update_bits() takes an unsigned long? I > > suspect the constants need to be defined as unsigned. > Does it? I see it taking 'unsigned int' for all of its parameters. Sorry, I misread the warning there (or was perhaps looking at another one) and for some reason though it was about dropping signs but it's rather due to BIT() being defined for longs which is a rather bad landmine given how common a pattern negation is. I do see that synclink.h has some convenient BITn macros for some reason which would do the trick but really I think it's just a case of open coding the BIT() usage, or defining a BITI() or something but that seems ugly. Probably just open code the definitions. signature.asc Description: PGP signature
Re: (subset) [PATCH v7 00/16] Add audio support for the MediaTek Genio 350-evk board
On Fri, Sep 06, 2024 at 07:27:01PM +0100, Mark Brown wrote:
> Are these just warnings introduced by recent versions of the toolchains?
> These commits passed an x86 allmodconfig with GCC 12 at each step (I did
> catch one warning there with another patch in the series that didn't get
> applied) and 0day didn't say anything at any point.
Not sure, I did not look too hard. At cursory glance, I am not sure x86
allmodconfig would catch these, as this code depends on ARCH_MEDIATEK
(not COMPILE_TEST), which only exists for arm and arm64.
> > Clang 19:
>
> That's relatively modern, though some of the warnings don't look
> particularly new and exciting.
Fair although I still see some of them on old versions too:
https://github.com/ClangBuiltLinux/continuous-integration2/actions/runs/10738441894
> > sound/soc/mediatek/mt8365/mt8365-dai-adda.c:93:8: error: implicit
> > conversion from 'unsigned long' to 'unsigned int' changes value from
> > 18446744073709551614 to 4294967294 [-Werror,-Wconstant-conversion]
> > 91 | regmap_update_bits(afe->regmap,
> > AFE_ADDA_UL_DL_CON0,
> > | ~~
> > 92 |AFE_ADDA_UL_DL_ADDA_AFE_ON,
> > 93 |~AFE_ADDA_UL_DL_ADDA_AFE_ON);
> > |^~~
> > 1 error generated.
>
> That's a bit surprising, regmap_update_bits() takes an unsigned long? I
> suspect the constants need to be defined as unsigned.
Does it? I see it taking 'unsigned int' for all of its parameters.
$ sed -n '1242,1250p' include/linux/regmap.h
int regmap_update_bits_base(struct regmap *map, unsigned int reg,
unsigned int mask, unsigned int val,
bool *change, bool async, bool force);
static inline int regmap_update_bits(struct regmap *map, unsigned int reg,
unsigned int mask, unsigned int val)
{
return regmap_update_bits_base(map, reg, mask, val, NULL, false, false);
}
Cheers,
Nathan
Re: (subset) [PATCH v7 00/16] Add audio support for the MediaTek Genio 350-evk board
On Fri, Sep 06, 2024 at 07:27:04PM +0100, Mark Brown wrote: > These commits passed an x86 allmodconfig with GCC 12 at each step (I did > catch one warning there with another patch in the series that didn't get > applied) and 0day didn't say anything at any point. Oh, actually right as I was writing that message 0day decided to report that (and some other things that look rather preexisting), though only with clang 20. signature.asc Description: PGP signature
Re: (subset) [PATCH v7 00/16] Add audio support for the MediaTek Genio 350-evk board
On Fri, Sep 06, 2024 at 11:03:48AM -0700, Nathan Chancellor wrote: > I am seeing several warnings/errors from both GCC and Clang with > ARCH=arm64 allmodconfig after this series appeared in next-20240906. > As far as I can tell, they appear to agree. I wondered how this was not > caught during the series development but perhaps it was written against > a development tree that did not have Arnd's extrawarn series from 6.10 > in it yet? I was going to work on a series but I was not sure about the > best way to address the overflow errors, hence just the report. Are these just warnings introduced by recent versions of the toolchains? These commits passed an x86 allmodconfig with GCC 12 at each step (I did catch one warning there with another patch in the series that didn't get applied) and 0day didn't say anything at any point. > Clang 19: That's relatively modern, though some of the warnings don't look particularly new and exciting. > sound/soc/mediatek/mt8365/mt8365-dai-adda.c:93:8: error: implicit > conversion from 'unsigned long' to 'unsigned int' changes value from > 18446744073709551614 to 4294967294 [-Werror,-Wconstant-conversion] > 91 | regmap_update_bits(afe->regmap, AFE_ADDA_UL_DL_CON0, > | ~~ > 92 |AFE_ADDA_UL_DL_ADDA_AFE_ON, > 93 |~AFE_ADDA_UL_DL_ADDA_AFE_ON); > |^~~ > 1 error generated. That's a bit surprising, regmap_update_bits() takes an unsigned long? I suspect the constants need to be defined as unsigned. signature.asc Description: PGP signature
Re: (subset) [PATCH v7 00/16] Add audio support for the MediaTek Genio 350-evk board
On Wed, Sep 04, 2024 at 12:16:48PM +0100, Mark Brown wrote: > [01/16] ASoC: dt-bindings: mediatek,mt8365-afe: Add audio afe document > commit: ceb3ca2876243e3ea02f78b3d488b1f2d734de49 > [02/16] ASoC: dt-bindings: mediatek,mt8365-mt6357: Add audio sound card > document > commit: 76d80dcdd55f70b28930edb97b96ee375e1cce5a > [03/16] dt-bindings: mfd: mediatek: Add codec property for MT6357 PMIC > commit: 761cab667898d86c04867948f1b7aec1090be796 > [04/16] ASoC: mediatek: mt8365: Add common header > commit: 38c7c9ddc74033406461d64e541bbc8268e77f73 > [05/16] ASoC: mediatek: mt8365: Add audio clock control support > commit: ef307b40b7f0042d54f020bccb3e728ced292282 > [06/16] ASoC: mediatek: mt8365: Add I2S DAI support > commit: 402bbb13a195caa83b3279ebecdabfb11ddee084 > [07/16] ASoC: mediatek: mt8365: Add ADDA DAI support > commit: 7c58c88e524180e8439acdfc44872325e7f6d33d > [08/16] ASoC: mediatek: mt8365: Add DMIC DAI support > commit: 1c50ec75ce6c0c6b5736499393e522f73e19d0cf > [09/16] ASoC: mediatek: mt8365: Add PCM DAI support > commit: 5097c0c8634d703e3c59cfb89831b7db9dc46339 > [10/16] ASoc: mediatek: mt8365: Add a specific soundcard for EVK > commit: 1bf6dbd75f7603dd026660bebf324f812200dc1b > [11/16] ASoC: mediatek: mt8365: Add the AFE driver support > commit: e1991d102bc2abb32331c462f8f3e77059c69578 I am seeing several warnings/errors from both GCC and Clang with ARCH=arm64 allmodconfig after this series appeared in next-20240906. As far as I can tell, they appear to agree. I wondered how this was not caught during the series development but perhaps it was written against a development tree that did not have Arnd's extrawarn series from 6.10 in it yet? I was going to work on a series but I was not sure about the best way to address the overflow errors, hence just the report. Clang 19: sound/soc/mediatek/mt8365/mt8365-afe-clk.c:298:5: error: no previous prototype for function 'mt8365_afe_hd_engen_enable' [-Werror,-Wmissing-prototypes] 298 | int mt8365_afe_hd_engen_enable(struct mtk_base_afe *afe, bool apll1) | ^ sound/soc/mediatek/mt8365/mt8365-afe-clk.c:298:1: note: declare 'static' if the function is not intended to be used outside of this translation unit 298 | int mt8365_afe_hd_engen_enable(struct mtk_base_afe *afe, bool apll1) | ^ | static sound/soc/mediatek/mt8365/mt8365-afe-clk.c:310:5: error: no previous prototype for function 'mt8365_afe_hd_engen_disable' [-Werror,-Wmissing-prototypes] 310 | int mt8365_afe_hd_engen_disable(struct mtk_base_afe *afe, bool apll1) | ^ sound/soc/mediatek/mt8365/mt8365-afe-clk.c:310:1: note: declare 'static' if the function is not intended to be used outside of this translation unit 310 | int mt8365_afe_hd_engen_disable(struct mtk_base_afe *afe, bool apll1) | ^ | static sound/soc/mediatek/mt8365/mt8365-afe-clk.c:314:24: error: implicit conversion from 'unsigned long' to 'unsigned int' changes value from 18446744073709551614 to 4294967294 [-Werror,-Wconstant-conversion] 313 | regmap_update_bits(afe->regmap, AFE_HD_ENGEN_ENABLE, | ~~ 314 |AFE_22M_PLL_EN, ~AFE_22M_PLL_EN); |^~~ sound/soc/mediatek/mt8365/mt8365-afe-clk.c:317:24: error: implicit conversion from 'unsigned long' to 'unsigned int' changes value from 18446744073709551613 to 4294967293 [-Werror,-Wconstant-conversion] 316 | regmap_update_bits(afe->regmap, AFE_HD_ENGEN_ENABLE, | ~~ 317 |AFE_24M_PLL_EN, ~AFE_24M_PLL_EN); |^~~ 4 errors generated. sound/soc/mediatek/mt8365/mt8365-dai-adda.c:93:8: error: implicit conversion from 'unsigned long' to 'unsigned int' changes value from 18446744073709551614 to 4294967294 [-Werror,-Wconstant-conversion] 91 | regmap_update_bits(afe->regmap, AFE_ADDA_UL_DL_CON0, | ~~ 92 |AFE_ADDA_UL_DL_ADDA_AFE_ON, 93 |~AFE_ADDA_UL_DL_ADDA_AFE_ON); |^~~ 1 error generated. sound/soc/mediatek/mt8365/mt8365-mt6357.c:293:22: error: unused variable 'platform_node' [-Werror,-Wunused-variable] 293 | struct device_node *platform_node; | ^ sound/soc/mediatek/mt8365/mt8365-mt6357.c:295:6: error: unused variable 'i' [-Werror,-Wunused-variable] 295 | int i, ret; | ^ 2 errors generated. sound/soc/mediatek/mt8365/mt8365-dai-dmic.c:64:7: error: implicit conversion from 'unsi
Re: (subset) [PATCH v7 00/16] Add audio support for the MediaTek Genio 350-evk board
On Mon, 22 Jul 2024 08:53:29 +0200, Alexandre Mergnat wrote: > This serie aim to add the following audio support for the Genio 350-evk: > - Playback > - 2ch Headset Jack (Earphone) > - 1ch Line-out Jack (Speaker) > - 8ch HDMI Tx > - Capture > - 1ch DMIC (On-board Digital Microphone) > - 1ch AMIC (On-board Analogic Microphone) > - 1ch Headset Jack (External Analogic Microphone) > > [...] Applied to https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next Thanks! [01/16] ASoC: dt-bindings: mediatek,mt8365-afe: Add audio afe document commit: ceb3ca2876243e3ea02f78b3d488b1f2d734de49 [02/16] ASoC: dt-bindings: mediatek,mt8365-mt6357: Add audio sound card document commit: 76d80dcdd55f70b28930edb97b96ee375e1cce5a [03/16] dt-bindings: mfd: mediatek: Add codec property for MT6357 PMIC commit: 761cab667898d86c04867948f1b7aec1090be796 [04/16] ASoC: mediatek: mt8365: Add common header commit: 38c7c9ddc74033406461d64e541bbc8268e77f73 [05/16] ASoC: mediatek: mt8365: Add audio clock control support commit: ef307b40b7f0042d54f020bccb3e728ced292282 [06/16] ASoC: mediatek: mt8365: Add I2S DAI support commit: 402bbb13a195caa83b3279ebecdabfb11ddee084 [07/16] ASoC: mediatek: mt8365: Add ADDA DAI support commit: 7c58c88e524180e8439acdfc44872325e7f6d33d [08/16] ASoC: mediatek: mt8365: Add DMIC DAI support commit: 1c50ec75ce6c0c6b5736499393e522f73e19d0cf [09/16] ASoC: mediatek: mt8365: Add PCM DAI support commit: 5097c0c8634d703e3c59cfb89831b7db9dc46339 [10/16] ASoc: mediatek: mt8365: Add a specific soundcard for EVK commit: 1bf6dbd75f7603dd026660bebf324f812200dc1b [11/16] ASoC: mediatek: mt8365: Add the AFE driver support commit: e1991d102bc2abb32331c462f8f3e77059c69578 [12/16] ASoC: codecs: add MT6357 support (no commit info) [13/16] ASoC: mediatek: Add MT8365 support (no commit info) All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark
