Hi,
On May 3 2017 19:07, Subhransu S. Prusty wrote:
> Signed-off-by: Subhransu S. Prusty <subhransu.s.prusty@xxxxxxxxx>
> ---
> include/uapi/sound/snd_sst_tokens.h | 4 ++--
> sound/soc/intel/skylake/skl-topology.c | 2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/include/uapi/sound/snd_sst_tokens.h b/include/uapi/sound/snd_sst_tokens.h
> index 93392bedcc58..685149dda66b 100644
> --- a/include/uapi/sound/snd_sst_tokens.h
> +++ b/include/uapi/sound/snd_sst_tokens.h
> @@ -213,8 +213,8 @@ enum SKL_TKNS {
> SKL_TKN_U32_LIB_COUNT,
> SKL_TKN_STR_LIB_NAME,
> SKL_TKN_U32_PMODE,
> - SKL_TKL_U32_D0I3_CAPS,
> - SKL_TKN_MAX = SKL_TKL_U32_D0I3_CAPS,
> + SKL_TKN_U32_D0I3_CAPS,
> + SKL_TKN_MAX = SKL_TKN_U32_D0I3_CAPS,
> };
>
> #endif
> diff --git a/sound/soc/intel/skylake/skl-topology.c b/sound/soc/intel/skylake/skl-topology.c
> index 3a99712e44a8..b28199a5348c 100644
> --- a/sound/soc/intel/skylake/skl-topology.c
> +++ b/sound/soc/intel/skylake/skl-topology.c
> @@ -1995,7 +1995,7 @@ static int skl_tplg_get_token(struct device *dev,
> mconfig->converter = tkn_elem->value;
> break;
>
> - case SKL_TKL_U32_D0I3_CAPS:
> + case SKL_TKN_U32_D0I3_CAPS:
> mconfig->d0i3_caps = tkn_elem->value;
> break;
$ git annotate include/uapi/sound/snd_sst_tokens.h
8a0cb2360ddb9 (Vinod Koul 2016-11-03 17:07:18 +0530 215)
SKL_TKN_U32_PMODE,
6bd9dcf339ebb (Vinod Koul 2016-11-03 17:07:19 +0530 216)
SKL_TKL_U32_D0I3_CAPS,
6bd9dcf339ebb (Vinod Koul 2016-11-03 17:07:19 +0530 217)
SKL_TKN_MAX = SKL_TKL_U32_D0I3_CAPS,
15f0d4f531d84 (Shreyas NC 2016-08-12 12:29:50 +0530 218)};
$ git name-rev --tags 6bd9dcf339ebb
6bd9dcf339ebb tags/sound-4.10-rc1~3^2~15^2~17
The typo was already included in Linux kernel v4.10 and exposed to
userland. In general of kernel development, it's the worst idea to
change it, because it can break userland.
Regards
Takashi Sakamoto
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
