* Re: [PATCH 1/3] MAINTAINERS: Remove asterisk from EFI directory names
@ 2016-05-13 15:11 Xose Vazquez Perez
2016-05-13 15:23 ` Joe Perches
0 siblings, 1 reply; 6+ messages in thread
From: Xose Vazquez Perez @ 2016-05-13 15:11 UTC (permalink / raw)
To: Matt Fleming, Mark Rutland, Greg KH, Joe Perches, LKML
Matt Fleming wrote:
> Mark reported that having asterisks on the end of directory names
> confuses get_maintainer.pl when it encounters subdirectories, and that
If this's a get_maintainer.pl bug, it should also happen in:
F: arch/arm64/boot/dts/qcom/*
F: arch/mips/bcm47xx/*
F: arch/mips/bmips/*
F: arch/mips/include/asm/mach-bcm47xx/*
F: arch/mips/include/asm/mach-bmips/*
F: arch/x86/kernel/cpu/mcheck/*
F: arch/x86/kernel/cpu/microcode/*
F: arch/x86/net/*
F: Documentation/devicetree/bindings/arc/*
F: Documentation/misc-devices/mei/*
F: Documentation/ptp/*
F: drivers/block/xen-blkback/*
F: drivers/char/*
F: drivers/cpuidle/*
F: drivers/firmware/broadcom/*
F: drivers/hwtracing/coresight/*
F: drivers/media/i2c/s5c73m3/*
F: drivers/media/pci/netup_unidvb/*
F: drivers/media/platform/vivid/*
F: drivers/media/usb/pwc/*
F: drivers/misc/*
F: drivers/misc/mei/*
F: drivers/mtd/nand/gpmi-nand/*
F: drivers/net/ethernet/pasemi/*
F: drivers/net/ethernet/tehuti/*
F: drivers/net/wireless/ath/*
F: drivers/net/xen-netback/*
F: drivers/nvdimm/*
F: drivers/ptp/*
F: drivers/soc/samsung/*
F: drivers/soc/ti/*
F: fs/*
F: kernel/events/*
F: net/wireless/*
F: sound/soc/blackfin/*
F: sound/soc/samsung/*
F: tools/testing/selftests/seccomp/*
> my name does not appear when run on drivers/firmware/efi/libstub.
> Reported-by: Mark Rutland <mark.rutland@arm.com>
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: Catalin Marinas <catalin.marinas@arm.com>
> Cc: <stable@vger.kernel.org>
> Signed-off-by: Matt Fleming <matt@codeblueprint.co.uk>
> ---
> MAINTAINERS | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 42e65d128d01..4dca3b3895ba 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -4223,8 +4223,8 @@ F: Documentation/efi-stub.txt
> F: arch/ia64/kernel/efi.c
> F: arch/x86/boot/compressed/eboot.[ch]
> F: arch/x86/include/asm/efi.h
> -F: arch/x86/platform/efi/*
> -F: drivers/firmware/efi/*
> +F: arch/x86/platform/efi/
> +F: drivers/firmware/efi/
> F: include/linux/efi*.h
>
> EFI VARIABLE FILESYSTEM
> --
> 2.7.3
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/3] MAINTAINERS: Remove asterisk from EFI directory names
2016-05-13 15:11 [PATCH 1/3] MAINTAINERS: Remove asterisk from EFI directory names Xose Vazquez Perez
@ 2016-05-13 15:23 ` Joe Perches
2016-05-13 15:34 ` Matt Fleming
2016-05-13 16:15 ` Xose Vazquez Perez
0 siblings, 2 replies; 6+ messages in thread
From: Joe Perches @ 2016-05-13 15:23 UTC (permalink / raw)
To: Xose Vazquez Perez, Matt Fleming, Mark Rutland, Greg KH, LKML
On Fri, 2016-05-13 at 17:11 +0200, Xose Vazquez Perez wrote:
> Matt Fleming wrote:
>
> >
> > Mark reported that having asterisks on the end of directory names
> > confuses get_maintainer.pl when it encounters subdirectories, and
> > that
> If this's a get_maintainer.pl bug, it should also happen in:
Mark is confused.
The meaning of * is all the files in a particular
directory but not any subdirectories.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/3] MAINTAINERS: Remove asterisk from EFI directory names
2016-05-13 15:23 ` Joe Perches
@ 2016-05-13 15:34 ` Matt Fleming
2016-05-13 16:15 ` Xose Vazquez Perez
1 sibling, 0 replies; 6+ messages in thread
From: Matt Fleming @ 2016-05-13 15:34 UTC (permalink / raw)
To: Joe Perches; +Cc: Xose Vazquez Perez, Mark Rutland, Greg KH, LKML
On Fri, 13 May, at 08:23:47AM, Joe Perches wrote:
> On Fri, 2016-05-13 at 17:11 +0200, Xose Vazquez Perez wrote:
> > Matt Fleming wrote:
> >
> > >
> > > Mark reported that having asterisks on the end of directory names
> > > confuses get_maintainer.pl when it encounters subdirectories, and
> > > that
> > If this's a get_maintainer.pl bug, it should also happen in:
>
> Mark is confused.
Sorry, "confuses" was my word, not Mark's.
> The meaning of * is all the files in a particular
> directory but not any subdirectories.
Exactly. This is not what EFI maintainer entries should be using. I am
responsible for everything under drivers/firmware/efi, including
subdirectories.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/3] MAINTAINERS: Remove asterisk from EFI directory names
2016-05-13 15:23 ` Joe Perches
2016-05-13 15:34 ` Matt Fleming
@ 2016-05-13 16:15 ` Xose Vazquez Perez
1 sibling, 0 replies; 6+ messages in thread
From: Xose Vazquez Perez @ 2016-05-13 16:15 UTC (permalink / raw)
To: Joe Perches, Matt Fleming, Mark Rutland, Greg KH, LKML
On 05/13/2016 05:23 PM, Joe Perches wrote:
> The meaning of * is all the files in a particular
> directory but not any subdirectories.
Then "*" is only relevant for these entries:
drivers/char/*
drivers/misc/*
drivers/net/wireless/ath/*
fs/*
drivers/cpuidle/* <---- This is a bug, it should be drivers/cpuidle
All the others only have files inside, and "*"
is redundant.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [GIT PULL 0/3] EFI urgent fixes
@ 2016-05-03 19:29 Matt Fleming
[not found] ` <1462303781-8686-1-git-send-email-matt-mF/unelCI9GS6iBeEJttW/XRex20P6io@public.gmane.org>
0 siblings, 1 reply; 6+ messages in thread
From: Matt Fleming @ 2016-05-03 19:29 UTC (permalink / raw)
To: Ingo Molnar, Thomas Gleixner, H . Peter Anvin
Cc: Matt Fleming, Ard Biesheuvel, linux-kernel, linux-efi,
Catalin Marinas, David Herrmann, Josh Boyer, Josh Triplett,
Mark Rutland, Môshe van der Sterre, Peter Jones, stable,
Tomi Valkeinen, Wang YanQing
Folks, here are a few small fixes. One from Josh to stop the ACPI BGRT
driver wrecking the splash screen even though the "quiet" kernel
parameter is passed, another to fix sysfb_efi on a ThinkPad E550, and
one to MAINTAINERS so that get_maintainer.pl finds the EFI entry for
all subdirectores of drivers/firmware/efi/.
The following changes since commit 04974df8049fc4240d22759a91e035082ccd18b4:
Linux 4.6-rc6 (2016-05-01 15:52:31 -0700)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/mfleming/efi.git tags/efi-urgent
for you to fetch changes up to c255f4fb93d2d92527ede3f57f90b4eb1698bbec:
x86/efi-bgrt: Switch all pr_err() to pr_notice() for invalid BGRT (2016-05-03 20:02:25 +0100)
----------------------------------------------------------------
* Fix sysfb_efi to detect BARs starting at 0x0, which otherwise makes
video=efifb not work on some the ThinkPad E550 - Wang YanQing
* Update the MAINTAINERS entry for EFI by removing the trailing
asterisk characters which was intended to denote "all
subdirectories" but which breaks get_maintainer.pl - Matt Fleming
* Convert ACPI BGRT driver messages from pr_err() to pr_notice() to
avoid spewing messages all over the display and making it
impossible for users to have a pretty boot experience - Josh Boyer
----------------------------------------------------------------
Josh Boyer (1):
x86/efi-bgrt: Switch all pr_err() to pr_notice() for invalid BGRT
Matt Fleming (1):
MAINTAINERS: Remove asterisk from EFI directory names
Wang YanQing (1):
x86/sysfb_efi: Fix valid BAR address range check
MAINTAINERS | 4 ++--
arch/x86/kernel/sysfb_efi.c | 14 ++++++++++++--
arch/x86/platform/efi/efi-bgrt.c | 18 +++++++++---------
3 files changed, 23 insertions(+), 13 deletions(-)
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-05-13 16:15 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-05-13 15:11 [PATCH 1/3] MAINTAINERS: Remove asterisk from EFI directory names Xose Vazquez Perez
2016-05-13 15:23 ` Joe Perches
2016-05-13 15:34 ` Matt Fleming
2016-05-13 16:15 ` Xose Vazquez Perez
-- strict thread matches above, loose matches on Subject: below --
2016-05-03 19:29 [GIT PULL 0/3] EFI urgent fixes Matt Fleming
[not found] ` <1462303781-8686-1-git-send-email-matt-mF/unelCI9GS6iBeEJttW/XRex20P6io@public.gmane.org>
2016-05-03 19:29 ` [PATCH 1/3] MAINTAINERS: Remove asterisk from EFI directory names Matt Fleming
2016-05-03 19:29 ` Matt Fleming
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.