From: Vincent McIntyre <vincent.mcintyre@gmail.com>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: "Jasmin J." <jasmin@anw.at>,
linux-media@vger.kernel.org, d.scheller@gmx.net
Subject: Re: [PATCH] build: Added missing timer_setup_on_stack
Date: Sat, 9 Dec 2017 10:16:08 +1100 [thread overview]
Message-ID: <20171208231606.GA5540@shambles.windy> (raw)
In-Reply-To: <21ccbddb-eada-fa44-93ea-f0b80e17d409@xs4all.nl>
On Fri, Dec 08, 2017 at 10:12:05PM +0100, Hans Verkuil wrote:
...
>
> I've applied all your patches. Thank you very much for working on this.
>
> Let's see what the result of the nightly build will be.
>
> In general reverting kernel patches to make a driver compile is something of a
> last resort. It tends to be painful to maintain in the long run, at least, that's
> been my experience.
>
Hi,
thanks both for your work on this. Not quite there yet however.
$ make allyesconfig
make -C /home/me/git/clones/media_build/v4l allyesconfig
make[1]: Entering directory '/home/me/git/clones/media_build/v4l'
No version yet, using 4.4.0-103-generic
make[2]: Entering directory '/home/me/git/clones/media_build/linux'
Syncing with dir ../media/
Applying patches for kernel 4.4.0-103-generic
patch -s -f -N -p1 -i ../backports/api_version.patch
patch -s -f -N -p1 -i ../backports/pr_fmt.patch
1 out of 1 hunk FAILED
Makefile:130: recipe for target 'apply_patches' failed
make[2]: *** [apply_patches] Error 1
make[2]: Leaving directory '/home/me/git/clones/media_build/linux'
Makefile:374: recipe for target 'allyesconfig' failed
make[1]: *** [allyesconfig] Error 2
make[1]: Leaving directory '/home/me/git/clones/media_build/v4l'
Makefile:26: recipe for target 'allyesconfig' failed
make: *** [allyesconfig] Error 2
can't select all drivers at ./build line 525
+ status=29
FWIW I also get the same failure on a 4.10 machine:
+ cat /proc/version_signature
Ubuntu 4.10.0-38.42~16.04.1-generic 4.10.17
I will have a look to see if I can spot this one.
Cheers
Vince
next prev parent reply other threads:[~2017-12-08 23:16 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-08 21:00 [PATCH] build: Added missing timer_setup_on_stack Jasmin J.
2017-12-08 21:06 ` Jasmin J.
2017-12-08 21:12 ` Hans Verkuil
2017-12-08 23:16 ` Vincent McIntyre [this message]
2017-12-08 23:29 ` Jasmin J.
2017-12-09 8:00 ` Hans Verkuil
2017-12-10 6:27 ` Vincent McIntyre
2017-12-10 14:26 ` Jasmin J.
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=20171208231606.GA5540@shambles.windy \
--to=vincent.mcintyre@gmail.com \
--cc=d.scheller@gmx.net \
--cc=hverkuil@xs4all.nl \
--cc=jasmin@anw.at \
--cc=linux-media@vger.kernel.org \
/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