Compilation error drivers/amlogic/amports/amvideocap.c after commit 8e799d8976d036aa1f90dcb035f952e9dfce72a7

Fl0-0 commented 7 years ago

Compilation error after commit 8e799d8976d036aa1f90dcb035f952e9dfce72a7 amvideocap.c

GCC v 6.3.0

CC      drivers/amlogic/amports/amvideocap.o
In file included from drivers/amlogic/amports/amports_priv.h:25:0,
                 from drivers/amlogic/amports/amvideocap.c:49:
drivers/amlogic/amports/amvideocap.c: In function ‘amvideocap_YUV_to_RGB’:
drivers/amlogic/amports/amvideocap.c:349:7: error: ‘struct vframe_s’ has no member named ‘prog_proc_config’
     vf->prog_proc_config);
       ^
drivers/amlogic/amports/arch/log.h:11:37: note: in definition of macro ‘codec_print’
   printk(KERN_LEVEL INFO_PREFIX ":" args)
                                     ^~~~
drivers/amlogic/amports/arch/log.h:25:26: note: in expansion of macro ‘codec_info’
 #define pr_info(args...) codec_info(args)
                          ^~~~~~~~~~
drivers/amlogic/amports/amvideocap.c:348:4: note: in expansion of macro ‘pr_info’
    pr_info("vf->prog_proc_config = %d",
    ^~~~~~~
drivers/amlogic/amports/amvideocap.c:350:12: error: ‘struct vframe_s’ has no member named ‘prog_proc_config’
    if ((!vf->prog_proc_config) &&
            ^~
drivers/amlogic/amports/amvideocap.c:358:36: error: implicit declaration of function ‘is_meson_gxtvbb_cpu’ [-Werror=implicit-function-declaration]
  } else if (is_meson_g9tv_cpu() || is_meson_gxtvbb_cpu()) {
                                    ^~~~~~~~~~~~~~~~~~~
drivers/amlogic/amports/amvideocap.c:386:21: error: implicit declaration of function ‘codec_mm_alloc_for_dma’ [-Werror=implicit-function-declaration]
   phybufaddr_8bit = codec_mm_alloc_for_dma(CMA_NAME,
                     ^~~~~~~~~~~~~~~~~~~~~~
drivers/amlogic/amports/amvideocap.c:386:44: error: ‘CMA_NAME’ undeclared (first use in this function)
   phybufaddr_8bit = codec_mm_alloc_for_dma(CMA_NAME,
                                            ^~~~~~~~
drivers/amlogic/amports/amvideocap.c:386:44: note: each undeclared identifier is reported only once for each function it appears in
drivers/amlogic/amports/amvideocap.c:388:20: error: ‘CODEC_MM_FLAGS_CPU’ undeclared (first use in this function)
    4 + PAGE_SHIFT, CODEC_MM_FLAGS_CPU);
                    ^~~~~~~~~~~~~~~~~~
drivers/amlogic/amports/amvideocap.c:469:3: error: implicit declaration of function ‘codec_mm_dma_flush’ [-Werror=implicit-function-declaration]
   codec_mm_dma_flush(pdst,
   ^~~~~~~~~~~~~~~~~~
drivers/amlogic/amports/amvideocap.c:572:10: error: implicit declaration of function ‘codec_mm_free_for_dma’ [-Werror=implicit-function-declaration]
    ret = codec_mm_free_for_dma(CMA_NAME, phybufaddr_8bit);
          ^~~~~~~~~~~~~~~~~~~~~
  CC      fs/nfs/nfs4xdr.o
cc1: some warnings being treated as errors
scripts/Makefile.build:308: recipe for target 'drivers/amlogic/amports/amvideocap.o' failed
make[3]: *** [drivers/amlogic/amports/amvideocap.o] Error 1
scripts/Makefile.build:455: recipe for target 'drivers/amlogic/amports' failed
make[2]: *** [drivers/amlogic/amports] Error 2
scripts/Makefile.build:455: recipe for target 'drivers/amlogic' failed
make[1]: *** [drivers/amlogic] Error 2
Makefile:845: recipe for target 'drivers' failed
make: *** [drivers] Error 2

Fl0-0 commented 7 years ago

The commit does not resolve the issue, i have the same error at compilation.

Fl0-0 commented 7 years ago

The revert resolves amvideocap.c compilation errors but some errors appears in aml_dvb.c )-:

In file included from drivers/amlogic/dvb_tv/aml_dvb.c:43:0:
drivers/amlogic/dvb_tv/aml_dvb.h:261:11: warning: ‘enum dmx_source_t’ declared inside parameter list will not be visible outside of this definition or declaration
      enum dmx_source_t src);
           ^~~~~~~~~~~~
drivers/amlogic/dvb_tv/aml_dvb.h:262:60: warning: ‘enum dmx_source_t’ declared inside parameter list will not be visible outside of this definition or declaration
 extern int aml_stb_hw_set_source(struct aml_dvb *dvb, enum dmx_source_t src);
                                                            ^~~~~~~~~~~~
drivers/amlogic/dvb_tv/aml_dvb.h:264:10: warning: ‘enum dmx_source_t’ declared inside parameter list will not be visible outside of this definition or declaration
     enum dmx_source_t src, enum dmx_source_t dst);
          ^~~~~~~~~~~~
drivers/amlogic/dvb_tv/aml_dvb.h:265:60: warning: ‘enum dmx_source_t’ declared inside parameter list will not be visible outside of this definition or declaration
 extern int aml_tso_hw_set_source(struct aml_dvb *dvb, enum dmx_source_t src);
                                                            ^~~~~~~~~~~~
drivers/amlogic/dvb_tv/aml_dvb.c: In function ‘aml_dvb_dsc_init’:
drivers/amlogic/dvb_tv/aml_dvb.c:451:25: error: ‘DVB_DEVICE_DSC’ undeclared (first use in this function)
       &dvbdev_dsc, dsc, DVB_DEVICE_DSC);
                         ^~~~~~~~~~~~~~
