e some subtle C-issue that is evaded here by setting
head->next to NULL prior to copying over it?
(I know this piece of code only got copied around in this patch and this is
therefor not something that this patch actually introduced.)
> +
> + func(rq);
> + }
> +}
Thanks,
Paul Bolle
]: *** [scripts/Makefile.build:283: sound/soc/sof/intel/hda-codec.o]
Error 1
make[3]: *** [scripts/Makefile.build:500: sound/soc/sof/intel] Error 2
make[2]: *** [scripts/Makefile.build:500: sound/soc/sof] Error 2
make[1]: *** [scripts/Makefile.build:500: sound/soc] Error 2
make: *** [Makefile:1778: sound] Error 2
There's indeed no error label in v5.9.5. (There is one in v5.10-rc2, I just
checked.) Is no-one else running into this?
Thanks,
Paul Bolle
The following commit has been merged into the locking/core branch of tip:
Commit-ID: d89d5f855f84ccf3f7e648813b4bb95c780bd7cd
Gitweb:
https://git.kernel.org/tip/d89d5f855f84ccf3f7e648813b4bb95c780bd7cd
Author:Paul Bolle
AuthorDate:Thu, 01 Oct 2020 22:20:28 +02:00
n_ia32_syscall() just checking CONFIG_IA32_EMULATION
should do too.
(Way outside my limited expertise, but anyway: is does look odd to see a call
to in_ia32_syscall() in drivers/. All other calls are in arch/x86/. Isn't this
a bit too x86 specific for an arch independent driver?)
Thanks,
Paul Bolle
Andy Shevchenko schreef op wo 07-10-2020 om 14:58 [+0300]:
> Does [1] fix the issue?
>
> [1]:
> https://lore.kernel.org/linux-gpio/20201005131044.87276-1-andriy.shevche...@linux.intel.com/
Yes, it fixes the build error.
Thanks,
Paul Bolle
- guest app either from:
> 1. x86_64 - x86_64 - i386;
> 2. x86_64 - i386 - i386.
I ran into this build error too.
Perhaps the UML maintainers have an idea what to do here. The commit that
triggers this error is 5ad284ab3a01 ("gpiolib: Fix line event handling in
syscall compatible mode").
Thanks,
Paul Bolle
The sha1sum of include/linux/atomic-arch-fallback.h isn't checked by
check-atomics.sh. It's not clear why it's skipped so let's check it too.
Signed-off-by: Paul Bolle
---
It seems it never has been checked. So this does cast some doubt about
the usefulness of these tests. But I'm clueless about
162 of these warnings in about six
days, while the session before that (v5.2.y) showed none.
Just like in Norbert's case, wifi appears to work fine.
Paul Bolle
Greg KH schreef op ma 26-08-2019 om 06:34 [+0200]:
> It's on that key already, have you refreshed your version of it?
I have now. sas...@kernel.org (and another identity) is now on that key.
Thanks,
Paul Bolle
use one of gpg2's many options
to add an
aka "Sasha Levin "
line to that key. (I assume "--recv-key" then would have found your key.)
Or would that be a sin against kernel.org policy, state-of-the-art
cryptographic practices, or whatever?
Thanks,
Paul Bolle
Hi Phong,
Phong Tran schreef op za 27-07-2019 om 08:56 [+0700]:
> On 7/26/19 9:22 PM, Paul Bolle wrote:
> > Phong Tran schreef op vr 26-07-2019 om 20:35 [+0700]:
> > > diff --git a/drivers/isdn/gigaset/usb-gigaset.c
> > > b/drivers/isdn/gigaset/usb-gigaset
/gfs2_ondisk.h
-GIGASET ISDN DRIVERS
-M: Paul Bolle
-L: gigaset307x-com...@lists.sourceforge.net
-W: http://gigaset307x.sourceforge.net/
-S: Odd Fixes
-F: drivers/staging/isdn/gigaset/
-
GNSS SUBSYSTEM
M: Johan Hovold
T: git git://git.kernel.org/pub/scm/linux/kernel/git
uld check. (The subject of that thread
contains "[5.2 REGRESSION]", but I'd say "[5.3-rc1 REGRESSION]" would more
accurate, but whatever.)
Thanks,
Paul Bolle
] Stopped Journal Service.
And without systemd-journald I couldn't get userspace up and running.
A bit of tinkering showed that "vdso32=0" on the kernel command line allows me
to get a usable userspace.
Any idea where I should look next to pinpoint this?
Thanks,
Paul Bolle
+if (!endpoint) {
> + dev_err(cs->dev, "Endpoint not available\n");
> + retval = -ENODEV;
> + goto error;
> + }
>
> ucs->busy = 0;
>
Please note that I'm very close to getting cut off from the ISDN network, so
the chances of being able to testi this on a live system are getting small.
Thanks,
Paul Bolle
Hi Jose,
James Bottomley schreef op do 18-07-2019 om 06:29 [+0900]:
> On Wed, 2019-07-17 at 23:27 +0200, Paul Bolle wrote:
> > I've just reached a day of uptime with your revert. (The proper
> > uptime is just a fraction of a day, this being a laptop.) Anyhow, no
> > sc
Paul Bolle schreef op di 09-04-2019 om 21:11 [+0200]:
> Does it still apply?
It does, cleanly. Output is (for v5.1-rc4):
./usr/include/asm-generic/fcntl.h:119: leaks CONFIG_64BIT to userspace where it
is not valid
./usr/include/asm-generic/mman-common.h:22: le
al branch for over five years, but
never dared to push it upstream.
Does it still apply?
Thanks,
Paul Bolle
-- >8 ---
>From 0f73c8ee776c197e3029c4eed21af0f121a8f9d3 Mon Sep 17 00:00:00 2001
From: Paul Bolle
Date: Tue, 4 Feb 2014 22:22:48 +0100
Subject: [PATCH] headers_check: enable ch
as used: -Wimplicit-fallthrough=3
>
> Notice that, in this particular case, the code comment is modified
> in accordance with what GCC is expecting to find.
>
> This patch is part of the ongoing efforts to enable
> -Wimplicit-fallthrough.
>
> Signed-off-by: Gustavo A. R. Silva
Acked-by: Paul Bolle
Thanks,
Paul Bolle
he ISDN
subsystem in the tree after September 1, 2019, and if so, in what form. I'm in
no position to properly answer that question.
Paul Bolle
didn't touch the procinfo method in the other ISDN
drivers, as far as I can see.
(If it was intentional, gigaset_procinfo() can of course be removed.)
> - iif->ctr.proc_fops = _proc_fops;
> + iif->ctr.proc_show = gigaset_proc_show,
> INIT_LIST_HEAD(>appls);
> skb_queue_head_init(>sendqueue);
> atomic_set(>sendqlen, 0);
Thanks,
Paul Bolle
didn't touch the procinfo method in the other ISDN
drivers, as far as I can see.
(If it was intentional, gigaset_procinfo() can of course be removed.)
> - iif->ctr.proc_fops = _proc_fops;
> + iif->ctr.proc_show = gigaset_proc_show,
> INIT_LIST_HEAD(>appls);
> skb_queue_head_init(>sendqueue);
> atomic_set(>sendqlen, 0);
Thanks,
Paul Bolle
send cleanup patches for this whenever
you feel like doing so.
Thanks,
Paul Bolle
send cleanup patches for this whenever
you feel like doing so.
Thanks,
Paul Bolle
t;)] == '\0')
> + zconfprint("warning: '%s' defined with blank help text",
> +current_entry->sym->name ?: "");
> +
Does this go to stderr?
Another fix would be to ignore empty help texts and not render them at all. Is
that possible?
Thanks,
Paul Bolle
t;)] == '\0')
> + zconfprint("warning: '%s' defined with blank help text",
> +current_entry->sym->name ?: "");
> +
Does this go to stderr?
Another fix would be to ignore empty help texts and not render them at all. Is
that possible?
Thanks,
Paul Bolle
USERINCLUDE
variable, which is actually part of the LINUXINCLUDE variable, but split off
to make things confusing.)
Why do you need to include linux/kconfig.h here?
> #include
> #include
Thanks,
Paul Bolle
ch is actually part of the LINUXINCLUDE variable, but split off
to make things confusing.)
Why do you need to include linux/kconfig.h here?
> #include
> #include
Thanks,
Paul Bolle
ep on doing
that for years. But still, I'd love to hear someone say: yes, I still care
about mainline ISDN.
Does that person still exists?
Thanks,
Paul Bolle
ep on doing
that for years. But still, I'd love to hear someone say: yes, I still care
about mainline ISDN.
Does that person still exists?
Thanks,
Paul Bolle
again shows, I'd say, that the gigaset code mainly
exists to see if there's still a pulse in mainline ISDN. Is that enough to
bother? Or should mainline ISDN go the way of, say, IRDA?
But I digress. An alternative patch would be much appreciated.
Thanks,
Paul Bolle
again shows, I'd say, that the gigaset code mainly
exists to see if there's still a pulse in mainline ISDN. Is that enough to
bother? Or should mainline ISDN go the way of, say, IRDA?
But I digress. An alternative patch would be much appreciated.
Thanks,
Paul Bolle
for all three drivers (including
bas_gigaset). So driver->cs itself is invalid here.
And since the patch uses
struct cardstate *cs = urb->context;
in a few places my guess is that it's really the patch that triggers this.
Thanks,
Paul Bolle
for all three drivers (including
bas_gigaset). So driver->cs itself is invalid here.
And since the patch uses
struct cardstate *cs = urb->context;
in a few places my guess is that it's really the patch that triggers this.
Thanks,
Paul Bolle
On Thu, 2017-10-19 at 23:03 +0200, Paul Bolle wrote:
> On Mon, 2017-10-16 at 17:29 -0700, Kees Cook wrote:
> > In preparation for unconditionally passing the struct timer_list pointer to
> > all timer callbacks, switch to using the new timer_setup() and from_timer()
> > to p
On Thu, 2017-10-19 at 23:03 +0200, Paul Bolle wrote:
> On Mon, 2017-10-16 at 17:29 -0700, Kees Cook wrote:
> > In preparation for unconditionally passing the struct timer_list pointer to
> > all timer callbacks, switch to using the new timer_setup() and from_timer()
> > to p
On Mon, 2017-10-16 at 17:29 -0700, Kees Cook wrote:
> In preparation for unconditionally passing the struct timer_list pointer to
> all timer callbacks, switch to using the new timer_setup() and from_timer()
> to pass the timer pointer explicitly.
Acked-by: Paul Bolle <pebo..
On Mon, 2017-10-16 at 17:29 -0700, Kees Cook wrote:
> In preparation for unconditionally passing the struct timer_list pointer to
> all timer callbacks, switch to using the new timer_setup() and from_timer()
> to pass the timer pointer explicitly.
Acked-by: Paul Bolle
For the record: t
On Mon, 2017-10-16 at 17:29 -0700, Kees Cook wrote:
> This replaces a kmalloc followed by a bunch of per-field zeroing with a
> single kzalloc call, reducing the lines of code.
Acked-by: Paul Bolle <pebo...@tiscali.nl>
Thanks,
Paul Bolle
On Mon, 2017-10-16 at 17:29 -0700, Kees Cook wrote:
> This replaces a kmalloc followed by a bunch of per-field zeroing with a
> single kzalloc call, reducing the lines of code.
Acked-by: Paul Bolle
Thanks,
Paul Bolle
inters in these timer
callbacks is removed.)
Thanks,
Paul Bolle
inters in these timer
callbacks is removed.)
Thanks,
Paul Bolle
ook at
them properly!): I'd prefer it if that was done separately in a preceding
patch. Would that bother you?
Thanks,
Paul Bolle
ook at
them properly!): I'd prefer it if that was done separately in a preceding
patch. Would that bother you?
Thanks,
Paul Bolle
. But I just wanted to point out the lack of consistency here.
A major benefit is that
#if IS_ENABLED(CONFIG_HYPERV)
is shorter than
#if defined(CONFIG_HYPERV) || defined(CONFIG_HYPERV_MODULE)
and less prone to typos.
Paul Bolle
. But I just wanted to point out the lack of consistency here.
A major benefit is that
#if IS_ENABLED(CONFIG_HYPERV)
is shorter than
#if defined(CONFIG_HYPERV) || defined(CONFIG_HYPERV_MODULE)
and less prone to typos.
Paul Bolle
d-off-by: Joe Perches <j...@perches.com>
> ---
> drivers/isdn/gigaset/interface.c| 2 +-
Gigaset hunk looks good to me:
Acked-by: Paul Bolle <pebo...@tiscali.nl>
> drivers/isdn/hardware/mISDN/avmfritz.c | 17 +
> drivers/isdn/hardware/mISDN/hfcmulti
gned-off-by: Joe Perches
> ---
> drivers/isdn/gigaset/interface.c| 2 +-
Gigaset hunk looks good to me:
Acked-by: Paul Bolle
> drivers/isdn/hardware/mISDN/avmfritz.c | 17 +
> drivers/isdn/hardware/mISDN/hfcmulti.c | 8
> drivers/isdn/hardware/mISD
n identical patch in my ever growing stack of stuff that I should
actually submit:
Acked-by: Paul Bolle <pebo...@tiscali.nl>
Thanks,
Paul Bolle
stack of stuff that I should
actually submit:
Acked-by: Paul Bolle
Thanks,
Paul Bolle
On Tue, 2017-01-03 at 23:25 +0100, Arnd Bergmann wrote:
> As far as I'm concerned, we are totally fine as long as there exists a
> longterm supported kernel that has i4l in drivers/staging.
Or in drivers/isdn, right?
Paul Bolle
On Tue, 2017-01-03 at 23:25 +0100, Arnd Bergmann wrote:
> As far as I'm concerned, we are totally fine as long as there exists a
> longterm supported kernel that has i4l in drivers/staging.
Or in drivers/isdn, right?
Paul Bolle
nups?
How often did a bunch of drivers re-enter the tree after being sent to
staging?
Paul Bolle
nups?
How often did a bunch of drivers re-enter the tree after being sent to
staging?
Paul Bolle
On Thu, 2016-12-22 at 01:50 +0300, Ozgur Karatas wrote:
> I don't have a problem with C programming
I'm sorry, but you do need to learn C, at a basic level, first.
Paul Bolle
On Thu, 2016-12-22 at 01:50 +0300, Ozgur Karatas wrote:
> I don't have a problem with C programming
I'm sorry, but you do need to learn C, at a basic level, first.
Paul Bolle
net/wireless/reg.o' failed
make: *** [net/wireless/reg.o] Error 2
Didn't Thomas Gleixner suggest that you do a basic C course just yesterday?
Paul Bolle
net/wireless/reg.o' failed
make: *** [net/wireless/reg.o] Error 2
Didn't Thomas Gleixner suggest that you do a basic C course just yesterday?
Paul Bolle
include/asm/ got removed in v1.1.45. Two decades later it's about time
to stop worrying whether patches still touch it.
Signed-off-by: Paul Bolle <pebo...@tiscali.nl>
---
Lightly tested.
scripts/checkpatch.pl | 5 -
1 file changed, 5 deletions(-)
diff --git a/scripts/checkpatc
include/asm/ got removed in v1.1.45. Two decades later it's about time
to stop worrying whether patches still touch it.
Signed-off-by: Paul Bolle
---
Lightly tested.
scripts/checkpatch.pl | 5 -
1 file changed, 5 deletions(-)
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
]; then
echo "#define __KSYM_module_layout 1" >> "$new_ksyms_file"
fi
> I'm trying to determine the best way to fix it. Stay tuned.
Will do. I'm curious to see what a proper fix might look like.
Thanks,
Paul Bolle
]; then
echo "#define __KSYM_module_layout 1" >> "$new_ksyms_file"
fi
> I'm trying to determine the best way to fix it. Stay tuned.
Will do. I'm curious to see what a proper fix might look like.
Thanks,
Paul Bolle
On Thu, 2016-12-01 at 10:01 +0100, Paul Bolle wrote:
> Perhaps this is all documented somewhere. But even then it would be nice if
> the build would fail right at the start. Ie, the build probably should fail if
> one does "make bzImage" while having a .config with
> CONF
On Thu, 2016-12-01 at 10:01 +0100, Paul Bolle wrote:
> Perhaps this is all documented somewhere. But even then it would be nice if
> the build would fail right at the start. Ie, the build probably should fail if
> one does "make bzImage" while having a .config with
> CONF
NUSED_KSYMS=y
CONFIG_MODULES=y
# CONFIG_MODVERSIONS is not set
Because it seems in that case the subsequent "make modules" will then end in
this flood of ERRORs.
Paul Bolle
NUSED_KSYMS=y
CONFIG_MODULES=y
# CONFIG_MODVERSIONS is not set
Because it seems in that case the subsequent "make modules" will then end in
this flood of ERRORs.
Paul Bolle
On Wed, 2016-11-30 at 22:42 +0100, Paul Bolle wrote:
> My current theory is that setting MODVERSIONS, somehow, hides the ERROR spew.
> Because that could explain your bisect. Linus' commit turns of MODVERSIONS the
> hard way. And, naturally, this theory fails if your .configs
On Wed, 2016-11-30 at 22:42 +0100, Paul Bolle wrote:
> My current theory is that setting MODVERSIONS, somehow, hides the ERROR spew.
> Because that could explain your bisect. Linus' commit turns of MODVERSIONS the
> hard way. And, naturally, this theory fails if your .configs
xplain your bisect. Linus' commit turns of MODVERSIONS the
hard way. And, naturally, this theory fails if your .configs never had
MODVERSIONS set in the first place.
I'm testing all this right now, but you probably command more powerful
machines and could beat me in testing this theory.
Thanks,
Paul Bolle
xplain your bisect. Linus' commit turns of MODVERSIONS the
hard way. And, naturally, this theory fails if your .configs never had
MODVERSIONS set in the first place.
I'm testing all this right now, but you probably command more powerful
machines and could beat me in testing this theory.
Thanks,
Paul Bolle
at's actually going on
> here.
Likewise (ie, both the modules splat going away if doing a single make and
being baffled, but more that a wee bit).
Paul Bolle
at's actually going on
> here.
Likewise (ie, both the modules splat going away if doing a single make and
being baffled, but more that a wee bit).
Paul Bolle
pkg-config in scripts/kconfig/Makefile. Or in
scripts/kconfig/lxdialog/check-lxdialog.sh. Not sure, I didn't yet dive deeper
into this.
Hope this helps,
Paul Bolle
pkg-config in scripts/kconfig/Makefile. Or in
scripts/kconfig/lxdialog/check-lxdialog.sh. Not sure, I didn't yet dive deeper
into this.
Hope this helps,
Paul Bolle
Commit-ID: 9190e21780dfeff524a67c6e7b806c8a9d496086
Gitweb: http://git.kernel.org/tip/9190e21780dfeff524a67c6e7b806c8a9d496086
Author: Paul Bolle <pebo...@tiscali.nl>
AuthorDate: Fri, 25 Nov 2016 13:41:47 +0100
Committer: Ingo Molnar <mi...@kernel.org>
CommitDate: Mon, 28 No
Commit-ID: 9190e21780dfeff524a67c6e7b806c8a9d496086
Gitweb: http://git.kernel.org/tip/9190e21780dfeff524a67c6e7b806c8a9d496086
Author: Paul Bolle
AuthorDate: Fri, 25 Nov 2016 13:41:47 +0100
Committer: Ingo Molnar
CommitDate: Mon, 28 Nov 2016 07:49:17 +0100
x86/build: Remove three
Commit-ID: 06cbbac0f57d947656a12c30a0a69d4cf0ac6dea
Gitweb: http://git.kernel.org/tip/06cbbac0f57d947656a12c30a0a69d4cf0ac6dea
Author: Paul Bolle <pebo...@tiscali.nl>
AuthorDate: Fri, 25 Nov 2016 13:38:34 +0100
Committer: Ingo Molnar <mi...@kernel.org>
CommitDate: Mon, 28 No
Commit-ID: 06cbbac0f57d947656a12c30a0a69d4cf0ac6dea
Gitweb: http://git.kernel.org/tip/06cbbac0f57d947656a12c30a0a69d4cf0ac6dea
Author: Paul Bolle
AuthorDate: Fri, 25 Nov 2016 13:38:34 +0100
Committer: Ingo Molnar
CommitDate: Mon, 28 Nov 2016 07:49:17 +0100
x86/build: Don't use
Signed-off-by: Paul Bolle <pebo...@tiscali.nl>
---
arch/x86/include/asm/Kbuild | 4
1 file changed, 4 deletions(-)
diff --git a/arch/x86/include/asm/Kbuild b/arch/x86/include/asm/Kbuild
index 2cfed174e3c9..2b892e2313a9 100644
--- a/arch/x86/include/asm/Kbuild
+++ b/arch/x86/include/asm/Kbui
Signed-off-by: Paul Bolle
---
arch/x86/include/asm/Kbuild | 4
1 file changed, 4 deletions(-)
diff --git a/arch/x86/include/asm/Kbuild b/arch/x86/include/asm/Kbuild
index 2cfed174e3c9..2b892e2313a9 100644
--- a/arch/x86/include/asm/Kbuild
+++ b/arch/x86/include/asm/Kbuild
@@ -6,10 +6,6 @@ g
-by: Paul Bolle <pebo...@tiscali.nl>
---
arch/x86/realmode/rm/Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/realmode/rm/Makefile b/arch/x86/realmode/rm/Makefile
index 25012abc3409..4463fa72db94 100644
--- a/arch/x86/realmode/rm/Makefile
+++ b/arch/x86/realm
-by: Paul Bolle
---
arch/x86/realmode/rm/Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/realmode/rm/Makefile b/arch/x86/realmode/rm/Makefile
index 25012abc3409..4463fa72db94 100644
--- a/arch/x86/realmode/rm/Makefile
+++ b/arch/x86/realmode/rm/Makefile
@@ -69,7 +69,7
) twice.
Signed-off-by: Paul Bolle <pebo...@tiscali.nl>
---
Build tested only!
arch/mips/boot/compressed/Makefile | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/mips/boot/compressed/Makefile
b/arch/mips/boot/compressed/Makefile
index 011b14512c05..41564a4db6f7
) twice.
Signed-off-by: Paul Bolle
---
Build tested only!
arch/mips/boot/compressed/Makefile | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/mips/boot/compressed/Makefile
b/arch/mips/boot/compressed/Makefile
index 011b14512c05..41564a4db6f7 100644
--- a/arch/mips/boot
but pointless. So stop appending to these flags.
Signed-off-by: Paul Bolle <pebo...@tiscali.nl>
---
Build tested only!
arch/cris/boot/compressed/Makefile | 3 ---
arch/cris/boot/rescue/Makefile | 3 ---
2 files changed, 6 deletions(-)
diff --git a/arch/cris/boot/compressed/Makefile
b/arc
but pointless. So stop appending to these flags.
Signed-off-by: Paul Bolle
---
Build tested only!
arch/cris/boot/compressed/Makefile | 3 ---
arch/cris/boot/rescue/Makefile | 3 ---
2 files changed, 6 deletions(-)
diff --git a/arch/cris/boot/compressed/Makefile
b/arch/cris/boot/compressed
1262410.html
> https://marc.info/?l=linux-kernel=147780880425626
> Stat: n/a
> Note: nothing happened yet; BTW: Should build regressions be on this list at
> all?
Fixed by commit 818f38c5b7c4 ("MIPS: Fix build of compressed image"). So this
is actually fixed since v4.9-rc4, but I only looked into this again today.
Thanks,
Paul Bolle
1262410.html
> https://marc.info/?l=linux-kernel=147780880425626
> Stat: n/a
> Note: nothing happened yet; BTW: Should build regressions be on this list at
> all?
Fixed by commit 818f38c5b7c4 ("MIPS: Fix build of compressed image"). So this
is actually fixed since v4.9-rc4, but I only looked into this again today.
Thanks,
Paul Bolle
incorrect. The example that comes to mind is
depends on SH
that I have spotted a few times in the past years. But this treewide pass will
incur some runtime cost and might not be easy to implement cleanly. Perhaps
we're better of with using your script for that.
Thanks,
Paul Bolle
incorrect. The example that comes to mind is
depends on SH
that I have spotted a few times in the past years. But this treewide pass will
incur some runtime cost and might not be easy to implement cleanly. Perhaps
we're better of with using your script for that.
Thanks,
Paul Bolle
th it. Unless you spot some issues I really should be properly
resubmitting it.
Thanks,
Paul Bolle
---- >8 >8 >8 >8 ---
From: Paul Bolle <pebo...@tiscali.nl>
Subject: [PATCH] kconfig: warn if an unknown symbol is selected
A select of an unknown symbol i
th it. Unless you spot some issues I really should be properly
resubmitting it.
Thanks,
Paul Bolle
---- >8 >8 >8 >8 ---
From: Paul Bolle
Subject: [PATCH] kconfig: warn if an unknown symbol is selected
A select of an unknown symbol is basically treated as a nop and is
sile
blic License as published by the Free
> + * Software Foundation, version 2.
> +MODULE_LICENSE("GPL");
Nit: according to the comments in include/linux/module.h the "ident" that
matches the license described at the top of this file is "GPL v2".
Paul Bolle
blic License as published by the Free
> + * Software Foundation, version 2.
> +MODULE_LICENSE("GPL");
Nit: according to the comments in include/linux/module.h the "ident" that
matches the license described at the top of this file is "GPL v2".
Paul Bolle
On Mon, 2016-11-07 at 14:38 +0100, Heiko Carstens wrote:
> On Mon, Nov 07, 2016 at 02:13:06PM +0100, Paul Bolle wrote:
> > --- /dev/null
> > +++ b/arch/s390/include/asm/facilities.h
> > @@ -0,0 +1,43 @@
> > +#ifndef __ASM_FACILITIES_H
> > +#define __AS
On Mon, 2016-11-07 at 14:38 +0100, Heiko Carstens wrote:
> On Mon, Nov 07, 2016 at 02:13:06PM +0100, Paul Bolle wrote:
> > --- /dev/null
> > +++ b/arch/s390/include/asm/facilities.h
> > @@ -0,0 +1,43 @@
> > +#ifndef __ASM_FACILITIES_H
> > +#define __AS
Hi Mashiro,
On Tue, 2016-11-08 at 10:50 +0900, Masahiro Yamada wrote:
> 2016-11-07 21:52 GMT+09:00 Paul Bolle <pebo...@tiscali.nl>:
> > So it seems the odd $(LINUXINCLUDE) variable in that Makefile could be
> > replaced with something like:
> > -include $(srctree)/i
Hi Mashiro,
On Tue, 2016-11-08 at 10:50 +0900, Masahiro Yamada wrote:
> 2016-11-07 21:52 GMT+09:00 Paul Bolle :
> > So it seems the odd $(LINUXINCLUDE) variable in that Makefile could be
> > replaced with something like:
> > -include $(srctree)/include/generated/autoc
goal is to define FACILITIES_ALS and FACILITIES_KVM.
Pasted below is an attempt to use asm/facilities.h instead of
generated/facilities.h. That allows to drop arch/s390/tools/ entirely. I don't
actually have an s390 machine at hand to test this, but this does build and
the preprocessed code this generates l
mpt to use asm/facilities.h instead of
generated/facilities.h. That allows to drop arch/s390/tools/ entirely. I don't
actually have an s390 machine at hand to test this, but this does build and
the preprocessed code this generates looks sane.
(Yes, asm/facilities.h might need another level of pre
1 - 100 of 4150 matches
Mail list logo
