[GIT PULL] STM32 defconfig changes for v4.15 #1

[Alexandre Torgue]

On 10/20/2017 09:40 AM, Arnd Bergmann wrote:
> On Fri, Oct 20, 2017 at 9:33 AM, Alexandre Torgue
> <alexandre.torgue at st.com> wrote:
>> On 10/19/2017 10:57 PM, Arnd Bergmann wrote:
>>> On Tue, Oct 17, 2017 at 12:16 PM, Alexandre Torgue
>>>
>>> Pulled into next/soc.
>>>
>>> Splitting out the first patch is fine, but there really is no need to have
>>> one
>>> patch per new Kconfig option you enable, it's less work and more readable
>>> to
>>> have a combined patch that explains what the options are and why you need
>>> all of them.
>>>
>>>           Arnd
>>>
>>
>> ok. So you mean I would have to merge "Add DMAMUX support", "Add I2C F7
>> support", "Add MDMA support" in one patch before to submit the pull request
>> ? If yes, it will be done for next PR.
> 
> Right, it's about finding the right balance. Most of the time, splitting up a
> patch into smaller independent units makes sense when you explain
> what each one does, but in cases like this (adding just one config option
> or one DT node), it becomes more readable if you do multiple related
> changes at once.
> 
> The best test is usually what you put in the changelog. If you have multiple
> patches with almost-identical descriptions, merge them. If you have one
> patch with a description that talks about unrelated problems, split it up.
> 
>         Arnd
> 

Ok. Thanks for the tips ;)

Regards
Alex