[BusyBox 0001299]: mdev can't create character devices

[bugs at busybox.net]

The following issue has been CLOSED 
====================================================================== 
http://busybox.net/bugs/view.php?id=1299 
====================================================================== 
Reported By:                Souf
Assigned To:                BusyBox
====================================================================== 
Project:                    BusyBox
Issue ID:                   1299
Category:                   Other
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     closed
Resolution:                 unable to reproduce
Fixed in Version:           
====================================================================== 
Date Submitted:             04-02-2007 14:07 PDT
Last Modified:              06-13-2007 02:30 PDT
====================================================================== 
Summary:                    mdev can't create character devices
Description: 
I built an embedded system and I installed it on a mini PC (CPU: GeodeGX1),
, when I load the sound driver (this driver is under development):
modprobe snd-cs5530
mdev creates a block devices instead a character devices:

ls -l /dev/snd/
brw-rw-rw-    1 root     root     116,   5 Apr  2 22:29 controlC0
brw-rw-rw-    1 root     root     116,   4 Apr  2 22:29 pcmC0D0c
brw-rw-rw-    1 root     root     116,   3 Apr  2 22:29 pcmC0D0p
brw-rw-rw-    1 root     root     116,   2 Apr  2 22:29 timer

whereas the sound devices must be created as character devices.

I believe that the problem comes from this line:
type = (path[5]=='c') ? S_IFCHR : S_IFBLK;
which must be like this:
type = (path[5]=='b') ? S_IFBLK : S_IFCHR;

====================================================================== 

---------------------------------------------------------------------- 
 vapier - 06-13-07 02:30  
---------------------------------------------------------------------- 
seems to work fine for me:
$ ls -l dev/*pcm*
crw-rw---- 1 root root 116, 11 2007-06-13 05:19 dev/pcmC0D0c
crw-rw---- 1 root root 116, 10 2007-06-13 05:19 dev/pcmC0D0p
crw-rw---- 1 root root 116,  9 2007-06-13 05:19 dev/pcmC0D1c
crw-rw---- 1 root root 116,  8 2007-06-13 05:19 dev/pcmC0D2c
crw-rw---- 1 root root 116,  7 2007-06-13 05:19 dev/pcmC0D2p
crw-rw---- 1 root root 116,  6 2007-06-13 05:19 dev/pcmC0D3p 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
04-02-07 14:07  Souf           New Issue                                    
04-02-07 14:07  Souf           Status                   new => assigned     
04-02-07 14:07  Souf           Assigned To               => BusyBox         
06-13-07 02:30  vapier         Note Added: 0002480                          
06-13-07 02:30  vapier         Status                   assigned => closed  
06-13-07 02:30  vapier         Resolution               open => unable to
reproduce
======================================================================