lkml.org 
[lkml]   [2018]   [Sep]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v3 12/13] ASoC: apq8096: add slim support
    Hi Srinivas,

    I love your patch! Perhaps something to improve:

    [auto build test WARNING on asoc/for-next]
    [also build test WARNING on v4.19-rc2 next-20180906]
    [if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

    url: https://github.com/0day-ci/linux/commits/Srinivas-Kandagatla/ASoC-Add-support-to-WCD9335-Audio-Codec/20180907-143558
    base: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
    config: arm-allmodconfig (attached as .config)
    compiler: arm-linux-gnueabi-gcc (Debian 7.2.0-11) 7.2.0
    reproduce:
    wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
    chmod +x ~/bin/make.cross
    # save the attached .config to linux build tree
    GCC_VERSION=7.2.0 make.cross ARCH=arm

    All warnings (new ones prefixed by >>):

    sound/soc//qcom/apq8096.c: In function 'apq8096_be_hw_params_fixup':
    >> sound/soc//qcom/apq8096.c:26:1: warning: no return statement in function returning non-void [-Wreturn-type]
    }
    ^

    vim +26 sound/soc//qcom/apq8096.c

    15
    16 static int apq8096_be_hw_params_fixup(struct snd_soc_pcm_runtime *rtd,
    17 struct snd_pcm_hw_params *params)
    18 {
    19 struct snd_interval *rate = hw_param_interval(params,
    20 SNDRV_PCM_HW_PARAM_RATE);
    21 struct snd_interval *channels = hw_param_interval(params,
    22 SNDRV_PCM_HW_PARAM_CHANNELS);
    23
    24 rate->min = rate->max = 48000;
    25 channels->min = channels->max = 2;
    > 26 }
    27

    ---
    0-DAY kernel test infrastructure Open Source Technology Center
    https://lists.01.org/pipermail/kbuild-all Intel Corporation
    [unhandled content-type:application/gzip]
    \
     
     \ /
      Last update: 2018-09-10 10:26    [W:4.649 / U:0.432 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site