From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lokesh Vutla Date: Tue, 14 Mar 2017 16:22:45 +0530 Subject: [U-Boot] [PATCH 02/17] SPL: FIT: refactor FDT loading In-Reply-To: <1488335129-12363-3-git-send-email-andre.przywara@arm.com> References: <1488335129-12363-1-git-send-email-andre.przywara@arm.com> <1488335129-12363-3-git-send-email-andre.przywara@arm.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wednesday 01 March 2017 07:55 AM, Andre Przywara wrote: > Currently the SPL FIT loader uses the spl_fit_select_fdt() function to > find the offset to the right DTB within the FIT image. > For this it iterates over all subnodes of the /configuration node in > the FIT tree and compares all "description" strings therein using a > board specific matching function. > If that finds a match, it uses the string in the "fdt" property of that > subnode to locate the matching subnode in the /images node, which points > to the DTB data. > Now this works very well, but is quite specific to cover this particular > use case. To open up the door for a more generic usage, let's split this > function into: > 1) a function that just returns the node offset for the matching > configuration node (spl_fit_find_config_node()) > 2) a function that returns the image data any given property in a given > configuration node points to, additionally using a given index into > a possbile list of strings (spl_fit_select_index()) > This allows us to replace the specific function above by asking for the > image the _first string of the "fdt" property_ in the matching > configuration subnode points to. > > This patch introduces no functional changes, it just refactors the code > to allow reusing it later. > > (diff is overly clever here and produces a hard-to-read patch, so I > recommend to throw a look at the result instead). > > Signed-off-by: Andre Przywara Reviewed-by: Lokesh Vutla Thanks and regards, Lokesh