lkml.org 
[lkml]   [2009]   [Feb]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[patch 20/56] firewire: sbp2: add workarounds for 2nd and 3rd generation iPods
    2.6.27-stable review patch.  If anyone has any objections, please let us know.
    ------------------

    From: Stefan Richter <stefanr@s5r6.in-berlin.de>

    Commit c8c4707cf7ca8ff7dcc1653447e48cb3de0bf114 upstream.

    According to https://bugs.launchpad.net/bugs/294391
    - 3rd generation iPods need the "fix capacity" workaround after all
    (apparently they crash after the last sector was accessed),
    - 2nd generation iPods need the "128 kB maximum request size"
    workaround.

    Alas both iPod generations feature the same model ID in the config ROM,
    hence we can only define a shared quirks list entry for them. Luckily
    the fix capacity workaround did not show a negative effect in Jarod's
    tests with 2nd gen. iPod.

    A side note: Apple computers in target mode (or at least an x86 Mac
    mini) don't have firmware_version and model_id, hence none of the iPod
    quirks list entries is active for them.

    Tested-by: Jarod Wilson <jarod@redhat.com>
    Acked-by: Jarod Wilson <jarod@redhat.com>
    Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
    Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

    ---
    drivers/firewire/fw-sbp2.c | 16 +++++++++-------
    1 file changed, 9 insertions(+), 7 deletions(-)

    --- a/drivers/firewire/fw-sbp2.c
    +++ b/drivers/firewire/fw-sbp2.c
    @@ -350,15 +350,17 @@ static const struct {
    .model = ~0,
    .workarounds = SBP2_WORKAROUND_128K_MAX_TRANS,
    },
    -
    /*
    - * There are iPods (2nd gen, 3rd gen) with model_id == 0, but
    - * these iPods do not feature the read_capacity bug according
    - * to one report. Read_capacity behaviour as well as model_id
    - * could change due to Apple-supplied firmware updates though.
    + * iPod 2nd generation: needs 128k max transfer size workaround
    + * iPod 3rd generation: needs fix capacity workaround
    */
    -
    - /* iPod 4th generation. */ {
    + {
    + .firmware_revision = 0x0a2700,
    + .model = 0x000000,
    + .workarounds = SBP2_WORKAROUND_128K_MAX_TRANS |
    + SBP2_WORKAROUND_FIX_CAPACITY,
    + },
    + /* iPod 4th generation */ {
    .firmware_revision = 0x0a2700,
    .model = 0x000021,
    .workarounds = SBP2_WORKAROUND_FIX_CAPACITY,


    \
     
     \ /
      Last update: 2009-02-11 01:37    [W:3.434 / U:0.060 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site