From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Bolle Subject: Re: [PATCH 3/4] tty: allow tty drivers to rename their device nodes Date: Wed, 28 May 2014 23:06:41 +0200 Message-ID: <1401311201.6186.22.camel@x220> References: <243519fa733359d22ce008b795f5686411e6038a.1400449372.git.tilman@imap.cc> <20140528205656.GA3117@kroah.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: Tilman Schmidt , netdev@vger.kernel.org, isdn4linux@listserv.isdn4linux.de, "Keil, Karsten" , Jiri Slaby To: Greg Kroah-Hartman Return-path: Received: from cpsmtpb-ews05.kpnxchange.com ([213.75.39.8]:64723 "EHLO cpsmtpb-ews05.kpnxchange.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932303AbaE1VGm (ORCPT ); Wed, 28 May 2014 17:06:42 -0400 In-Reply-To: <20140528205656.GA3117@kroah.com> Sender: netdev-owner@vger.kernel.org List-ID: On Wed, 2014-05-28 at 13:56 -0700, Greg Kroah-Hartman wrote: > On Wed, May 21, 2014 at 11:39:26PM +0200, Tilman Schmidt wrote: > > From: Paul Bolle >> > > The device nodes for tty drivers are named using a straightforward > > scheme: tty_driver->name with an (increasing) digit appended. But the > > capi driver (a part of one of the current ISDN subsystems) requires a > > different naming scheme for its "capi_nc" tty_driver: > > /dev/capi/0 > > /dev/capi/1 > > [...] > > Can't you just use a '!' character to represent the '/' and the tty core > will handle it all properly for you without this tty core change needed? As in: set struct tty_driver.name to "capi!"? > > So add a devnode() callback to struct tty_driver to allow tty drivers > > to use a more elaborate naming scheme. And let tty_devnode(), the > > devnode() callback for the "tty" class, call that new callback if a tty > > driver uses one. This allows the capi driver to add a callback to > > enable its scheme. > > And why the sudden need for this feature, what changed in isdn to > warrant this change? Did you already read the explanation to 4/4? It contains a summary of the events that got us in the current situation. Paul Bolle