[PATCH][Upstream] USB: Add ids of SamsungYP-M1 to restrict max sector

Stefan Bader stefan.bader at canonical.com
Mon Apr 12 14:11:05 UTC 2010


Surbhi Palande wrote:
> This quirk fixes bug 503790 on launchpad where a user complained of having to
> manually set the max sectors for a USB thumb drive to make it work. This small
> quirk sets the sector size explicitly in the kernel code. This patch is tested
> by the reporter of the bug. Do consider merging this for Lucid.
> 
> From 13a918b7c5917a702f14eaa938f10194417482ba Mon Sep 17 00:00:00 2001
> From: Surbhi Palande <surbhi.palande at canonical.com>
> Date: Fri, 19 Mar 2010 15:58:29 +0200
> Subject: [PATCH] USB: Add ids of SamsungYP-M1 to restrict max sectors
> 
> http://launchpad.net/bugs/503790
> 
> The Samsung YP-M1 does not work unless the following is done manually "echo
> 128 > /sys/block/sdb/device/max_sectors" as root. This quirk fixes having to
> do this manually.
> 
> BugLink: http://launchpad.net/bugs/lp503790
> 
> Signed-off-by: Surbhi Palande <surbhi.palande at canonical.com>
> ---
>  drivers/usb/storage/unusual_devs.h |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/usb/storage/unusual_devs.h b/drivers/usb/storage/unusual_devs.h
> index f6ab789..6469673 100644
> --- a/drivers/usb/storage/unusual_devs.h
> +++ b/drivers/usb/storage/unusual_devs.h
> @@ -133,6 +133,13 @@ UNUSUAL_DEV(  0x0419, 0xaace, 0x0100, 0x0100,
>  		US_SC_DEVICE, US_PR_DEVICE, NULL,
>  		US_FL_IGNORE_RESIDUE ),
>  

> +/* Reported by Surbhi Palande <surbhi.palande at canonical.com> */

In addition to what Leann said (as you tag the patch upstream, I assume you have
sent it upstream, right? Have you checked to apply it to 2.6.32.y and 2.6.33.y
and if good, added cc: stable at kernel.org?), I also would not put your name here
but José's name as he reported the problem.

> +UNUSUAL_DEV(  0x04e8, 0x5123, 0x0000, 0x0000,
> +		"Samsung",
> +		"YP-M1",
> +		US_SC_DEVICE, US_PR_DEVICE, NULL,
> +		US_FL_MAX_SECTORS_64),
> +
>  /* Reported by Christian Leber <christian at leber.de> */
>  UNUSUAL_DEV(  0x0419, 0xaaf5, 0x0100, 0x0100,
>  		"TrekStor",






More information about the kernel-team mailing list