On 1/23/18, Michael Ellerman wrote:
> This has been broken in linux-next for ~6 weeks now, can we please merge
> this and get it fixed.
Added Stephen Rothwell to cc
--
Vincent Legoll
change
the config dependencies.
v2: Added "default y" to avoid breaking existing configs
v3: Fixed wrong indentation, added *-by from Randy
Signed-off-by: Vincent Legoll
Reviewed-by: Randy Dunlap
Tested-by: Randy Dunlap # works for me
---
drivers/virtio/Kconfig | 8 ++--
1 file
change
the config dependencies.
v2: added "default y" to avoid breaking existing configs
Signed-off-by: Vincent Legoll
---
lib/Kconfig.debug | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
index 9d5b78aad4c5..732f7b09a
Argh, sorry for the forgotten "git add", hopefuly this one will be good
change
the config dependencies.
v2: added "default y" to avoid breaking existing configs
Signed-off-by: Vincent Legoll
---
lib/Kconfig.debug | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
index 9d5b78aad4c5..ef6db62a5
Is this one better ?
s an example although there
> are others).
Yes, you're right, I'll resend a v2 with the default y added
Thanks
--
Vincent Legoll
change
the config dependencies.
v2: Added "default y" to avoid breaking existing configs
Signed-off-by: Vincent Legoll
change
the config dependencies.
v2: add "default y" to avoid breaking existing configs
Signed-off-by: Vincent Legoll
---
drivers/virtio/Kconfig | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/virtio/Kconfig b/drivers/virtio/Kconfig
index cf
ork.
Do you want me to respin it with the added default ?
Tchuss
--
Vincent Legoll
an try to work on that too, as it would nicely complement
the menuconfigifcation.
--
Vincent Legoll
The following v2 patch adds the "default y" to avoid breaking existing
configs.
I don't know if using the CONFIG_EFI symbol would be better, it is spread
in multiple arch/*/Kconfig files, and would make the changes more invasive.
Please advise the way forward
Thanks
config dependencies.
v2: added "default y" to menuconfig to avoid breaking existing
configs.
Signed-off-by: Vincent Legoll
---
drivers/firmware/efi/Kconfig | 14 +-
1 file changed, 9 insertions(+), 5 deletions(-)
diff --git a/drivers/firmware/efi/Kconfig b/drivers/fi
change
the config dependencies.
v2: added "default y" to avoid breaking existing configs
Signed-off-by: Vincent Legoll
---
drivers/hv/Kconfig | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/hv/Kconfig b/drivers/hv/Kconfig
index 50b89ea0e60f..a6cda7121
Hello,
The following patch add a "default y" to the menuconfig to avoid breaking
existing configs.
Please advise if this is better, and sufficient.
Tell me if these kind of changes are not welcome in the kernel, so that I
won't continue with other similar work. I am an actual user of menuconfig
Hello,
>> This looks fine to me. Ard?
>
> Doesn't this break existing configs?
Would adding a "default yes" on the new menuconfig be OK.
If yes, I'd respin it for a v2
--
Vincent Legoll
Hello,
looks like this one will need another spin, the kbuild test robot
found some config mix that has unmet dependencies.
Please drop until v3
--
Vincent Legoll
bout how to do it without using menuconfig ?
Thanks
--
Vincent Legoll
s one is not alone,
I've got a bunch of other similar patches in-flight)
Thanks
--
Vincent Legoll
f Kconfig.kasan
Signed-off-by: Vincent Legoll
---
lib/Kconfig.debug | 28 +---
lib/Kconfig.kasan | 3 ---
2 files changed, 17 insertions(+), 14 deletions(-)
diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
index 947d3e2ed5c2..19ca76e7b9ed 100644
--- a/lib/Kconfig.debug
+++
The v2 of the patch tries to do that, but the Kconfig.kasan modification is kind
of infortunate, as it moves the HAVE_ARCH_KASAN config option out of that file.
If this is not acceptable please advise on how I can achieve it.
Thanks
the config dependencies.
Signed-off-by: Vincent Legoll
---
fs/Kconfig | 21 +++--
1 file changed, 15 insertions(+), 6 deletions(-)
diff --git a/fs/Kconfig b/fs/Kconfig
index 7aee6d699fd6..94ee1422e995 100644
--- a/fs/Kconfig
+++ b/fs/Kconfig
@@ -105,29 +105,38 @@ source &qu
ig UI, not change
the config dependencies.
Signed-off-by: Vincent Legoll
---
kernel/rcu/Kconfig.debug | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/kernel/rcu/Kconfig.debug b/kernel/rcu/Kconfig.debug
index 0ec7d1d33a14..f709bc50ac23 100644
--- a/kernel/rcu/Kconfig.d
change
the config dependencies.
Signed-off-by: Vincent Legoll
---
lib/Kconfig.debug | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
index 947d3e2ed5c2..9436714254b7 100644
--- a/lib/Kconfig.debug
+++ b/lib/Kconfig.debug
@@ -1660,7
config dependencies.
Signed-off-by: Vincent Legoll
---
drivers/firmware/efi/Kconfig | 13 -
1 file changed, 8 insertions(+), 5 deletions(-)
diff --git a/drivers/firmware/efi/Kconfig b/drivers/firmware/efi/Kconfig
index 2b4c39fdfa91..8727b1aa8ef6 100644
--- a/drivers/firmware/efi
config dependencies.
Signed-off-by: Vincent Legoll
---
drivers/nvme/Kconfig | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/nvme/Kconfig b/drivers/nvme/Kconfig
index 04008e0bbe81..aad33a0dfbce 100644
--- a/drivers/nvme/Kconfig
+++ b/drivers/nvme/Kconfig
@@ -1,6
change
the config dependencies.
Signed-off-by: Vincent Legoll
---
drivers/remoteproc/Kconfig | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/remoteproc/Kconfig b/drivers/remoteproc/Kconfig
index b609e1d3654b..29341c7d836d 100644
--- a/drivers/remoteproc/Kconfig
change
the config dependencies.
Signed-off-by: Vincent Legoll
---
drivers/virtio/Kconfig | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/virtio/Kconfig b/drivers/virtio/Kconfig
index cff773f15b7e..d485a63a8233 100644
--- a/drivers/virtio/Kconfig
+++ b/drivers
change
the config dependencies.
Signed-off-by: Vincent Legoll
---
drivers/hv/Kconfig | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/hv/Kconfig b/drivers/hv/Kconfig
index 50b89ea0e60f..5804081d936d 100644
--- a/drivers/hv/Kconfig
+++ b/drivers/hv/Kconfig
@@ -1,4
the config dependencies.
Signed-off-by: Vincent Legoll
---
drivers/ptp/Kconfig | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/ptp/Kconfig b/drivers/ptp/Kconfig
index a21ad10d613c..d3f3a9f2b0b1 100644
--- a/drivers/ptp/Kconfig
+++ b/drivers/ptp/Kconfig
@@ -2,7
t OK to do ?
Would moving them out of the if/endif block be acceptable ?
Thanks
--
Vincent Legoll
is only intended to change menuconfig UI, not change
the config dependencies.
Signed-off-by: Vincent Legoll
---
lib/Kconfig.debug | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
index 947d3e2ed5c2..5062f52dbeec 100644
--- a/lib/Kco
the config dependencies.
Signed-off-by: Vincent Legoll
---
drivers/ssb/Kconfig | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/drivers/ssb/Kconfig b/drivers/ssb/Kconfig
index d8e4219c2324..f48a2ee587a4 100644
--- a/drivers/ssb/Kconfig
+++ b/drivers/ssb/Kconfig
Hello,
thanks for the feedback, is that better ?
The following patch makes it easier to disable all SSB config
options without entering the submenu. It will also enable one
to see that en-disabled state from the outside menu.
This is only intended to change menuconfig UI, not change
the config dependencies.
No need to get into the submenu to disable all SSB-related config entries
Signed-off-by: Vincent Legoll
---
drivers/ssb/Kconfig | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/drivers/ssb/Kconfig b/drivers/ssb/Kconfig
index d8e4219c2324..f48a2ee587a4 100644
--- a
No need to get into the submenu to disable all ANDROID-related config entries
Signed-off-by: Vincent Legoll
---
drivers/android/Kconfig | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/drivers/android/Kconfig b/drivers/android/Kconfig
index 7dce3795b887..e9434777acba
No need to get into the submenu to disable all MIC-related config entries
Signed-off-by: Vincent Legoll
---
drivers/misc/mic/Kconfig | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/misc/mic/Kconfig b/drivers/misc/mic/Kconfig
index 227cc7443671..badc84356715
No need to get into the submenu to disable all FSI-related config entries
Signed-off-by: Vincent Legoll
---
drivers/fsi/Kconfig | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/drivers/fsi/Kconfig b/drivers/fsi/Kconfig
index 6821ed0cd5e8..513e35173aaa 100644
--- a
v2: using bool instead of tristate
No need to get into the submenu to disable all TI_ST-related config entries
Signed-off-by: Vincent Legoll
---
drivers/misc/ti-st/Kconfig | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/misc/ti-st/Kconfig b/drivers/misc/ti-st/Kconfig
index f34dcc514730
ecide that this documentation is for clueful ones.
WDYT ?
--
Vincent Legoll
Hello,
On Wed, Jul 5, 2017 at 11:15 PM, Stephen Boyd wrote:
> On 07/02, Vincent Legoll wrote:
>> -menu "Common Clock Framework"
>> +menuconfig COMMON_CLK_FRAMEWORK
>> + tristate "Common Clock Framework"
>> depends on COMMON_CLK
>
>
Hello,
Please drop that patch, it will need to be fixed & tested more before resend
--
Vincent Legoll
Hello,
On Sun, Jul 2, 2017 at 5:53 PM, Vincent Legoll wrote:
> No need to get into the submenu to disable all SCSI Transports-related
> config entries
>
> Signed-off-by: Vincent Legoll
> ---
> drivers/scsi/Kconfig | 7 +--
Please drop that patch, it will need to be f
No need to get into the submenu to disable all common clock framework
-related config entries
Signed-off-by: Vincent Legoll
---
drivers/clk/Kconfig | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/clk/Kconfig b/drivers/clk/Kconfig
index 36cfea3..a1b174c 100644
No need to get into the submenu to disable all SCSI Transports-related
config entries
Signed-off-by: Vincent Legoll
---
drivers/scsi/Kconfig | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/scsi/Kconfig b/drivers/scsi/Kconfig
index 3c52867..53c97cc 100644
--- a
No need to get into the submenu to disable all EEPROM-related config entries
Signed-off-by: Vincent Legoll
---
drivers/misc/eeprom/Kconfig | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/drivers/misc/eeprom/Kconfig b/drivers/misc/eeprom/Kconfig
index
Commit-ID: 5a29ef22098874db79af7bf92a247a0f503bfa6e
Gitweb: http://git.kernel.org/tip/5a29ef22098874db79af7bf92a247a0f503bfa6e
Author: Vincent Legoll
AuthorDate: Tue, 9 May 2017 10:34:09 +0200
Committer: Thomas Gleixner
CommitDate: Fri, 26 May 2017 14:54:05 +0200
genirq: Make
eport a bug
[0.784009] PCI: Using ACPI for IRQ routing
[0.795599] PCI: pci_cache_line_size set to 64 bytes
[0.811154] PCI: CLS 0 bytes, default 64
Signed-off-by: Vincent Legoll
---
arch/x86/pci/pcbios.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/pci/pc
[0.681478] ACPI : EC: interrupt blocked
[0.684798] ACPI: Interpreter enabled
[0.684835] ACPI: (supports S0 S3 S4 S5)
Signed-off-by: Vincent Legoll
---
drivers/acpi/acpi_dbg.c | 2 +-
drivers/acpi/button.c | 2 +-
drivers/acpi/ec.c | 2 +-
drivers/acpi/ioapic.c | 2 +-
4 files
f-by: Vincent Legoll
---
kernel/irq/irqdesc.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/kernel/irq/irqdesc.c b/kernel/irq/irqdesc.c
index 00bb0ae..fb53da0 100644
--- a/kernel/irq/irqdesc.c
+++ b/kernel/irq/irqdesc.c
@@ -480,7 +480,8 @@ int __init early_irq_init(
The early_irq_init() function was not telling what all the displayed
information is.
- Add some missing whitespace & commas for easier reading
- Tell that the third number is the number of preallocated irqs
returned by arch_probe_nr_irqs()
Signed-off-by: Vincent Legoll
---
kernel
On Tue, May 9, 2017 at 9:08 AM, Thomas Gleixner wrote:
> On Wed, 3 May 2017, Vincent Legoll wrote:
>
>> Subject : [PATCH] Fix NR_IRQS printk()
>
> The subject line is missing a subsystem token. Please consult
>
> Documentation/process/submitting-patches.rst
>
Hello,
Any feedback on this ?
Should I forward it to someone else ?
Thanks
--
Vincent Legoll
- Missing some whitespace
- Tell that the third number is "initcnt" (whatever that is)
Signed-off-by: Vincent Legoll
---
kernel/irq/irqdesc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/irq/irqdesc.c b/kernel/irq/irqdesc.c
index 00bb0ae..b18526f 10
No need to get into the submenu to disable all DMABUF-related config entries
Make the selecters also select the new DMABUF menuconfig
Signed-off-by: Vincent Legoll
---
drivers/dma-buf/Kconfig | 7 ---
drivers/gpu/drm/Kconfig | 1 +
drivers/gpu/drm/msm/Kconfig | 1 +
3 files changed
Argh,
looks like the "--in-reply-to" did not hook it up properly...
This was intended as a reply to:
http://lkml.iu.edu/hypermail/linux/kernel/1704.1/04654.html
Re: [PATCH] Make AMBA a menuconfig to ease disabling it all
from (Fri Apr 14 2017 - 08:33:57 EST)
Sorry
--
Vincent Legoll
Hello,
this is more what I wanted to achieve, an easily disableable BCMA menuconfig.
This is working as intended on x86_64 defconfig, whereas it is harder to disable
for ARCH=arm because of "select"s. But that's less of an issue, as those arch's
defconfigs may very well want the feature enbled,
No need to get into the submenu to disable all BCMA-related config entries
Signed-off-by: Vincent Legoll
---
drivers/bcma/Kconfig | 9 ++---
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/drivers/bcma/Kconfig b/drivers/bcma/Kconfig
index b5c48a8..54f81c5 100644
--- a/drivers
No need to get into the submenu to disable all FPGA-related config entries
Signed-off-by: Vincent Legoll
---
drivers/fpga/Kconfig | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig
index ce861a2..2cb9b81 100644
--- a/drivers/fpga
Hello Randy,
you mean like the following ?
Looks like this is working as intended and much simpler, thanks.
l config in a few minutes years ago, I now have to take tens of minutes
going through all. This work is a step trying to make this step quicker.
Is that better ?
Thanks
--
Vincent Legoll
ot confused
by its naming: "Broadcom specific AMBA".
Please disregard that patch, and sorry for the noise.
--
Vincent Legoll
No need to get into the submenu to disable all AMBA-related config entries
Signed-off-by: Vincent Legoll
---
drivers/amba/Kconfig | 12 +++-
1 file changed, 3 insertions(+), 9 deletions(-)
diff --git a/drivers/amba/Kconfig b/drivers/amba/Kconfig
index 294ba6f..3f8fcb8 100644
--- a
submenu to then select each one and disable it.
With a menuentry (and config options depending on it), you get
a way to disable all those config entries at once without having
to enter the menu.
Does that make sense ?
Is there a better way to achieve the disabling-easiness goal ?
Any input a
No need to get into the submenu to disable all TI_ST-related config entries
Signed-off-by: Vincent Legoll
---
drivers/misc/ti-st/Kconfig | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/drivers/misc/ti-st/Kconfig b/drivers/misc/ti-st/Kconfig
index f34dcc5..9b297ca
No need to get into the submenu to disable all FPGA-related config entries
Signed-off-by: Vincent Legoll
---
drivers/fpga/Kconfig | 21 -
1 file changed, 8 insertions(+), 13 deletions(-)
diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig
index ce861a2..fa1a95e 100644
No need to get into the submenu to disable all ANDROID-related config entries
Signed-off-by: Vincent Legoll
---
drivers/android/Kconfig | 12 ++--
1 file changed, 2 insertions(+), 10 deletions(-)
diff --git a/drivers/android/Kconfig b/drivers/android/Kconfig
index a82fc02..c2b6c37
So that there's no need to get into the submenu to disable all related config
entries.
The BROKEN PPS_GENERATOR_PARPORT now also depends on PPS
Signed-off-by: Vincent Legoll
---
drivers/pps/Kconfig| 12 +++-
drivers/pps/clients/Kconfig| 6 ++
drivers/pps/gener
TODO: maybe make more explicit the "POWER" part, as in IBM POWER
architecture or OpenPOWER...
Signed-off-by: Vincent Legoll
---
drivers/fsi/Kconfig | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/fsi/Kconfig b/drivers/fsi/Kconfig
index 04c1a0e..6b8a
On 8/12/07, Randy Dunlap <[EMAIL PROTECTED]> wrote:
> There are also def_boolean and def_tristate.
> I suppose that I should resubmit this patch:
> http://marc.info/?l=linux-kernel&m=115471883121076&w=2
This one is OK for me, you can add:
Signed-off-by: Vincent L
:48: error: previous declaration of 'vg_count' was here
looking in the 2.4.35 patch, this has been modified, is there already
a fix for that ?
Reverting the attached patch hunk fixed the compilation for me...
--
Vincent Legoll
patch-fix-lvm-compil
Description: Binary data
Hello,
here is a small documentation patch for the KConfig file format
"def_bool" type definition that was missing.
I hope the patch is not mangled...
>From 4742eb7a3169001304524e82a88ec1c94f70aa56 Mon Sep 17 00:00:00 2001
From: Vincent Legoll <[EMAIL PROTECTED]>
Date: Sat,
>From 065f7037a99d14c8678384c60046b8d39b1524c8 Mon Sep 17 00:00:00 2001
From: Vincent Legoll <[EMAIL PROTECTED]>
Date: Sat, 4 Aug 2007 17:07:25 +0200
Subject: [PATCH] - Fix typo in Documentation/SubmitChecklist
Signed-off-by: Vincent Legoll <[EMAIL PROTECTED]>
---
Documentation/
I had the same problem, and your patch fixes it.
I found that disabling CONFIG_RTC_DRV_CMOS
fixes^Wworkarounds it too...
Acked-by: Vincent Legoll <[EMAIL PROTECTED]>
--
Vincent Legoll
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a m
At least, that's how it works here :)
I wish I could make it work here too ;-)
--
Vincent Legoll
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Argh I think I screwed the mail threading, I was refering to:
http://ussg.iu.edu/hypermail/linux/kernel/0702.1/1060.html
From: Patrick Ale
Date: Sun Feb 11 2007 - 05:28:21 EST
Sorry
--
Vincent Legoll
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the
inserted in it. The BIOS may initialize it differently then.
--
Vincent Legoll
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FA
Hello,
what about the following, on top of your patch ?
It's trivial modif to kernel-doc style comment...
--
Vincent Legoll
diff --git a/kernel/sched.c b/kernel/sched.c
index f92a239..2c51ec0 100644
--- a/kernel/sched.c
+++ b/kernel/sched.c
@@ -56,8 +56,8 @@
#include
-/*
- * Sche
se meaning depends on @type.
*/
int do_syslog(int type, char __user *buf, int len)
{
@@ -190,11 +192,11 @@
That part looks good to me, and the kernel-doc hunks from
"[patch 3/4] Refactor do_syslog interface"
too, are:
Acked-by: Vincent Legoll <[EMAIL PROTECTED]>
thanks for
Hello,
what about something along the lines of the following,
on top of your patch ?
Or should the kernel-doc be put on another function
instead of that one ?
--
Vincent Legoll
Add do_syslog() kernel-doc
---
commit 95b0721d8b4b46ddf83113fe49492810d7d92060
tree
text properly dispatched in the apropriate
changelogs ?
P.S.: I'm not subscribed, and couldn't find a way to do a
proper reply to Arjan's message, so, sorry to break the
proposed rule myself...
--
Vincent Legoll
Add a suggestion to properly thread patch series
Discussion started
83 matches
Mail list logo