I have updated the patch according to feedbacks I received. The main 
modification was to warn the user and ask for confirmation before downloading a 
firmware image. A -y option has been added that suppresses the confirmation-- 
the same as the one for the format command.

The new patch is against HEAD and could be downloaded from:
http://people.freebsd.org/~emaste/patches/fwdownload.diff


Reviews/suggestions are greatly appreciated.


Nima Misaghian 
[email protected]


> -----Original Message-----
> From: [email protected] [mailto:[email protected]] On Behalf Of Craig
> Rodrigues
> Sent: Thursday, November 03, 2011 10:06 PM
> To: Nima Misaghian
> Cc: [email protected]
> Subject: Re: Adding disk firmware programming capability to camcontrol
> 
> On Fri, Oct 28, 2011 at 1:47 PM, Nima Misaghian
> <[email protected]> wrote:
> > Hi,
> >
> > I have got code developed by Andre Albsmeier that is capable of
> > programming firmware of hard drives from several vendors and  turned
> > it into a camcontrol command.
> 
> +1
> 
> I took a look at your patch and it looks great.  I have worked on a
> storage product before where there
> was a requirement to reprogram the firmware of hard drives.  On tha
> product,
> proprietary Windows tools had to be used.  It would have been nice to
> be able to use camcontrol in FreeBSD.
> 
> I think the patch is fine in its current form.  Very few "regular
> users" need to reprogram hard drive firmware.
> It is a special administrative operation.
> 
> --
> Craig Rodrigues
> [email protected]

Attachment: fwdownload.diff
Description: fwdownload.diff

_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "[email protected]"

Reply via email to