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

Leann Ogasawara leann.ogasawara at canonical.com
Thu Apr 8 19:02:31 UTC 2010


On Wed, 2010-04-07 at 15:59 +0300, 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

Missing "BugLink:" prefix.

> 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

We probably don't need the bug link listed twice in the commit. And this
url is invalid anyways.

> Signed-off-by: Surbhi Palande <surbhi.palande at canonical.com>

Has this been submitted upstream as well?  Seems this should be
something that would also qualify for stable.

Thanks,
Leann

> ---
>  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> */
> +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",
> -- 
> 1.6.3.3
> 
> 






More information about the kernel-team mailing list