From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerry Van Baren Date: Tue, 05 Apr 2011 10:24:44 -0400 Subject: [U-Boot] Please pull u-boot-mpc85xx.git In-Reply-To: <20110404194227.BF932151F83@gemini.denx.de> References: <16CF6CAD-477C-4E04-93E0-1DADCF05C724@kernel.crashing.org> <20110404194227.BF932151F83@gemini.denx.de> Message-ID: <4D9B262C.1070101@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 04/04/2011 03:42 PM, Wolfgang Denk wrote: > Dear Kumar Gala, > > In message<16CF6CAD-477C-4E04-93E0-1DADCF05C724@kernel.crashing.org> you wrote: [snip] > Timur: does this fix any real problem? > >>> powerpc/85xx: introduce 'fdt verify' command >> >> I'm wondering if its better to just set the dtb props instead of verifying. > > Unfortunately Timur's posting broke the mail thread because he failed > to add keep reference headers in his patch postings. I'm not happy > with this implementation either. > > In any case this needs the ACK from the responsible custodian (FDT => > gvb). I don't see any problem with the patch other than a minor typo On the last line, s/patch/match./ >> +/* >> + * Verify the addresses for all of the PCI controllers >> + * >> + * PCI is complicated because there is no correlation between the numbering >> + * of the controllers by U-Boot and the numbering the device tree. So we need >> + * to search all of the aliases until we find a patch >> + */ It appears to provide a valuable service. Acked-by: Gerald Van Baren > Best regards, > > Wolfgang Denk >