lkml.org 
[lkml]   [2002]   [Jan]   [8]   [last100]   RSS Feed
Views: [more markup]   [less markup]   [headers]   [forward]  
 
Messages in this thread
/
DateTue, 8 Jan 2002 07:15:48 +0100
FromDavid Weinehall <>
SubjectRe: Serial Driver Name Question (kernels 2.4.x)
On Mon, Jan 07, 2002 at 09:03:35PM -0800, Andrew Morton wrote:
> [ tty driver name breakage ]
> 
> Richard, can we please get this wrapped up?
> 
> My preferred approach is to change the driver naming scheme
> so that we don't have to put printf control-strings everywhere.
> We can remove a number of ifdefs that way.

Wouldn't it be cleaner to have:

#ifdef CONFIG_DEVFS_FS
	serial_driver.name = "tts/";
#else
	serial_driver.name = "tts";
#endif

and

	sprintf("buf, "%s%d", name, idx + tty->driver.name_base);

respectively?!


/David

> So for serial.c:
> 
> --- linux-2.4.18-pre2/drivers/char/tty_io.c	Mon Jan  7 16:48:02 2002
> +++ linux-akpm/drivers/char/tty_io.c	Mon Jan  7 20:56:38 2002
> @@ -193,10 +193,13 @@ _tty_make_name(struct tty_struct *tty, c
> 
>  	if (!tty) /* Hmm.  NULL pointer.  That's fun. */
>  		strcpy(buf, "NULL tty");
> -	else
> -		sprintf(buf, name,
> -			idx + tty->driver.name_base);
> -		
> +	else {
> +#ifdef CONFIG_DEVFS_FS
> +		sprintf(buf, "%s/%d", name, idx + tty->driver.name_base);
> +#else
> +		sprintf(buf, "%s%d", name, idx + tty->driver.name_base);
> +#endif
> +	}		
>  	return buf;
>  }
> 
> --- linux-2.4.18-pre2/drivers/char/serial.c	Mon Jan  7 16:48:02 2002
> +++ linux-akpm/drivers/char/serial.c	Mon Jan  7 20:58:09 2002
> @@ -5387,7 +5387,7 @@ static int __init rs_init(void)
>  	serial_driver.driver_name = "serial";
>  #endif
>  #if (LINUX_VERSION_CODE > 0x2032D && defined(CONFIG_DEVFS_FS))
> -	serial_driver.name = "tts/%d";
> +	serial_driver.name = "tts";
>  #else
>  	serial_driver.name = "ttyS";
>  #endif

  _                                                                 _
 // David Weinehall <tao@acc.umu.se> /> Northern lights wander      \\
//  Maintainer of the v2.0 kernel   //  Dance across the winter sky //
\>  http://www.acc.umu.se/~tao/    </   Full colour fire           </
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/
\
 
 \ /
  Last update: 2005-03-22 11:15    [from the cache]
©2003-2008