From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: Peter Ujfalusi <peter.ujfalusi@ti.com>,
Arnd Bergmann <arnd@arndb.de>, <linaro-kernel@lists.linaro.org>
Cc: Dave Airlie <airlied@redhat.com>,
<kernel-build-reports@lists.linaro.org>,
<dri-devel@lists.sourceforge.net>, <linux-kernel@vger.kernel.org>
Subject: Re: v4.7-rc2 build: 0 failures 5 warnings (v4.7-rc2)
Date: Mon, 6 Jun 2016 13:13:03 +0300 [thread overview]
Message-ID: <57554CAF.3080004@ti.com> (raw)
In-Reply-To: <dbafb5b6-9a2d-3299-2347-ac6c661445a4@ti.com>
[-- Attachment #1.1: Type: text/plain, Size: 1357 bytes --]
On 06/06/16 13:08, Peter Ujfalusi wrote:
> On 06/06/16 12:44, Arnd Bergmann wrote:
>> On Monday, June 6, 2016 2:09:07 AM CEST Build bot for Mark Brown wrote:
>>> arm-allmodconfig : PASS, 0 errors, 2 warnings, 0 section mismatches
>>>
>>> Warnings:
>>> ../drivers/gpu/drm/omapdrm/dss/dsi.c:1170:6: warning: unused variable 'r' [-Wunused-variable]
>>> ../drivers/gpu/drm/omapdrm/dss/hdmi5.c:123:6: warning: unused variable 'r' [-Wunused-variable]
>>
>> These got introduced by Fixes: 973999aa0140 ("drm/omap: Remove
>> regulator API abuse") in 4.7-rc2, and are trivial to fix, I can provide a
>> patch if needed.
>
> Hrm, I don't remember sending that patch, I have sent:
> https://lkml.org/lkml/2016/4/28/361
>
> The patch I have sent did not have 'r' unused.
>
> I can send follow-up patches to fix the unused variable.
Oops.
There was a partial cleanup going for 4.7 merge window, but it missed
that. But arch/arm side cleanups went in, so I was able to do proper
cleanup. But in the end I somehow managed to mess it up, and the 'int
r's were left. Sorry about that.
I have sent a fix patch for those last week.
>> 845cd86005ea [SUBMITTED 20160511] drm/omap: include linux/of.h where needed
This one shouldn't be necessary, as there's d9e32ecda41b ("drm/omap: Fix
missing includes").
Tomi
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
next prev parent reply other threads:[~2016-06-06 10:14 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <E1b9j2R-0002zq-R2@optimist>
2016-06-06 9:44 ` Arnd Bergmann
2016-06-06 10:08 ` Peter Ujfalusi
2016-06-06 10:13 ` Tomi Valkeinen [this message]
2016-06-06 10:29 ` Arnd Bergmann
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=57554CAF.3080004@ti.com \
--to=tomi.valkeinen@ti.com \
--cc=airlied@redhat.com \
--cc=arnd@arndb.de \
--cc=dri-devel@lists.sourceforge.net \
--cc=kernel-build-reports@lists.linaro.org \
--cc=linaro-kernel@lists.linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=peter.ujfalusi@ti.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®