On Wed, Dec 17, 2014 at 6:10 PM, Octavian Purdila
<[email protected]> wrote:
> On Wed, Dec 17, 2014 at 10:59 AM, Alexandre Courbot <[email protected]> wrote:
>> On Wed, Dec 17, 2014 at 5:57 PM, Octavian Purdila
>> <[email protected]> wrote:
>>> On Wed, Dec 17, 2014 at 10:50 AM, Alexandre Courbot <[email protected]> 
>>> wrote:
>>>> On Wed, Dec 17, 2014 at 1:52 AM, Daniel Baluta <[email protected]> 
>>>> wrote:
>>>>>
>>>>>
>>>>> On 12/16/2014 04:40 PM, Axel Lin wrote:
>>>>>>
>>>>>> dln2_gpio_direction_output() ignored the state passed into it. Fix it.
>>>>>>
>>>>>> Signed-off-by: Axel Lin <[email protected]>
>>>>>
>>>>>
>>>>> Tested-by: Daniel Baluta <[email protected]>
>>>>
>>>> Acked-by: Alexandre Courbot <[email protected]>
>>>>
>>>> But this seems to apply to patches in mid-flight, could it be squashed
>>>> there maybe?
>>>
>>> Sure, I can add it to the existing series, but I prefer to keep it as
>>> separate patch. Is that ok with you?
>>
>> Why? This is clearly a fix, so if the series is not merged yet,
>> doesn't it make more sense to squash it and have the desired
>> functionality from the start?
>
> The fix is not for issues introduced by the series, but for an issue
> existing in the already merged code. Also it is a separate issue then
> the one fixed in the other patches in the series.

Allright, I thought none of the DL2 support has been fixed yet (as I
cannot see it in Linus W.'s tree). You know better what the state of
your patches is, so please do what you think is best.
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to