Re: [PATCH] modified: sound/pci/hda/patch_realtek.c

[Takashi Iwai]

On Tue, 25 Mar 2025 16:05:10 +0100,
Sharwesh05 wrote:
> 

Thanks for the patch.

But, please make the patch in a proper format, namely, have the
appropriate Subject line starting with the prefix "ALSA: hda/realtek:"
and containing the concise description about the patch.  Then put more
information about the changes (why it's needed and what it fixes /
changes) in the patch description, followed by a blank line, and most
importantly, your Signed-off-by tag with your real name and address;
it's a legal requirement.

Please check Documentation/admin/submitting-patches.rst for details.

About the code change:

> ---
>  sound/pci/hda/patch_realtek.c | 21 +++++++++++++++++++++
>  1 file changed, 21 insertions(+)
> 
> diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
> index a84857a3c2bf..8c2375476952 100644
> --- a/sound/pci/hda/patch_realtek.c
> +++ b/sound/pci/hda/patch_realtek.c
> @@ -4739,6 +4739,21 @@ static void alc245_fixup_hp_mute_led_coefbit(struct hda_codec *codec,
>  		snd_hda_gen_add_mute_led_cdev(codec, coef_mute_led_set);
>  	}
>  }
> +static void alc245_fixup_hp_mute_led_v1_coefbit(struct hda_codec *codec,
> +					    const struct hda_fixup *fix,
> +						int action)

Put a blank line before the function.

Could you correct the above and resubmit?


thanks,

Takashi