Subject: Re: [PATCH 00/18] firmware: moving drivers to request_firmware()
From: David Woodhouse <dwmw2@infradead.org>
To: Jeff Garzik <jeff@garzik.org>
Cc: linux-kernel@vger.kernel.org
In-Reply-To: <4848489C.6010903@garzik.org>
References: <20080605.foo@pmac.infradead.org>  <48483BA9.1070200@garzik.org>
	 <1212695675.32207.288.camel@pmac.infradead.org>
	 <4848489C.6010903@garzik.org>
Content-Type: text/plain
Date: Thu, 05 Jun 2008 21:53:10 +0100
Message-Id: <1212699190.32207.293.camel@pmac.infradead.org>
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 978
Lines: 25

On Thu, 2008-06-05 at 16:12 -0400, Jeff Garzik wrote:
> Well, I should ask, is this purely an internal build detail?
>
> The developer (and user) should never ever see a .ihex file, outside of 
> an active kernel compile.  Wrapping the original firmware makes it more 
> difficult to verify, compare and/or change.  In-tree, we should see the 
> vendor firmware blobs as shipped, with no wrapping or modification or 
> anything.

It's a build-and-shipping detail, at least in the kernel source tree.

Rather than putting binary blobs into git (which admittedly we could,
but it's suboptimal), we just run them through 'objcopy -Oihex' first.

It's not 'wrapped'; it's not any more difficult to verify, compare or
change.

-- 
dwmw2

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/