[PATCH v2 0/4] staging: speakup: refactor to make room for serial comms changes
Okash Khawaja
okash.khawaja at gmail.com
Mon Nov 21 05:30:37 EST 2016
> On 21 Nov 2016, at 10:05, Greg KH <gregkh at linuxfoundation.org> wrote:
>
>> On Sat, Nov 19, 2016 at 11:03:28PM +0100, Samuel Thibault wrote:
>> Hello,
>>
>> Just looking over the patch, it looks good. I don't think it needs to be
>> split in 4 pieces though, I'd say merge patches 1, 3, and 4 together,
>> where patch 3 would be modified to make dummy use spk_serial_out too, to
>> get a patch which does just one complete thing: make serial_out a
>> method.
>>
>> Then you'll have other patches introducing spk_serial_out_tty and making
>> some drivers use it.
>>
>> Also note that as Greg said, the patches you have just submitted do
>> not really make sense alone. I don't know if staging people feel like
>> applying them while it's only pavement for future work. If so, then
>> good; otherwise, then fine too: it's good that you have sent them so
>> that we could check how they look like, I just wanted to let you know
>> that they might not get applied yet, just because one may want to see
>> the future patches before applying what you have done so far.
>
> I would like to see the full patch series that uses these changes as
> right now these patches do nothing on their own, so I don't want to
> apply them.
>
> thanks,
>
> greg k-h
Sure, will send complete patch set.
Thanks,
Okash
More information about the Speakup
mailing list