Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752256AbdFNMXX (ORCPT ); Wed, 14 Jun 2017 08:23:23 -0400 Received: from hera.aquilenet.fr ([141.255.128.1]:36456 "EHLO hera.aquilenet.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751719AbdFNMXW (ORCPT ); Wed, 14 Jun 2017 08:23:22 -0400 Date: Wed, 14 Jun 2017 14:18:19 +0200 From: Samuel Thibault To: Greg Kroah-Hartman Cc: okash.khawaja@gmail.com, Jiri Slaby , linux-kernel@vger.kernel.org, devel@driverdev.osuosl.org, Kirk Reiser , speakup@linux-speakup.org, Chris Brannon Subject: Re: [patch 1/2] staging: speakup: add function to convert dev name to number Message-ID: <20170614121819.tob426qpysswg5g6@var.youpi.perso.aquilenet.fr> Mail-Followup-To: Samuel Thibault , Greg Kroah-Hartman , okash.khawaja@gmail.com, Jiri Slaby , linux-kernel@vger.kernel.org, devel@driverdev.osuosl.org, Kirk Reiser , speakup@linux-speakup.org, Chris Brannon References: <20170613223702.534373529@gmail.com> <20170613224050.646277124@gmail.com> <20170614101326.GA7606@kroah.com> <20170614101541.GA8065@kroah.com> <20170614112609.uqpvl2z5aoi5jchc@var.youpi.perso.aquilenet.fr> <20170614114802.GA23345@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170614114802.GA23345@kroah.com> Organization: I am not organized User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 843 Lines: 22 Greg KH, on mer. 14 juin 2017 13:48:02 +0200, wrote: > On Wed, Jun 14, 2017 at 01:26:09PM +0200, Samuel Thibault wrote: > > Greg KH, on mer. 14 juin 2017 12:15:41 +0200, wrote: > > > Ah, no, nevermind, you just need the major/minor. So why not just take > > > that as the input here, and not a string? > > > > Because real users don't know about major/minor numbers. > > I'm not disagreeing, it's just that the kernel doesn't know about > "device names" either :) Well, it does for the console= parameter, for instance. I know it's not actually related with major/minor numbering, but it's still meant to designate the same thing. > And trying to have the kernel do the mapping based on strings like this > is not ok, sorry. So what is the solution? Users would find it completely crazy to have to provide major/minor numbers. Samuel