2021-12-28 20:06:47

by Jan Engelhardt

[permalink] [raw]
Subject: [PATCH] android: use "menuconfig" menu entry type for CONFIG_ANDROID

Allow disabling the Android driver section from within the Device
Drivers submenu, without having to descend into the Android drivers
submenu first.

Signed-off-by: Jan Engelhardt <[email protected]>
---
drivers/android/Kconfig | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)

diff --git drivers/android/Kconfig drivers/android/Kconfig
index 53b22e26266c..dc0a373ab879 100644
--- drivers/android/Kconfig
+++ drivers/android/Kconfig
@@ -1,7 +1,5 @@
# SPDX-License-Identifier: GPL-2.0
-menu "Android"
-
-config ANDROID
+menuconfig ANDROID
bool "Android Drivers"
help
Enable support for various drivers needed on the Android platform
@@ -55,5 +53,3 @@ config ANDROID_BINDER_IPC_SELFTEST
alignments.

endif # if ANDROID
-
-endmenu
--
2.34.1



2021-12-30 13:14:30

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [PATCH] android: use "menuconfig" menu entry type for CONFIG_ANDROID

On Tue, Dec 28, 2021 at 09:06:34PM +0100, Jan Engelhardt wrote:
> Allow disabling the Android driver section from within the Device
> Drivers submenu, without having to descend into the Android drivers
> submenu first.
>
> Signed-off-by: Jan Engelhardt <[email protected]>
> ---
> drivers/android/Kconfig | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git drivers/android/Kconfig drivers/android/Kconfig
> index 53b22e26266c..dc0a373ab879 100644
> --- drivers/android/Kconfig
> +++ drivers/android/Kconfig

You made this patch at one level too deep, it should look like:

--- a/drivers/android/Kconfig
+++ b/drivers/android/Kconfig

As-is, this will not apply.

A normal git diff will output the correct patch file, as will 'git
format-patch', how did you create this?

thanks,

greg k-h

2021-12-30 14:14:31

by Jan Engelhardt

[permalink] [raw]
Subject: Re: [PATCH] android: use "menuconfig" menu entry type for CONFIG_ANDROID


On Thursday 2021-12-30 14:14, Greg KH wrote:
>On Tue, Dec 28, 2021 at 09:06:34PM +0100, Jan Engelhardt wrote:
>> Allow disabling the Android driver section from within the Device
>> Drivers submenu, without having to descend into the Android drivers
>> submenu first.
>>
>> Signed-off-by: Jan Engelhardt <[email protected]>
>> ---
>> drivers/android/Kconfig | 6 +-----
>> 1 file changed, 1 insertion(+), 5 deletions(-)
>>
>> diff --git drivers/android/Kconfig drivers/android/Kconfig
>> index 53b22e26266c..dc0a373ab879 100644
>> --- drivers/android/Kconfig
>> +++ drivers/android/Kconfig
>
>You made this patch at one level too deep, it should look like:
>
>--- a/drivers/android/Kconfig
>+++ b/drivers/android/Kconfig
>
>As-is, this will not apply.
>
>A normal git diff will output the correct patch file, as will 'git
>format-patch', how did you create this?

Uh, I once again ran afoul of my default "diff.noprefix=true" in
~/.gitconfig. Unless there is a magical barrier, `git am -p0`
should momentarily work for you as I edit my ~/linux/.git/config to
set diff.noprefix=false.

(which of course breaks part of my workflow just like xkcd.com/1172
predicts, but alas... -- I've already conveyed to the git list it's
suboptimal for me,
https://lore.kernel.org/all/[email protected]/ )

2021-12-30 14:29:54

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [PATCH] android: use "menuconfig" menu entry type for CONFIG_ANDROID

On Thu, Dec 30, 2021 at 03:14:26PM +0100, Jan Engelhardt wrote:
>
> On Thursday 2021-12-30 14:14, Greg KH wrote:
> >On Tue, Dec 28, 2021 at 09:06:34PM +0100, Jan Engelhardt wrote:
> >> Allow disabling the Android driver section from within the Device
> >> Drivers submenu, without having to descend into the Android drivers
> >> submenu first.
> >>
> >> Signed-off-by: Jan Engelhardt <[email protected]>
> >> ---
> >> drivers/android/Kconfig | 6 +-----
> >> 1 file changed, 1 insertion(+), 5 deletions(-)
> >>
> >> diff --git drivers/android/Kconfig drivers/android/Kconfig
> >> index 53b22e26266c..dc0a373ab879 100644
> >> --- drivers/android/Kconfig
> >> +++ drivers/android/Kconfig
> >
> >You made this patch at one level too deep, it should look like:
> >
> >--- a/drivers/android/Kconfig
> >+++ b/drivers/android/Kconfig
> >
> >As-is, this will not apply.
> >
> >A normal git diff will output the correct patch file, as will 'git
> >format-patch', how did you create this?
>
> Uh, I once again ran afoul of my default "diff.noprefix=true" in
> ~/.gitconfig. Unless there is a magical barrier, `git am -p0`
> should momentarily work for you as I edit my ~/linux/.git/config to
> set diff.noprefix=false.

I use the tool 'b4' to apply patches, which sucks it out of the public
archive and then uses 'git am' to apply the patch. So I can't apply
your patch as-is, sorry.

Please resend.

thanks,

greg k-h