drivers/amlogic/dvb_tv/aml_dvb.c:451:25: note: each undeclared identifier is reported only once for each function it appears in
drivers/amlogic/dvb_tv/aml_dvb.c: In function ‘stb_store_source’:
drivers/amlogic/dvb_tv/aml_dvb.c:509:7: error: variable ‘src’ has initializer but incomplete type
  enum dmx_source_t src = -1;  
       ^~~~~~~~~~~~
drivers/amlogic/dvb_tv/aml_dvb.c:509:20: error: storage size of ‘src’ isn’t known
  enum dmx_source_t src = -1;
                    ^~~
drivers/amlogic/dvb_tv/aml_dvb.c:526:42: error: type of formal parameter 2 is incomplete
   aml_stb_hw_set_source(&aml_dvb_device, src);
                                          ^~~
drivers/amlogic/dvb_tv/aml_dvb.c:509:20: warning: unused variable ‘src’ [-Wunused-variable]
  enum dmx_source_t src = -1;
                    ^~~
drivers/amlogic/dvb_tv/aml_dvb.c: In function ‘dsc0_store_source’:
drivers/amlogic/dvb_tv/aml_dvb.c:576:7: error: variable ‘src’ has initializer but incomplete type
  enum dmx_source_t src = -1, dst = -1;\
       ^
drivers/amlogic/dvb_tv/aml_dvb.c:623:2: note: in expansion of macro ‘DSC_SOURCE_FUNC_DECL’
  DSC_SOURCE_FUNC_DECL(0)
  ^~~~~~~~~~~~~~~~~~~~
drivers/amlogic/dvb_tv/aml_dvb.c:576:20: error: storage size of ‘src’ isn’t known
  enum dmx_source_t src = -1, dst = -1;\
                    ^
drivers/amlogic/dvb_tv/aml_dvb.c:623:2: note: in expansion of macro ‘DSC_SOURCE_FUNC_DECL’
  DSC_SOURCE_FUNC_DECL(0)
  ^~~~~~~~~~~~~~~~~~~~
drivers/amlogic/dvb_tv/aml_dvb.c:576:7: error: variable ‘dst’ has initializer but incomplete type
  enum dmx_source_t src = -1, dst = -1;\
       ^

hardkernel / linux

Compilation error drivers/amlogic/amports/amvideocap.c after commit 8e799d8976d036aa1f90dcb035f952e9dfce72a7 #273