From: swarren@wwwdotorg.org (Stephen Warren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v8 11/12] leds: Kconfig for Palmas LEDs
Date: Wed, 13 Mar 2013 14:31:46 -0600 [thread overview]
Message-ID: <5140E232.2080701@wwwdotorg.org> (raw)
In-Reply-To: <513A1C25.7050204@slimlogic.co.uk>
On 03/08/2013 10:13 AM, Ian Lartey wrote:
> On 08/03/13 01:13, Kim, Milo wrote:
>>> Add the Kconfig and Makefile for the Palmas LED driver.
>>
>> It would be better if the Palmas LED driver files are combined into
>> one patch.
>> [PATCH v8 09/12] leds: Add support for Palmas LEDs
>> [PATCH v8 11/12] leds: Kconfig for Palmas LEDs
>
> [PATCH v8 11/12] leds: Kconfig for Palmas LEDs
> can often cause a merge conflict if there are already
> additional drivers/files in the leds directory
>
> [PATCH v8 09/12] leds: Add support for Palmas LEDs
> has more chance of being applied without any issues
> (unless the person taking the patch already has
> their own leds-palmas.c.
A merge conflict isn't really influenced at all by how many other files
are included in the patch that causes it, and both files are going to
end up going through the same tree at the same time. So, there's no
benefit at all that I can see from splitting the C file and Kconfig.
next prev parent reply other threads:[~2013-03-13 20:31 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-07 13:17 [PATCH v8 01/12] mfd: DT bindings for the palmas family MFD Ian Lartey
2013-03-07 13:17 ` [PATCH v8 02/12] mfd: palmas: is_palmas_charger needed by multiple drivers Ian Lartey
2013-03-07 13:17 ` [PATCH v8 03/12] mfd: palmas add variant and OTP detection Ian Lartey
2013-03-07 13:17 ` [PATCH v8 04/12] regulator: palmas correct dt parsing Ian Lartey
2013-03-08 9:44 ` Mark Brown
2013-03-07 13:17 ` [PATCH v8 05/12] watchdog: add Palmas Watchdog support Ian Lartey
2013-03-07 13:17 ` [PATCH v8 06/12] watchdog: Kconfig for Palmas watchdog Ian Lartey
2013-03-07 13:17 ` [PATCH v8 07/12] gpio: palmas: add in GPIO support for palmas charger Ian Lartey
2013-03-13 14:15 ` Linus Walleij
2013-03-14 11:58 ` Ian Lartey
2013-03-07 13:17 ` [PATCH v8 08/12] gpio: palmas: Enable DT support for palmas gpio Ian Lartey
2013-03-13 14:20 ` Linus Walleij
2013-03-07 13:17 ` [PATCH v8 09/12] leds: Add support for Palmas LEDs Ian Lartey
2013-03-08 1:13 ` Kim, Milo
2013-03-11 15:16 ` Ian Lartey
2013-03-07 13:17 ` [PATCH v8 10/12] clk: Kconfig for Palmas clock driver Ian Lartey
2013-03-08 1:13 ` Kim, Milo
2013-03-08 17:14 ` Ian Lartey
2013-03-07 13:17 ` [PATCH v8 11/12] leds: Kconfig for Palmas LEDs Ian Lartey
2013-03-08 1:13 ` Kim, Milo
2013-03-08 17:13 ` Ian Lartey
2013-03-13 20:31 ` Stephen Warren [this message]
2013-03-13 23:48 ` Ian Lartey
2013-03-07 13:17 ` [PATCH v8 12/12] clk: add a clock driver for palmas Ian Lartey
2013-03-21 21:23 ` Mike Turquette
2013-03-07 13:23 ` [PATCH v8 0/12] Palmas Updates Ian Lartey
2013-03-08 7:12 ` Linus Walleij
2013-03-08 17:12 ` Ian Lartey
2013-03-13 20:28 ` [PATCH v8 01/12] mfd: DT bindings for the palmas family MFD Stephen Warren
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5140E232.2080701@wwwdotorg.org \
--to=swarren@wwwdotorg.org \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).