All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tejun Heo <tj@kernel.org>
To: Grant Grundler <grundler@google.com>
Cc: Tim Nufire <linux_ide_tim@ibink.com>, linux-ide@vger.kernel.org
Subject: Re: Frozen drives when using SiI3726
Date: Fri, 02 Jan 2009 12:28:40 +0900	[thread overview]
Message-ID: <495D89E8.2020307@kernel.org> (raw)
In-Reply-To: <da824cf30812291120x1ed3f554lb7bdfe25427cecb3@mail.gmail.com>

Hello, Grant.

Grant Grundler wrote:
>> Also, the timeout is one of the most generic failure mode there is.
>> It can be triggered by virtually anything including transmission
>> failure, power quality issues, drive problems and whatnot.  So, it's
>> impossible to tell what went wrong with the provided information.
> 
> Tejun,
> The only other possible major issue not listed above I can think of is
> Sil3276 firmware rev. The data sheet (*) "EEPROM Speicifications"
> on Page 20 says the firmware is versioned.
> But the data sheet doesn't specify how to read it.

I haven't seen firmware related transmission failures yet but it's not
like I have lots of 3726s with different revision firmwares.

> Can you ask Silicon Image to publish how to read the firmware rev?

IIRC, you can read it from the steelvine management utility.  You'll
probably need a Windows installation tho.

> Also SATA_PMP_GSCR_REV  (Spec compliance, not FW rev) seems
> to be used twice:
> 
> include/linux/ata.h:
> #define sata_pmp_gscr_rev(gscr)         (((gscr)[SATA_PMP_GSCR_REV] >>
> 8) & 0xff)
> 
> drivers/ata/libata-pmp.c:sata_pmp_spec_rev_str()
> 
> Would it make sense to rename one or the other so they match?
> The difference is one returns a string and the other a u8 value.
> Both take the same parameter.

No, spec_rev_str() returns spec compliance string (lower bits) while
gscr_rev() returns revision level of port multiplier (whatever that
means).

Thanks.

-- 
tejun

      reply	other threads:[~2009-01-02  3:28 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-12-23 10:02 Frozen drives when using SiI3726 Tim Nufire
2008-12-29  9:13 ` Tejun Heo
2008-12-29 19:20   ` Grant Grundler
2009-01-02  3:28     ` Tejun Heo [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=495D89E8.2020307@kernel.org \
    --to=tj@kernel.org \
    --cc=grundler@google.com \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux_ide_tim@ibink.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.