[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <b08ed96d-853f-2f00-d4d5-5704573722ee@linux.intel.com>
Date: Mon, 27 Sep 2021 09:47:09 -0500
From: Pierre-Louis Bossart <pierre-louis.bossart@...ux.intel.com>
To: Brent Lu <brent.lu@...el.com>, alsa-devel@...a-project.org
Cc: Libin Yang <libin.yang@...el.com>,
Guennadi Liakhovetski <guennadi.liakhovetski@...ux.intel.com>,
Cezary Rojewski <cezary.rojewski@...el.com>,
linux-kernel@...r.kernel.org,
Kai Vehmanen <kai.vehmanen@...ux.intel.com>,
Peter Ujfalusi <peter.ujfalusi@...ux.intel.com>,
Takashi Iwai <tiwai@...e.com>,
Jie Yang <yang.jie@...ux.intel.com>,
Liam Girdwood <liam.r.girdwood@...ux.intel.com>,
Tzung-Bi Shih <tzungbi@...gle.com>,
Mark Brown <broonie@...nel.org>,
Christophe JAILLET <christophe.jaillet@...adoo.fr>,
Bard Liao <bard.liao@...el.com>,
Paul Olaru <paul.olaru@....nxp.com>,
Rander Wang <rander.wang@...el.com>,
Gongjun Song <gongjun.song@...el.com>,
Vamshi Krishna Gopal <vamshi.krishna.gopal@...el.com>,
Yong Zhi <yong.zhi@...el.com>
Subject: Re: [PATCH] ASoC: intel: sof_rt5682: update platform device name for
Maxim amplifier
On 9/27/21 9:32 AM, Brent Lu wrote:
> To follow 20-character length limitation of platform device name, we
> have only 7 character space for amplifier. Therefore, the last
> character of mx98357a and mx98360a is removed to save space.
>
> Signed-off-by: Brent Lu <brent.lu@...el.com>
This patch fixes an issue reported in our Sparse checks:
sound/soc/intel/boards/sof_rt5682.c:1070:25: error: too long
initializer-string for array of char(no space for nul char)
Fixes: e224ef76fa8a ('ASoC: intel: sof_rt5682: support
jsl_rt5682s_mx98360a board')
Acked-by: Pierre-Louis Bossart <pierre-louis.bossart@...ux.intel.com>
FWIW here's what we use in our checks:
export ARCH=x86_64 CF="-Wsparse-error -Wsparse-all -Wno-bitwise-pointer
-Wno-pointer-arith -Wno-typesign -Wnoshadow"
make allmodconfig
make -k sound/soc/sof/ C=2
make -k sound/soc/intel/common/ C=2
make -k sound/soc/intel/boards/ C=2
make -k sound/soc/amd/ C=2
make -k drivers/soundwire/ C=2
Powered by blists - more mailing lists