Samsung YP-M1 media player missing

Bug #573219 reported by luc
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
media-player-info
Fix Released
Medium
media-player-info (Ubuntu)
Fix Released
Undecided
Martin Pitt

Bug Description

Binary package hint: media-player-info

It seems information for the Samsung YP-M1 player is not available in Karmic (and probably Lucid too).
The device is really close to the YP-P3 (it is basically its successor, only named M1 instead of P4). The supported files are the same and the files layout are the same.
From "udevadm info" the idProduct appears to be 5123 so all entries in the mpi files could be the same as the entries from samsung-yepp_yp-p3.mpi file except for the DeviceMatch entry which should probably be changed to:

DeviceMatch=usb:04e8:5123;

I did have some problems putting long files on the device, it hanged indefinitely. I had similar problems with a P3 which ultimately was transformed into a brick ... Following advices from http://forums.cnet.com/5208-13973_102-0.html?threadID=366598 thread, I manually set max_sectors to 64 (using echo 64 > /sys/block/sdc/device/max_sectors) and everything went well. So I think some options should be put for these Samsung devices in /lib/udev/rules.d/90-usb-media-players.rules to have this setting put automatically.

Related branches

Revision history for this message
In , Jonathan Ernst (jonathan.ernst) wrote :

This mpi adds the Samsung YP-M1 and generates the necessary udev rules
to make it recognized as a media player.

Revision history for this message
In , Jonathan Ernst (jonathan.ernst) wrote :

Created an attachment (id=38787)
samsung-yepp_yp-m1.mpi

Revision history for this message
Jonathan Ernst (jonathan.ernst) wrote :

Can you please try the following and let me know if it makes your player recognized without an .is_audio_player file ?

1) put the attached file (samsung-yepp_yp-m1.mpi) in /usr/share/media-player-info/
2) ALT+F2 and run : gksudo gedit /lib/udev/rules.d/40-usb-media-players.rules
3) paste the following two lines just before the last line (last line should be LABEL="media_player_end") :

# Yepp YP-M1
ATTRS{idVendor}=="04e8" , ATTRS{idProduct}=="5123" , ENV{ID_MEDIA_PLAYER}="samsung-yepp_yp-m1"

If it works, I'll send the required file upstream so your player is automaticaly recognized in the future.

Revision history for this message
Jonathan Ernst (jonathan.ernst) wrote :
Revision history for this message
Jonathan Ernst (jonathan.ernst) wrote :

(step 1) must be done as root, if you are not comfortable with command line, do ALT+F2 and run : gksudo nautilus then navigate to the /usr/share/media-player-info/ directory and put the file there)

Revision history for this message
luc (luc-maisonobe) wrote :

It does work, thanks.

The value for max_sectors is still 240 however (which seems normal as your patch doesn't set anything related to this). Is there a way we could automatically set this value to 64 using the udev rules ?

Revision history for this message
Jonathan Ernst (jonathan.ernst) wrote :

Thanks for your test.

For the max_secors issue you could add something like this to the udev rule I guess :

RUN+="/bin/sh -c '/bin/echo 64 > /sys/block/%k/device/max_sectors'"

However I don't know/think it can be set from the mpi file.

I will send my .mpi file to freedesktop.org and I suggest you open a new bug for the max_sector problem. If my suggestion above works, maybe add this information to the new bug and comment here to let us know the new bug number.

Revision history for this message
luc (luc-maisonobe) wrote :

Thank you for your advice. It works perfectly and sets the max_sector value for both the main memory and the SD card.
I have opended a separate issue as you suggested. This new issue is bug #642176.

Thanks for all.

Revision history for this message
In , Martin Pitt (pitti) wrote :
Revision history for this message
Martin Pitt (pitti) wrote :
Changed in media-player-info (Ubuntu):
assignee: nobody → Martin Pitt (pitti)
status: New → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package media-player-info - 9-1

---------------
media-player-info (9-1) unstable; urgency=low

  * New upstream release:
    - New players: Samsung YP-M1 (LP: #573219), iRiver E30 (LP: #601718),
      Motorola Dext/Cliq (LP: #642944), Motorola Droid debugging ID
      (LP: #642014)
    - Fix IDs for Sony Ericsson Xperia X10 (LP: #616910)
    - Fix IDs for video iPods (LP: #576738)
  * Add debian/watch.
 -- Martin Pitt <email address hidden> Mon, 20 Sep 2010 15:01:28 +0200

Changed in media-player-info (Ubuntu):
status: Fix Committed → Fix Released
Changed in media-player-info:
importance: Unknown → Medium
status: Unknown → Fix Released
Changed in media-player-info:
importance: Medium → Unknown
Changed in media-player-info:
importance: Unknown → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.