Re: get_maintainer.pl and too many names (was: Re: [PATCH v2 0/2] i2c: uniphier: add two I2C controller drivers for UniPhier SoC platform)
On Mon, 2015-10-26 at 11:59 +0100, Geert Uytterhoeven wrote: > Hi Joe, Hi Geert. > Would it be difficult to let get_maintainer.pl print a warning if more than 5 > names come up? You can do whatever you like using shell scripts. As is, there are mechanisms in get_maintainer to limit the number of maintainers shown. see: ./scripts/get_maintainer.pl --help -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
get_maintainer.pl and too many names (was: Re: [PATCH v2 0/2] i2c: uniphier: add two I2C controller drivers for UniPhier SoC platform)
Hi Joe,
On Mon, Oct 26, 2015 at 11:47 AM, Lee Jones wrote:
> On Mon, 26 Oct 2015, Geert Uytterhoeven wrote:
>> On Mon, Oct 26, 2015 at 9:27 AM, Masahiro Yamada
>> wrote:
>> > 2015-10-26 17:17 GMT+09:00 Lee Jones :
>> >>> This series adds two I2C controller drivers.
>> >>> (they are completely different IPs.)
>> >>>
>> >>> The first one is a very simple FIFO-less I2C controller,
>> >>> which is used on some older UniPhier SoCs.
>> >>>
>> >>> The other one is higher-performance I2C controller with TX/RX FIFO,
>> >>> used on newer UniPhier SoCs.
>> >>
>> >> And you have sent this to me because ... ?
>> >
>> >
>> > No special reason.
>> >
>> >
>> > I sent this series to linux-...@vger.kernel.org.
>> >
>> > I guess you were automatically CC'ed by scripts/get_maintainer.pl.
>> >
>> > Using get_maintainer.pl is a normal process when sending patches, I think.
>>
>> Please use common sense. It doesn't make much sense to CC everybody who
>> ever made a minor edit to an affected file.
>>
>> If checkpatch comes up with more than 5 names, this should ring a bell.
get_maintainer.pl, of course
> +1
Would it be difficult to let get_maintainer.pl print a warning if more than 5
names come up?
Thanks!
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH v2 0/2] i2c: uniphier: add two I2C controller drivers for UniPhier SoC platform
On Mon, 26 Oct 2015, Geert Uytterhoeven wrote: > On Mon, Oct 26, 2015 at 9:27 AM, Masahiro Yamada > wrote: > > 2015-10-26 17:17 GMT+09:00 Lee Jones : > >>> This series adds two I2C controller drivers. > >>> (they are completely different IPs.) > >>> > >>> The first one is a very simple FIFO-less I2C controller, > >>> which is used on some older UniPhier SoCs. > >>> > >>> The other one is higher-performance I2C controller with TX/RX FIFO, > >>> used on newer UniPhier SoCs. > >> > >> And you have sent this to me because ... ? > > > > > > No special reason. > > > > > > I sent this series to linux-...@vger.kernel.org. > > > > I guess you were automatically CC'ed by scripts/get_maintainer.pl. > > > > Using get_maintainer.pl is a normal process when sending patches, I think. > > Please use common sense. It doesn't make much sense to CC everybody who > ever made a minor edit to an affected file. > > If checkpatch comes up with more than 5 names, this should ring a bell. +1 -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH v2 0/2] i2c: uniphier: add two I2C controller drivers for UniPhier SoC platform
On Mon, Oct 26, 2015 at 09:31:15AM +0100, Geert Uytterhoeven wrote: > On Mon, Oct 26, 2015 at 9:27 AM, Masahiro Yamada > wrote: > > 2015-10-26 17:17 GMT+09:00 Lee Jones : > >>> This series adds two I2C controller drivers. > >>> (they are completely different IPs.) > >>> > >>> The first one is a very simple FIFO-less I2C controller, > >>> which is used on some older UniPhier SoCs. > >>> > >>> The other one is higher-performance I2C controller with TX/RX FIFO, > >>> used on newer UniPhier SoCs. > >> > >> And you have sent this to me because ... ? > > > > > > No special reason. > > > > > > I sent this series to linux-...@vger.kernel.org. > > > > I guess you were automatically CC'ed by scripts/get_maintainer.pl. > > > > Using get_maintainer.pl is a normal process when sending patches, I think. > > Please use common sense. It doesn't make much sense to CC everybody who > ever made a minor edit to an affected file. > > If checkpatch comes up with more than 5 names, this should ring a bell. I usually use get_maintainer.pl with --no-git-fallback to catch people listed in MAINTAINERS only. But mileages vary a lot in that area. signature.asc Description: Digital signature
Re: [PATCH v2 0/2] i2c: uniphier: add two I2C controller drivers for UniPhier SoC platform
On Mon, Oct 26, 2015 at 9:27 AM, Masahiro Yamada
wrote:
> 2015-10-26 17:17 GMT+09:00 Lee Jones :
>>> This series adds two I2C controller drivers.
>>> (they are completely different IPs.)
>>>
>>> The first one is a very simple FIFO-less I2C controller,
>>> which is used on some older UniPhier SoCs.
>>>
>>> The other one is higher-performance I2C controller with TX/RX FIFO,
>>> used on newer UniPhier SoCs.
>>
>> And you have sent this to me because ... ?
>
>
> No special reason.
>
>
> I sent this series to linux-...@vger.kernel.org.
>
> I guess you were automatically CC'ed by scripts/get_maintainer.pl.
>
> Using get_maintainer.pl is a normal process when sending patches, I think.
Please use common sense. It doesn't make much sense to CC everybody who
ever made a minor edit to an affected file.
If checkpatch comes up with more than 5 names, this should ring a bell.
Thanks!
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH v2 0/2] i2c: uniphier: add two I2C controller drivers for UniPhier SoC platform
Hi Lee, 2015-10-26 17:17 GMT+09:00 Lee Jones : >> This series adds two I2C controller drivers. >> (they are completely different IPs.) >> >> The first one is a very simple FIFO-less I2C controller, >> which is used on some older UniPhier SoCs. >> >> The other one is higher-performance I2C controller with TX/RX FIFO, >> used on newer UniPhier SoCs. > > And you have sent this to me because ... ? No special reason. I sent this series to linux-...@vger.kernel.org. I guess you were automatically CC'ed by scripts/get_maintainer.pl. Using get_maintainer.pl is a normal process when sending patches, I think. -- Best Regards Masahiro Yamada -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH v2 0/2] i2c: uniphier: add two I2C controller drivers for UniPhier SoC platform
> This series adds two I2C controller drivers. > (they are completely different IPs.) > > The first one is a very simple FIFO-less I2C controller, > which is used on some older UniPhier SoCs. > > The other one is higher-performance I2C controller with TX/RX FIFO, > used on newer UniPhier SoCs. And you have sent this to me because ... ? > Changes in v2: > - Delete if-block after platform_get_resource() > - Add I2C_FUNC_SMBUS_EMUL to functionality > - Add bindings document > - Add entry to MAINTAINERS > - Delete if-block after platform_get_resource() > - Add I2C_FUNC_SMBUS_EMUL to functionality > - Add bindings document > - Fix a typo in a comment block > > Masahiro Yamada (2): > i2c: uniphier: add UniPhier FIFO-less I2C driver > i2c: uniphier_f: add UniPhier FIFO-builtin I2C driver > > .../devicetree/bindings/i2c/i2c-uniphier-f.txt | 25 + > .../devicetree/bindings/i2c/i2c-uniphier.txt | 25 + > MAINTAINERS| 1 + > drivers/i2c/busses/Kconfig | 16 + > drivers/i2c/busses/Makefile| 2 + > drivers/i2c/busses/i2c-uniphier-f.c| 584 > + > drivers/i2c/busses/i2c-uniphier.c | 441 > 7 files changed, 1094 insertions(+) > create mode 100644 Documentation/devicetree/bindings/i2c/i2c-uniphier-f.txt > create mode 100644 Documentation/devicetree/bindings/i2c/i2c-uniphier.txt > create mode 100644 drivers/i2c/busses/i2c-uniphier-f.c > create mode 100644 drivers/i2c/busses/i2c-uniphier.c > -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH v2 0/2] i2c: uniphier: add two I2C controller drivers for UniPhier SoC platform
This series adds two I2C controller drivers. (they are completely different IPs.) The first one is a very simple FIFO-less I2C controller, which is used on some older UniPhier SoCs. The other one is higher-performance I2C controller with TX/RX FIFO, used on newer UniPhier SoCs. Changes in v2: - Delete if-block after platform_get_resource() - Add I2C_FUNC_SMBUS_EMUL to functionality - Add bindings document - Add entry to MAINTAINERS - Delete if-block after platform_get_resource() - Add I2C_FUNC_SMBUS_EMUL to functionality - Add bindings document - Fix a typo in a comment block Masahiro Yamada (2): i2c: uniphier: add UniPhier FIFO-less I2C driver i2c: uniphier_f: add UniPhier FIFO-builtin I2C driver .../devicetree/bindings/i2c/i2c-uniphier-f.txt | 25 + .../devicetree/bindings/i2c/i2c-uniphier.txt | 25 + MAINTAINERS| 1 + drivers/i2c/busses/Kconfig | 16 + drivers/i2c/busses/Makefile| 2 + drivers/i2c/busses/i2c-uniphier-f.c| 584 + drivers/i2c/busses/i2c-uniphier.c | 441 7 files changed, 1094 insertions(+) create mode 100644 Documentation/devicetree/bindings/i2c/i2c-uniphier-f.txt create mode 100644 Documentation/devicetree/bindings/i2c/i2c-uniphier.txt create mode 100644 drivers/i2c/busses/i2c-uniphier-f.c create mode 100644 drivers/i2c/busses/i2c-uniphier.c -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
