From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Wed, 17 Oct 2012 09:12:50 -0700 Subject: [U-Boot] [U-Boot, V2, 1/9] disk: part_efi: remove indent level from loop In-Reply-To: <507ED6FD.8030206@wwwdotorg.org> References: <1349720080-13119-1-git-send-email-swarren@wwwdotorg.org> <20121017150130.GE5527@bill-the-cat> <507ED6FD.8030206@wwwdotorg.org> Message-ID: <507ED902.7040001@ti.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 10/17/12 09:04, Stephen Warren wrote: > On 10/17/2012 09:01 AM, Tom Rini wrote: >> On Mon, Oct 08, 2012 at 08:14:32AM -0000, Stephen Warren wrote: >> >>> From: Stephen Warren >>> >>> Simplify the partition printing loop in print_part_efi() to >>> bail out early when the first invalid partition is found, >>> rather than indenting the whole body of the loop. This >>> simplifies later patches. >>> >>> Signed-off-by: Stephen Warren >> >> This, along with the rest of the series have been applied to >> u-boot/master, thanks! Note that for 9/9 I had to rework it >> slightly to take the new output formatting, can you please >> double check? Thanks! > > Thanks. The results looks fine; there is zero diff between my > local branch and u-boot/master on the files that series touched. > > I wonder if the rework you needed to do was because patch 8/9 > wasn't applied? I did do some cleanup of the printf strings in that > patch separately, but there isn't an equivalent commit in your > tree; it's as if 8/9 and 9/9 were squashed into one. There we go, I just missed 8/9 (I see it in my patchwork TODO list now). > BTW, I had also posted a minor fix "disk: initialize name/part > fields when returning a whole disk" which may have been overlooked? > Perhaps it's still in your queue. That is still in my queue. I'm trying to leave a buffer around when patches are posted vs when I pull them in to allow for more review by the folks doing U-Boot in their spare time. I should pick up that as well as the fat_set_blk_dev series soon. - -- Tom -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://www.enigmail.net/ iQIcBAEBAgAGBQJQftkCAAoJENk4IS6UOR1WQ/0QAJYx2bZV00XlnlaTqJCI3OqO jiYuf30LI2isnzQz+eaxom844s0nHexnviNKio9HBtDaD6FAgL2CyH6TKPB5Lh0C kl4LgagcxX2Vs+ZpMElr07xcdiUFuRo/y527pCndikG43wYUtVZ+Piubuh1ASVDa W6ex4B3MAMCLAdFIfz0UhJHVCvaXMioC8i4lPCx/3cAWIISQmITg7B5/hVMYzR06 6/yu4hxn2wOw7hTEVN7UO05N4kcKmehtzA09Kr/UxedbtrcfyAYCVzzIc2ynryAd 8Y7t674+omkcV8/ZN04qBsFqfT79pQK33r5Hb9qeHrW0tpDZKCHqN6VKpM1CvlDI TX1ni2LQ7T6FM0fkENLy3dpVDYQfEHk9Zln6+ttIHz3nbsHArUDoM8rV8Lq9PTjk 766NfTcxPfm6mGFhGLRwbhZZTTLl/YrigFx+M6ECnpBFsdp9oKe6KfCphSEwh2xC aREBUr6+TIZZ8ynXIg5ahy4JMEe8x1z+qvs08AQC4oAFvNMi+zPPf6aGU7lCuz9p SpiJSFMDIznZZz6Q+/aZy18l03T9cbxV6mEffBEyGNOg/yiBcoymrdxVoKnFlbs/ vU5aF7a0mXZ1E9zsLbpvwcomdUwmYsrqpz7nLiS/lG4qdA5AQp4/xSGuLYynjoho HjoUE934eU4EaYKJRjhv =k4ym -----END PGP SIGNATURE-----