From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by befuddled.reisers.ca (Postfix, from userid 65534) id 3D1DD1F0960; Sun, 9 Apr 2017 13:01:51 -0400 (EDT) Received: from mail-wm0-x243.google.com (mail-wm0-x243.google.com [IPv6:2a00:1450:400c:c09::243]) by befuddled.reisers.ca (Postfix) with ESMTPS id D8FE91F0958 for ; Sun, 9 Apr 2017 13:01:48 -0400 (EDT) Received: by mail-wm0-x243.google.com with SMTP id d79so5982837wmi.2 for ; Sun, 09 Apr 2017 10:01:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=7pi4AO+3H/f5jbu6ER4MlrsyrxjMmGWrfpfZDq+ZVE0=; b=DDkmJzZNlYvwELp/nFSmKS1KqbzfplvHCHiGuiibh6RXMXNxjcAIYL9f442Cv1Z89H z1dl2i8dnIvXPI5iVrQuP+obNP98f5WxcDG36p7VaopWT9gTx4k5ZZefIwK6nhDrnt6m 4rdHcp7hL8yFZ/CRWrbaFVRVqJZn3lX8I00R09qQ1YHT3pUm+73DH4qsxfBKzf6e2t2q 9UwdUr8Crci5JP15Mb5sK8d7Qk5jz3ZtzmTa1RPAk/KVIQeOFRyVaI5inQw+FNUIT38O fcB4KSQzrccV6FT35YDUWjkPUojx+dcqNfgPE5G9aABpMqwp2dODTuNA0Ussmo2abDjZ h4nw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=7pi4AO+3H/f5jbu6ER4MlrsyrxjMmGWrfpfZDq+ZVE0=; b=OmGPovGxMd46ddQf2WZGAigcqeNesv/qZOC0+4nI7Qp6Hj1yS0P1Y0i2sAvV7QkIgx /Vu3KA3bQMSN3oULpRv4YlJqY+09QX623dHKaAKcw/okf49B50IasWFWCm+MoTILRTgN /YKVUJ+TPb/pdfCy+4NL4ZbYd4PsSt+lYFbz3iOoT7N3I22SiVvSwg25nc3kG637zOz8 cSDRplk3TewaoquSt/rTCJMNJL8Rg9KELu/LDBL0zDuPL3T9vJ48hmDcmnMIJoywrsfp nU40S24qTocti2se2TLaFZmQzKOLAGtClKBpsddySoKvwtOyYyh2yztopuXQCKisTgcs ETBA== X-Gm-Message-State: AN3rC/6CEEu9RXSAhINcSsrdaLNriUBuRnGZgNBQvY7hkvtAi21CRenR m6oUu5MZrzk4Iw== X-Received: by 10.28.94.15 with SMTP id s15mr7167413wmb.89.1491756877723; Sun, 09 Apr 2017 09:54:37 -0700 (PDT) Received: from sanghar (host86-163-88-104.range86-163.btcentralplus.com. [86.163.88.104]) by smtp.gmail.com with ESMTPSA id s67sm14123958wrc.5.2017.04.09.09.54.36 (version=TLS1 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Sun, 09 Apr 2017 09:54:36 -0700 (PDT) Date: Sun, 9 Apr 2017 17:54:34 +0100 From: Okash Khawaja To: Samuel Thibault , Rob Herring , Greg Kroah-Hartman , Jiri Slaby , "linux-kernel@vger.kernel.org" , devel@driverdev.osuosl.org, "linux-serial@vger.kernel.org" , Kirk Reiser , Chris Brannon , speakup@linux-speakup.org Subject: Re: [patch 0/7] staging: speakup: introduce tty-based comms Message-ID: <20170409165434.GA1919@sanghar> References: <20170313220551.312820470@gmail.com> <20170313221404.GA22871@kroah.com> <20170314004701.kuecu2zu4md5ccrq@var.youpi.perso.aquilenet.fr> <20170314011814.p3t4zfcx3cxhgvwl@var.youpi.perso.aquilenet.fr> <20170315150323.pjda5qnxauxk5pcf@var.youpi.perso.aquilenet.fr> <20170322000524.7k653z7ui2zzgan6@var.youpi.perso.aquilenet.fr> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170322000524.7k653z7ui2zzgan6@var.youpi.perso.aquilenet.fr> User-Agent: Mutt/1.7.2 (2016-11-26) 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: Sun, 09 Apr 2017 17:01:51 -0000 Hi, Any updates on this? We can start working in tty_port_register_serdev_device() if that's okay? Thanks, Okash On Wed, Mar 22, 2017 at 01:05:24AM +0100, Samuel Thibault wrote: > Hello, > > So Rob, how do you see this going? Shall we introduce a serdev_device > *tty_port_register_serdev_device(tty, device)? Will you work on it or > should I give it a try? > > Samuel > > Samuel Thibault, on mer. 15 mars 2017 16:03:23 +0100, wrote: > > Rob Herring, on mer. 15 mars 2017 09:45:59 -0500, wrote: > > > On Mon, Mar 13, 2017 at 8:18 PM, Samuel Thibault > > > wrote: > > > > Samuel Thibault, on mar. 14 mars 2017 01:47:01 +0100, wrote: > > > >> Greg KH, on mar. 14 mars 2017 06:14:04 +0800, wrote: > > > >> > On Mon, Mar 13, 2017 at 10:05:51PM +0000, okash.khawaja@gmail.com wrote: > > > >> > > This patchset introduces a TTY-based way for the synths to communicate > > > >> > > with devices as an alternate for direct serial comms used by the synths > > > >> > > at the moment. It then migrates some of the synths to the TTY-based > > > >> > > comms. Synths migrated in this patchset are dummy, acntsa, bns and > > > >> > > txprt. > > > >> > > > > >> > What about using the serbus code that is now in the tree? That should > > > >> > make this a lot easier than your patchset from what I can see. > > > >> > > > >> Mmm... AIUI from reading tty_port_register_device_attr, one > > > >> would have to have registered a speakup serdev device driver > > > >> *before* tty_port_register_device_attr gets called, so that > > > >> serdev_tty_port_register matches the driver in the loop of > > > >> of_serdev_register_devices, and no TTY cdev is created? > > > > > > Not exactly. The driver doesn't have to be registered nor loaded, but > > > the device does have to be present so no tty cdev is created. The only > > > way a device is present currently is when a node is in the DT. I > > > expect the x86 folks will be adding ACPI support soon. > > > > Ok, but in our case there is no hardware device that the system can > > see/probe, it's just plugged externally. > > > > > >> That would mean that speakup can not be loaded as a module after ttyS0 > > > >> initialization, that won't fly for our use needs. The line discipline > > > >> mechanism allows us to attach ourself to an existing tty. Could we > > > >> imagine a tty_port function which removes the cdev and tries to register > > > >> the tty port again to serdev? > > > >> > > > >> What we basically need to be able to say on speakup module load is > > > >> e.g. "I'm now attaching a device to ttyS0, use this serdev_device_ops to > > > >> discuss with it". > > > > > > > > That for_each_available_child_of_node loop is really way more complex > > > > than what we need. And what's more, it's not working without CONFIG_OF > > > > (!) > > > > > > > > It would really make sense to me to have a > > > > > > > > serdev_device *tty_port_register_serdev_device(tty, device) > > > > > > > > which unregisters the character device of the tty, and creates instead a > > > > controler with the given device plugged to it. Really much like a line > > > > discipline, but way simpler :) > > > > > > What would trigger calling this function? > > > > From the user point of view, loading the speakup module for the external > > device, typically, with "ttyS0" as module parameter. Then the speakup > > init function can do whatever it needs to achieve this :)