diff mbox series

[-next] sound: sunxi: fix declaration compile error

Message ID 20220921033819.2188233-1-zengheng4@huawei.com (mailing list archive)
State New, archived
Headers show
Series [-next] sound: sunxi: fix declaration compile error | expand

Commit Message

Zeng Heng Sept. 21, 2022, 3:38 a.m. UTC
Just fix compile error without any logic changes.

sound/soc/sunxi/sun50i-dmic.c:62:1: error: ‘static’ is not at beginning of declaration [-Werror=old-style-declaration]
   62 | const static struct dmic_rate dmic_rate_s[] = {
      | ^~~~~

Signed-off-by: Zeng Heng <zengheng4@huawei.com>
---
 sound/soc/sunxi/sun50i-dmic.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Mark Brown Sept. 21, 2022, 9:28 a.m. UTC | #1
On Wed, Sep 21, 2022 at 11:38:19AM +0800, Zeng Heng wrote:
> Just fix compile error without any logic changes.

Please submit patches using subject lines reflecting the style for the
subsystem, this makes it easier for people to identify relevant patches.
Look at what existing commits in the area you're changing are doing and
make sure your subject lines visually resemble what they're doing.
There's no need to resubmit to fix this alone.
Mark Brown Sept. 21, 2022, 12:37 p.m. UTC | #2
On Wed, 21 Sep 2022 11:38:19 +0800, Zeng Heng wrote:
> Just fix compile error without any logic changes.
> 
> sound/soc/sunxi/sun50i-dmic.c:62:1: error: ‘static’ is not at beginning of declaration [-Werror=old-style-declaration]
>    62 | const static struct dmic_rate dmic_rate_s[] = {
>       | ^~~~~
> 
> 
> [...]

Applied to

   broonie/sound.git for-next

Thanks!

[1/1] sound: sunxi: fix declaration compile error
      commit: ee81cfb58286c1aed3263d2fc94b321e7d963f08

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
Zeng Heng Sept. 21, 2022, 1:28 p.m. UTC | #3
Thanks for your recommend.

I would pay more attention to proper subject line (like 'ASoC: sunxi: 
...') in the following patch.


在 2022/9/21 17:28, Mark Brown 写道:
> On Wed, Sep 21, 2022 at 11:38:19AM +0800, Zeng Heng wrote:
>> Just fix compile error without any logic changes.
> Please submit patches using subject lines reflecting the style for the
> subsystem, this makes it easier for people to identify relevant patches.
> Look at what existing commits in the area you're changing are doing and
> make sure your subject lines visually resemble what they're doing.
> There's no need to resubmit to fix this alone.
diff mbox series

Patch

diff --git a/sound/soc/sunxi/sun50i-dmic.c b/sound/soc/sunxi/sun50i-dmic.c
index cd3c07f2070f..86cff5a5b1bd 100644
--- a/sound/soc/sunxi/sun50i-dmic.c
+++ b/sound/soc/sunxi/sun50i-dmic.c
@@ -59,7 +59,7 @@  struct dmic_rate {
 	unsigned int rate_bit;
 };
 
-const static struct dmic_rate dmic_rate_s[] = {
+static const struct dmic_rate dmic_rate_s[] = {
 	{48000, 0x0},
 	{44100, 0x0},
 	{32000, 0x1},