SD slot on 7-in-1 USB card reader not recognised as SD

Bug #61834 reported by Malcolm Parsons on 2006-09-22
2
Affects Status Importance Assigned to Milestone
hal (Ubuntu)
Low
Martin Pitt

Bug Description

There are 4 slots on my 7-in-1 USB card reader.
HAL calls them "USB Storage-CFC", "USB Storage-MSC", "USB Storage-SMC" and "USB Storage-SDC".
HAL has the correct storage.drive_type for the compact flash, memory stick and smart media slots, but the SD slot has a storage.drive_type of "disk".

In /usr/share/hal/fdi/information/10freedesktop/10-usb-card-readers.fdi I see why:
      <match key="storage.model" contains="Storage-CFC">
        <merge key="storage.drive_type" type="string">compact_flash</merge>
      </match>
      <match key="storage.model" contains="Storage-MSC">
        <merge key="storage.drive_type" type="string">memory_stick</merge>
      </match>
      <match key="storage.model" contains="Storage-SMC">
        <merge key="storage.drive_type" type="string">smart_media</merge>
      </match>
      <match key="storage.model" contains="Storage-MMC">
        <merge key="storage.drive_type" type="string">sd_mmc</merge>
      </match>

HAL expects "Storage-MMC", but I have "Storage-SDC".

So the fix is to add this:
      <match key="storage.model" contains="Storage-SDC">
        <merge key="storage.drive_type" type="string">sd_mmc</merge>
      </match>

Sebastien Bacher (seb128) wrote :

Thanks for your bug. Marking as confirmed

Changed in hal:
importance: Undecided → Low
status: Unconfirmed → Confirmed
Martin Pitt (pitti) wrote :

Trivial fix, edgyable.

Changed in hal:
assignee: nobody → pitti
status: Confirmed → In Progress
Martin Pitt (pitti) on 2006-10-09
Changed in hal:
status: In Progress → Fix Committed
Martin Pitt (pitti) wrote :

 hal (0.5.7.1-0ubuntu15) edgy; urgency=low
 .
   * Add debian/patches/00upstream-05-fix-stale_nfs_handle_block.patch:
     - Do not stat NFS mounts when preparing for suspend to fix hang.
     - Patch taken from upstream git.
     - Closes: LP#64266
   * Add debian/patches/20_laptop-panel-fixes.patch: ThinkPad T60 controls
     screen brightness in hardware, so set the appropriate flag in
     10-laptop-panel-mgmt-policy.fdi. Closes: LP#61184
   * Add debian/patches/21_smartcard_drivetype.patch: Recognize 'Storage-SDC'
     as smart card drive type. Closes: LP#61834
   * Add debian/patches/22-prefer-pmi-over-powersave.patch: Prefer pmi over
     powersave in the suspend/hibernate scripts. Closes: LP#61920
   * debian/patches/17-mp3-player-fdi.patch: Add information about Sandisk
     Corp. Sansa e270. Closes: LP#64300
   * debian/patches/02_ignored_volumes.patch: Ignore Apple bootstrap partitions
     on removable devices, too. Closes: LP#61767
   * Add debian/patches/00upstream-06-stat-dev-not-mountpoint.patch:
     - hald/linux/blockdev.c, blockdev_refresh_mount_state(): Stat the special
       device file, not the mount point's underlying device. This makes hal
       correctly recognize fuse mounts.
     - Patch ported from upstream GIT, thanks to Florent Mertens!
       http://gitweb.freedesktop.org/?p=hal.git;a=commit;h=2ea340399bf8cf3d2bb6bd1b5c4ecbc2042e93d4
     - Closes: LP#35354

Changed in hal:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers