[Fwd: Re: kue firmware revision code]

This is a multi-part message in MIME format.
--------------040601070500050900060105
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

Well, here's Bill Paul's response to whether adjusting my other if_kue.c 
patch is valid... Basicallly, "if it works for you,..."


-scooter

--------------040601070500050900060105
Content-Type: message/rfc822;
 name="Re: kue firmware revision code"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline;
 filename="Re: kue firmware revision code"

Received: from mhultra.aero.org (mhultra.aero.org [130.221.88.102])
	by rushe.aero.org (8.11.7p1+Sun/8.11.7) with ESMTP id j1FKaQk11227
	for <[e-mail]>; Tue, 15 Feb 2005 12:36:26 -0800 (PST)
Received: from mail.aero.org (mail.aero.org [130.221.16.2]) by mhultra.aero.org with ESMTP for [e-mail]; Tue, 15 Feb 2005 12:36:25 -0800
Received: by mail.aero.org (Postfix)
	id 0D2D73B203; Tue, 15 Feb 2005 12:36:25 -0800 (PST)
Delivered-To: [e-mail]
Received: from mail.wrs.com (unknown-1-11.wrs.com [147.11.1.11])
	by mail.aero.org (Postfix) with ESMTP id CB0423B202
	for <[e-mail]>; Tue, 15 Feb 2005 12:36:24 -0800 (PST)
Received: from huisne.wrs.com (huisne [147.11.46.60])
	by mail.wrs.com (8.9.3/8.9.1) with ESMTP id MAA07229
	for <[e-mail]>; Tue, 15 Feb 2005 12:36:23 -0800 (PST)
From: william paul <[e-mail]>
Received: (from wpaul@localhost)
	by huisne.wrs.com (8.9.1/8.9.0) id MAA17195
	for [e-mail]; Tue, 15 Feb 2005 12:36:23 -0800 (PST)
Message-Id: <[e-mail]>
Subject: Re: kue firmware revision code
To: [e-mail]
Date: Tue, 15 Feb 2005 12:36:23 -0800 (PST)
In-Reply-To: <[e-mail]> from "Scott Michel" at Feb 15, 2005 11:43:32 AM
X-Mailer: ELM [version 2.5 PL2]
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit

Of all the gin joints in all the towns in all the world, Scott Michel
had to walk into mine and say:

> > No no, you've got it backwards. You try changing the code to test for
> > rev > 0x0202 and then you tell _me_ if it's safe. :)
> 
> I did and it **seems** to be OK, but I was asked to ask you for a final 
> blessing. :-)

Most of the time, the only way I know something works right is when
I try it and it doesn't explode.

> > Actually, I don't know. I assume it should work ok. I don't understand
> > why you're getting a revision code of 0x0208 in the first place, since
> > I thought it was generated by the firmware itself, and since I only have
> > the one firmware image (which comes with the driver), I never expected it
> > to change.
> > 
> > Just try it and tell me what happens.
> 
> I'm attempting to grok the way that driver works, in particular, what 
> the usb descriptor really points to relative to the firmware image and 
> the device. If the firmware rev is inside the image *and* it shows up in 
> the USB descriptor, then !BAM!, it's an easy fix.
> 
> So far, I haven't found a 0x02, 0x02 pattern in the image, but I did 
> find a 0x08, 0x02 pattern. So I'm punting a bit based on what I've observed.
> 
> 
> -scooter

The revision code is a major and minor number. It could be the minor
number comes from the NIC rather than the firmware. I really have no
idea what format the firmware image is in: KLSI just gave it to me
pretty much as-is, with a bit of sample code to show how to download
it to the device.

I don't know why the revision code is different for you. When I load
that firmware image into my sample KLSI device, I get 0x0202. The
firmware image is the same one that's been in the tree since the driver
was written. It hasn't been updated in years. So if the revision
code returned after the firmware is loaded really does come from
inside the firmware, it should be the same.

It's possible the revision code is indeed derived at least partly
from the chip, and they've just bumped the rev a bit recently.

-Bill

-- 
=============================================================================
-Bill Paul            [phone] | Senior Engineer, Master of Unix-Fu
                 [e-mail] | Wind River Systems
=============================================================================
              <adamw> you're just BEGGING to face the moose
=============================================================================

--------------040601070500050900060105--