From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by befuddled.reisers.ca (Postfix, from userid 65534) id 69B8B1EFD27; Sat, 19 Nov 2016 17:03:33 -0500 (EST) Received: from hera.aquilenet.fr (hera.aquilenet.fr [IPv6:2a01:474::1]) by befuddled.reisers.ca (Postfix) with ESMTP id 00B031EFD0F for ; Sat, 19 Nov 2016 17:03:32 -0500 (EST) Received: from localhost (localhost [127.0.0.1]) by hera.aquilenet.fr (Postfix) with ESMTP id DB9D77A6; Sat, 19 Nov 2016 23:03:29 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at aquilenet.fr Received: from hera.aquilenet.fr ([127.0.0.1]) by localhost (hera.aquilenet.fr [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id d4qx5ndyrrjY; Sat, 19 Nov 2016 23:03:29 +0100 (CET) Received: from var.youpi.perso.aquilenet.fr (unknown [IPv6:2a01:cb19:181:c200:3602:86ff:fe2c:6a19]) by hera.aquilenet.fr (Postfix) with ESMTPSA id 42BDF339; Sat, 19 Nov 2016 23:03:29 +0100 (CET) Received: from samy by var.youpi.perso.aquilenet.fr with local (Exim 4.88) (envelope-from ) id 1c8DjM-0002TU-O9; Sat, 19 Nov 2016 23:03:28 +0100 Date: Sat, 19 Nov 2016 23:03:28 +0100 From: Samuel Thibault To: Okash Khawaja Cc: kirk@reisers.ca, david.a.borowski@gmail.com, chris@the-brannons.com, w.d.hubbs@gmail.com, gregkh@linuxfoundation.org, speakup@linux-speakup.org, devel@driverdev.osuosl.org Subject: Re: [PATCH v2 0/4] staging: speakup: refactor to make room for serial comms changes Message-ID: <20161119220328.GH3422@var.home> References: <20161119183514.GA2077@sanghar> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20161119183514.GA2077@sanghar> User-Agent: Mutt/1.5.21+34 (58baf7c9f32f) (2010-12-30) X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 X-BeenThere: speakup@linux-speakup.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: "Speakup is a screen review system for Linux." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 19 Nov 2016 22:03:33 -0000 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. BTW, I have been looking at functions again, we'll also need a spk_synth_immediate_tty that drivers can use instead of spk_synth_immediate, in addition to the _in, and _out functions. For now (i.e. to make the dummy driver work, and probably a few simple more), the tty versions for _out and _synth_immediate will probably enough to get something working and commitable to the main kernel. Also, as mentioned, now the hard work is getting to open the tty from the kernel :) That'd be in a spk_tty_synth_probe function that drivers would use instead of spk_serial_synth_probe. Samuel