> 
> Is the device hotpluggable?   If so, this:
> 
>       qdev->index = cards_found;
> 
> in the probe() handler might do odd things - it'll just keep 
> increasing as the card is removed and re-added.
> 
> iirc, that's a common problem with net drivers.  AFAICT it'll 
> cause only cosmetic oddities here.
> 
>

Not sure if the device supports hotplug.  I'm looking into it.  Either
way, I will be removing qdev->index.  It's not needed. 

I am working on the remaining items that Jeff pointed out when he
reviewed my driver.  I will forward a patch built against what you
currently have when I'm done.

Now that the driver is in mm tree, is there anything I need to do to get
it into Linus' tree?
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to