Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sound doesn't work #4

Open
denysvitali opened this issue Aug 9, 2017 · 4 comments
Open

Sound doesn't work #4

denysvitali opened this issue Aug 9, 2017 · 4 comments
Labels
bug help wanted kind: kernel Kernel Related Issues priority: medium Medium priority issues

Comments

@denysvitali
Copy link
Owner

Sound deosn't work: I'm pretty sure some files in the kernel source are missing. We probably need to copy them over from linux-tegra and enable the related options

@denysvitali
Copy link
Owner Author

Seems like the tegra-alt modules aren't ready for 4.x yet

sound/soc/tegra-alt/tegra_asoc_hwdep_alt.c: In function 'tegra_asoc_hwdep_update_mapping_table':
sound/soc/tegra-alt/tegra_asoc_hwdep_alt.c:77:21: error: 'struct snd_soc_card' has no member named 'rtd'
   amx_adx_dai = card->rtd[AMX_ADX_LINK_IDX(amx_out_dai_link,
                     ^
sound/soc/tegra-alt/tegra_asoc_hwdep_alt.c:90:21: error: 'struct snd_soc_card' has no member named 'rtd'
   amx_adx_dai = card->rtd[AMX_ADX_LINK_IDX(adx_in_dai_link,
                     ^
sound/soc/tegra-alt/tegra_asoc_hwdep_alt.c:111:9: error: 'struct snd_soc_card' has no member named 'rtd'
     card->rtd[dai_idx + i].dai_link->params;
         ^
scripts/Makefile.build:313: recipe for target 'sound/soc/tegra-alt/tegra_asoc_hwdep_alt.o' failed
make[3]: *** [sound/soc/tegra-alt/tegra_asoc_hwdep_alt.o] Error 1
make[3]: *** Waiting for unfinished jobs....
sound/soc/tegra-alt/tegra210_i2s_alt.c:675:2: error: unknown field 'dapm_widgets' specified in initializer
  .dapm_widgets = tegra210_i2s_widgets,
  ^
sound/soc/tegra-alt/tegra210_i2s_alt.c:675:2: error: initialization from incompatible pointer type [-Werror]
sound/soc/tegra-alt/tegra210_i2s_alt.c:675:2: error: (near initialization for 'tegra210_i2s_codec.remove') [-Werror]
sound/soc/tegra-alt/tegra210_i2s_alt.c:676:2: error: unknown field 'num_dapm_widgets' specified in initializer
  .num_dapm_widgets = ARRAY_SIZE(tegra210_i2s_widgets),
  ^
In file included from ./include/linux/kernel.h:14:0,
                 from ./include/linux/delay.h:21,
                 from sound/soc/tegra-alt/tegra210_i2s_alt.c:19:
./include/linux/build_bug.h:29:45: error: initialization makes pointer from integer without a cast [-Werror]
 #define BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:(-!!(e)); }))
                                             ^
./include/linux/compiler-gcc.h:64:28: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
 #define __must_be_array(a) BUILD_BUG_ON_ZERO(__same_type((a), &(a)[0]))
                            ^
./include/linux/kernel.h:70:59: note: in expansion of macro '__must_be_array'
 #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
                                                           ^
sound/soc/tegra-alt/tegra210_i2s_alt.c:676:22: note: in expansion of macro 'ARRAY_SIZE'
  .num_dapm_widgets = ARRAY_SIZE(tegra210_i2s_widgets),
                      ^
./include/linux/build_bug.h:29:45: error: (near initialization for 'tegra210_i2s_codec.suspend') [-Werror]
 #define BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:(-!!(e)); }))
                                             ^
./include/linux/compiler-gcc.h:64:28: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
 #define __must_be_array(a) BUILD_BUG_ON_ZERO(__same_type((a), &(a)[0]))
                            ^
./include/linux/kernel.h:70:59: note: in expansion of macro '__must_be_array'
 #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
                                                           ^
sound/soc/tegra-alt/tegra210_i2s_alt.c:676:22: note: in expansion of macro 'ARRAY_SIZE'
  .num_dapm_widgets = ARRAY_SIZE(tegra210_i2s_widgets),
                      ^
sound/soc/tegra-alt/tegra210_i2s_alt.c:677:2: error: unknown field 'dapm_routes' specified in initializer
  .dapm_routes = tegra210_i2s_routes,
  ^
sound/soc/tegra-alt/tegra210_i2s_alt.c:677:2: error: initialization from incompatible pointer type [-Werror]
sound/soc/tegra-alt/tegra210_i2s_alt.c:677:2: error: (near initialization for 'tegra210_i2s_codec.resume') [-Werror]
sound/soc/tegra-alt/tegra210_i2s_alt.c:678:2: error: unknown field 'num_dapm_routes' specified in initializer
  .num_dapm_routes = ARRAY_SIZE(tegra210_i2s_routes),
  ^
In file included from ./include/linux/kernel.h:14:0,
                 from ./include/linux/delay.h:21,
                 from sound/soc/tegra-alt/tegra210_i2s_alt.c:19:
./include/linux/build_bug.h:29:45: error: missing braces around initializer [-Werror=missing-braces]
 #define BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:(-!!(e)); }))
                                             ^
./include/linux/compiler-gcc.h:64:28: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
 #define __must_be_array(a) BUILD_BUG_ON_ZERO(__same_type((a), &(a)[0]))
                            ^
./include/linux/kernel.h:70:59: note: in expansion of macro '__must_be_array'
 #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
                                                           ^
sound/soc/tegra-alt/tegra210_i2s_alt.c:678:21: note: in expansion of macro 'ARRAY_SIZE'
  .num_dapm_routes = ARRAY_SIZE(tegra210_i2s_routes),
                     ^
./include/linux/build_bug.h:29:45: error: (near initialization for 'tegra210_i2s_codec.component_driver') [-Werror=missing-braces]
 #define BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:(-!!(e)); }))
                                             ^
./include/linux/compiler-gcc.h:64:28: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
 #define __must_be_array(a) BUILD_BUG_ON_ZERO(__same_type((a), &(a)[0]))
                            ^
./include/linux/kernel.h:70:59: note: in expansion of macro '__must_be_array'
 #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
                                                           ^
sound/soc/tegra-alt/tegra210_i2s_alt.c:678:21: note: in expansion of macro 'ARRAY_SIZE'
  .num_dapm_routes = ARRAY_SIZE(tegra210_i2s_routes),
                     ^
./include/linux/build_bug.h:29:45: error: initialization makes pointer from integer without a cast [-Werror]
 #define BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:(-!!(e)); }))
                                             ^
./include/linux/compiler-gcc.h:64:28: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
 #define __must_be_array(a) BUILD_BUG_ON_ZERO(__same_type((a), &(a)[0]))
                            ^
./include/linux/kernel.h:70:59: note: in expansion of macro '__must_be_array'
 #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
                                                           ^
sound/soc/tegra-alt/tegra210_i2s_alt.c:678:21: note: in expansion of macro 'ARRAY_SIZE'
  .num_dapm_routes = ARRAY_SIZE(tegra210_i2s_routes),
                     ^
./include/linux/build_bug.h:29:45: error: (near initialization for 'tegra210_i2s_codec.component_driver.name') [-Werror]
 #define BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:(-!!(e)); }))
                                             ^
./include/linux/compiler-gcc.h:64:28: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
 #define __must_be_array(a) BUILD_BUG_ON_ZERO(__same_type((a), &(a)[0]))
                            ^
./include/linux/kernel.h:70:59: note: in expansion of macro '__must_be_array'
 #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
                                                           ^
sound/soc/tegra-alt/tegra210_i2s_alt.c:678:21: note: in expansion of macro 'ARRAY_SIZE'
  .num_dapm_routes = ARRAY_SIZE(tegra210_i2s_routes),
                     ^
sound/soc/tegra-alt/tegra210_i2s_alt.c:679:2: error: unknown field 'controls' specified in initializer
  .controls = tegra210_i2s_controls,
  ^
sound/soc/tegra-alt/tegra210_i2s_alt.c:679:2: error: initialization from incompatible pointer type [-Werror]
sound/soc/tegra-alt/tegra210_i2s_alt.c:679:2: error: (near initialization for 'tegra210_i2s_codec.set_sysclk') [-Werror]
sound/soc/tegra-alt/tegra210_i2s_alt.c:680:2: error: unknown field 'num_controls' specified in initializer
  .num_controls = ARRAY_SIZE(tegra210_i2s_controls),
  ^
In file included from ./include/linux/kernel.h:14:0,
                 from ./include/linux/delay.h:21,
                 from sound/soc/tegra-alt/tegra210_i2s_alt.c:19:
./include/linux/build_bug.h:29:45: error: initialization makes pointer from integer without a cast [-Werror]
 #define BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:(-!!(e)); }))
                                             ^
./include/linux/compiler-gcc.h:64:28: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
 #define __must_be_array(a) BUILD_BUG_ON_ZERO(__same_type((a), &(a)[0]))
                            ^
./include/linux/kernel.h:70:59: note: in expansion of macro '__must_be_array'
 #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
                                                           ^
sound/soc/tegra-alt/tegra210_i2s_alt.c:680:18: note: in expansion of macro 'ARRAY_SIZE'
  .num_controls = ARRAY_SIZE(tegra210_i2s_controls),
                  ^
./include/linux/build_bug.h:29:45: error: (near initialization for 'tegra210_i2s_codec.set_pll') [-Werror]
 #define BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:(-!!(e)); }))
                                             ^
./include/linux/compiler-gcc.h:64:28: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
 #define __must_be_array(a) BUILD_BUG_ON_ZERO(__same_type((a), &(a)[0]))
                            ^
./include/linux/kernel.h:70:59: note: in expansion of macro '__must_be_array'
 #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
                                                           ^
sound/soc/tegra-alt/tegra210_i2s_alt.c:680:18: note: in expansion of macro 'ARRAY_SIZE'
  .num_controls = ARRAY_SIZE(tegra210_i2s_controls),
                  ^
sound/soc/tegra-alt/tegra210_i2s_alt.c: In function 'tegra210_i2s_platform_probe':
sound/soc/tegra-alt/tegra210_i2s_alt.c:890:2: error: implicit declaration of function 'tegra_slcg_register_notifier' [-Werror=implicit-function-declaration]
  tegra_slcg_register_notifier(TEGRA_POWERGATE_AUD,
  ^
sound/soc/tegra-alt/tegra210_i2s_alt.c: In function 'tegra210_i2s_platform_remove':
sound/soc/tegra-alt/tegra210_i2s_alt.c:1013:2: error: implicit declaration of function 'tegra_slcg_unregister_notifier' [-Werror=implicit-function-declaration]
  tegra_slcg_unregister_notifier(TEGRA_POWERGATE_AUD,
  ^
sound/soc/tegra-alt/tegra210_xbar_alt.c:30:38: fatal error: linux/irqchip/tegra-agic.h: No such file or directory
 #include <linux/irqchip/tegra-agic.h>
                                      ^
compilation terminated.
scripts/Makefile.build:313: recipe for target 'sound/soc/tegra-alt/tegra210_xbar_alt.o' failed
make[3]: *** [sound/soc/tegra-alt/tegra210_xbar_alt.o] Error 1
cc1: all warnings being treated as errors
scripts/Makefile.build:313: recipe for target 'sound/soc/tegra-alt/tegra210_i2s_alt.o' failed
make[3]: *** [sound/soc/tegra-alt/tegra210_i2s_alt.o] Error 1
sound/soc/tegra-alt/tegra210_admaif_alt.c:606:2: error: unknown field 'dapm_widgets' specified in initializer
  .dapm_widgets = tegra210_admaif_widgets,
  ^
sound/soc/tegra-alt/tegra210_admaif_alt.c:606:2: error: initialization from incompatible pointer type [-Werror]
sound/soc/tegra-alt/tegra210_admaif_alt.c:606:2: error: (near initialization for 'tegra210_admaif_codec.remove') [-Werror]
sound/soc/tegra-alt/tegra210_admaif_alt.c:607:2: error: unknown field 'num_dapm_widgets' specified in initializer
  .num_dapm_widgets = ARRAY_SIZE(tegra210_admaif_widgets),
  ^
In file included from ./include/linux/kernel.h:14:0,
                 from ./include/linux/delay.h:21,
                 from sound/soc/tegra-alt/tegra210_admaif_alt.c:19:
./include/linux/build_bug.h:29:45: error: initialization makes pointer from integer without a cast [-Werror]
 #define BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:(-!!(e)); }))
                                             ^
./include/linux/compiler-gcc.h:64:28: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
 #define __must_be_array(a) BUILD_BUG_ON_ZERO(__same_type((a), &(a)[0]))
                            ^
./include/linux/kernel.h:70:59: note: in expansion of macro '__must_be_array'
 #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
                                                           ^
sound/soc/tegra-alt/tegra210_admaif_alt.c:607:22: note: in expansion of macro 'ARRAY_SIZE'
  .num_dapm_widgets = ARRAY_SIZE(tegra210_admaif_widgets),
                      ^
./include/linux/build_bug.h:29:45: error: (near initialization for 'tegra210_admaif_codec.suspend') [-Werror]
 #define BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:(-!!(e)); }))
                                             ^
./include/linux/compiler-gcc.h:64:28: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
 #define __must_be_array(a) BUILD_BUG_ON_ZERO(__same_type((a), &(a)[0]))
                            ^
./include/linux/kernel.h:70:59: note: in expansion of macro '__must_be_array'
 #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
                                                           ^
sound/soc/tegra-alt/tegra210_admaif_alt.c:607:22: note: in expansion of macro 'ARRAY_SIZE'
  .num_dapm_widgets = ARRAY_SIZE(tegra210_admaif_widgets),
                      ^
sound/soc/tegra-alt/tegra210_admaif_alt.c:608:2: error: unknown field 'dapm_routes' specified in initializer
  .dapm_routes = tegra210_admaif_routes,
  ^
sound/soc/tegra-alt/tegra210_admaif_alt.c:608:2: error: initialization from incompatible pointer type [-Werror]
sound/soc/tegra-alt/tegra210_admaif_alt.c:608:2: error: (near initialization for 'tegra210_admaif_codec.resume') [-Werror]
sound/soc/tegra-alt/tegra210_admaif_alt.c:609:2: error: unknown field 'num_dapm_routes' specified in initializer
  .num_dapm_routes = ARRAY_SIZE(tegra210_admaif_routes),
  ^
In file included from ./include/linux/kernel.h:14:0,
                 from ./include/linux/delay.h:21,
                 from sound/soc/tegra-alt/tegra210_admaif_alt.c:19:
./include/linux/build_bug.h:29:45: error: missing braces around initializer [-Werror=missing-braces]
 #define BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:(-!!(e)); }))
                                             ^
./include/linux/compiler-gcc.h:64:28: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
 #define __must_be_array(a) BUILD_BUG_ON_ZERO(__same_type((a), &(a)[0]))
                            ^
./include/linux/kernel.h:70:59: note: in expansion of macro '__must_be_array'
 #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
                                                           ^
sound/soc/tegra-alt/tegra210_admaif_alt.c:609:21: note: in expansion of macro 'ARRAY_SIZE'
  .num_dapm_routes = ARRAY_SIZE(tegra210_admaif_routes),
                     ^
./include/linux/build_bug.h:29:45: error: (near initialization for 'tegra210_admaif_codec.component_driver') [-Werror=missing-braces]
 #define BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:(-!!(e)); }))
                                             ^
./include/linux/compiler-gcc.h:64:28: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
 #define __must_be_array(a) BUILD_BUG_ON_ZERO(__same_type((a), &(a)[0]))
                            ^
./include/linux/kernel.h:70:59: note: in expansion of macro '__must_be_array'
 #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
                                                           ^
sound/soc/tegra-alt/tegra210_admaif_alt.c:609:21: note: in expansion of macro 'ARRAY_SIZE'
  .num_dapm_routes = ARRAY_SIZE(tegra210_admaif_routes),
                     ^
./include/linux/build_bug.h:29:45: error: initialization makes pointer from integer without a cast [-Werror]
 #define BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:(-!!(e)); }))
                                             ^
./include/linux/compiler-gcc.h:64:28: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
 #define __must_be_array(a) BUILD_BUG_ON_ZERO(__same_type((a), &(a)[0]))
                            ^
./include/linux/kernel.h:70:59: note: in expansion of macro '__must_be_array'
 #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
                                                           ^
sound/soc/tegra-alt/tegra210_admaif_alt.c:609:21: note: in expansion of macro 'ARRAY_SIZE'
  .num_dapm_routes = ARRAY_SIZE(tegra210_admaif_routes),
                     ^
./include/linux/build_bug.h:29:45: error: (near initialization for 'tegra210_admaif_codec.component_driver.name') [-Werror]
 #define BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:(-!!(e)); }))
                                             ^
./include/linux/compiler-gcc.h:64:28: note: in expansion of macro 'BUILD_BUG_ON_ZERO'
 #define __must_be_array(a) BUILD_BUG_ON_ZERO(__same_type((a), &(a)[0]))
                            ^
./include/linux/kernel.h:70:59: note: in expansion of macro '__must_be_array'
 #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))
                                                           ^
sound/soc/tegra-alt/tegra210_admaif_alt.c:609:21: note: in expansion of macro 'ARRAY_SIZE'
  .num_dapm_routes = ARRAY_SIZE(tegra210_admaif_routes),
                     ^
cc1: all warnings being treated as errors
scripts/Makefile.build:313: recipe for target 'sound/soc/tegra-alt/tegra210_admaif_alt.o' failed
make[3]: *** [sound/soc/tegra-alt/tegra210_admaif_alt.o] Error 1
scripts/Makefile.build:572: recipe for target 'sound/soc/tegra-alt' failed
make[2]: *** [sound/soc/tegra-alt] Error 2
scripts/Makefile.build:572: recipe for target 'sound/soc' failed
make[1]: *** [sound/soc] Error 2
Makefile:1023: recipe for target 'sound' failed
make: *** [sound] Error 2

@vartom
Copy link

vartom commented Feb 17, 2018

There is no sound support for the 210 chip in the core at all. Too much patch.

@denysvitali
Copy link
Owner Author

Maybe related: raphael/linux-samus#145

@vartom
Copy link

vartom commented Feb 26, 2018

No, it all looks like that you need to port this piece of code to support the sound and add more

@denysvitali denysvitali added kind: kernel Kernel Related Issues priority: medium Medium priority issues labels Mar 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug help wanted kind: kernel Kernel Related Issues priority: medium Medium priority issues
Projects
None yet
Development

No branches or pull requests

2 participants