From mboxrd@z Thu Jan 1 00:00:00 1970 From: Segher Boessenkool Subject: Re: [PATCH] powerpc/fsl: add device tree binding for QE firmware Date: Wed, 24 Mar 2010 18:00:18 +0100 Message-ID: <90D93687-940F-47FB-8CEA-F3C065EA611D@kernel.crashing.org> References: <1269380552-10418-1-git-send-email-timur@freescale.com> Mime-Version: 1.0 (Apple Message framework v753.1) Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org To: Timur Tabi Cc: linuxppc-dev-mnsaURCQ41sdnm+yROfE0A@public.gmane.org, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org List-Id: devicetree@vger.kernel.org >> Why the phandle redirection? Why not just put the firmware blob into >> a property in the QE node, or as a subnode? > > Because there might be multiple QE devices on a single chip, and each > will need to upload the same firmware. So instead of embedding the > firmware multiple times, just embed it once, and have a pointer. You're messing up the binding because of a (perceived) deficiency in the DTB format? Or maybe just the DTS format. Or maybe you shouldn't even care about size here. Or really, the device tree is the wrong place to store firmware blobs at all. Segher