All of lore.kernel.org
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
Cc: devicetree@vger.kernel.org, alsa-devel@alsa-project.org,
	peter.ujfalusi@ti.com, Jyri Sarha <jsarha@ti.com>,
	tomi.valkeinen@ti.com, arnaud.pouliquen@st.com,
	dri-devel@lists.freedesktop.org, liam.r.girdwood@linux.intel.com,
	tony@atomide.com, broonie@kernel.org, kbuild-all@01.org,
	bcousson@baylibre.com, rmk+kernel@arm.linux.org.uk,
	linux-omap@vger.kernel.org
Subject: Re: [PATCH v7 5/6] drm/i2c: tda998x: Register ASoC hdmi-codec and add audio DT binding
Date: Thu, 17 Mar 2016 18:19:52 +0800	[thread overview]
Message-ID: <201603171804.lR3q7OxC%fengguang.wu@intel.com> (raw)
In-Reply-To: <6f10d667e5e56806ec1a788b39412ab45086c38c.1458207661.git.jsarha@ti.com>

[-- Attachment #1: Type: text/plain, Size: 1798 bytes --]

Hi Jyri,

[auto build test WARNING on drm/drm-next]
[also build test WARNING on v4.5 next-20160317]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url:    https://github.com/0day-ci/linux/commits/Jyri-Sarha/Implement-generic-ASoC-HDMI-codec-and-use-it-in-tda998x/20160317-180926
base:   git://people.freedesktop.org/~airlied/linux.git drm-next
config: x86_64-randconfig-x019-201611 (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

   drivers/gpu/drm/i2c/tda998x_drv.c: In function 'tda998x_configure_audio':
>> drivers/gpu/drm/i2c/tda998x_drv.c:720:6: warning: unused variable 'ret' [-Wunused-variable]
     int ret;
         ^

vim +/ret +720 drivers/gpu/drm/i2c/tda998x_drv.c

   704		if (on) {
   705			reg_set(priv, REG_SOFTRESET, SOFTRESET_AUDIO);
   706			reg_clear(priv, REG_SOFTRESET, SOFTRESET_AUDIO);
   707			reg_set(priv, REG_AIP_CNTRL_0, AIP_CNTRL_0_RST_FIFO);
   708		} else {
   709			reg_clear(priv, REG_AIP_CNTRL_0, AIP_CNTRL_0_RST_FIFO);
   710		}
   711	}
   712	
   713	static int
   714	tda998x_configure_audio(struct tda998x_priv *priv,
   715				struct tda998x_audio_params *params,
   716				unsigned mode_clock)
   717	{
   718		u8 buf[6], clksel_aip, clksel_fs, cts_n, adiv;
   719		u32 n;
 > 720		int ret;
   721	
   722		/* Enable audio ports */
   723		reg_write(priv, REG_ENA_AP, params->config);
   724	
   725		/* Set audio input source */
   726		switch (params->format) {
   727		case AFMT_SPDIF:
   728			reg_write(priv, REG_ENA_ACLK, 0);

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 22599 bytes --]

[-- Attachment #3: Type: text/plain, Size: 160 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2016-03-17 10:19 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-17 10:03 [PATCH v7 0/6] Implement generic ASoC HDMI codec and use it in tda998x Jyri Sarha
2016-03-17 10:03 ` [PATCH v7 1/6] ALSA: pcm: add IEC958 channel status helper for hw_params Jyri Sarha
2016-03-17 10:03 ` [PATCH v7 2/6] ASoC: hdmi-codec: Add hdmi-codec for external HDMI-encoders Jyri Sarha
2016-03-17 10:03 ` [PATCH v7 3/6] ASoC: hdmi-codec: Add audio abort() callback for video side to use Jyri Sarha
2016-03-17 10:03 ` [PATCH v7 4/6] drm/i2c: tda998x: Improve tda998x_configure_audio() audio related pdata Jyri Sarha
2016-03-17 10:03 ` [PATCH v7 5/6] drm/i2c: tda998x: Register ASoC hdmi-codec and add audio DT binding Jyri Sarha
2016-03-17 10:19   ` kbuild test robot [this message]
2016-03-17 10:03 ` [PATCH v7 6/6] ARM: dts: am335x-boneblack: Add HDMI audio support Jyri Sarha

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201603171804.lR3q7OxC%fengguang.wu@intel.com \
    --to=lkp@intel.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=arnaud.pouliquen@st.com \
    --cc=bcousson@baylibre.com \
    --cc=broonie@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=jsarha@ti.com \
    --cc=kbuild-all@01.org \
    --cc=liam.r.girdwood@linux.intel.com \
    --cc=linux-omap@vger.kernel.org \
    --cc=peter.ujfalusi@ti.com \
    --cc=rmk+kernel@arm.linux.org.uk \
    --cc=tomi.valkeinen@ti.com \
    --cc=tony@atomide.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.