On Jan 30, 2018 3:17 AM, "Alexander Graf" <ag...@suse.de> wrote:
On 01/30/2018 12:41 AM, Derald D. Woods wrote: > On Mon, Jan 29, 2018 at 07:46:09AM -0600, Derald Woods wrote: > >> On Jan 29, 2018 6:57 AM, "Alexander Graf" <ag...@suse.de> wrote: >> >> Commit 608b0c4ad4e5ec0c ("serial: Use next serial device if probing >> fails") >> added code to search for more serial devices if the default one was not >> probed correctly. >> >> Unfortunately, that breaks omap3_evm. So while investigating why that is >> the case, let's disable the full search for everyone but bcm283x where it >> is needed. >> >> Fixes: 608b0c4ad4e5ec0c ("serial: Use next serial device if probing >> fails") >> Reported-by: Derald D. Woods <woods.techni...@gmail.com> >> Signed-off-by: Alexander Graf <ag...@suse.de> >> >> --- >> >> Derald, could you please test this patch and verify it does indeed unbreak >> omap3_evm? >> >> The omap3_evm boots now with this patch applied on master. If I enable > SERIAL_SEARCH_ALL, it does not boot. I always build cleanly using the > default config only. On failure, there is no console input/output and > the board unresponsive. > So SPL is already broken? Can you try a known working SPL with SERIAL_SEARCH_ALL=y U-Boot payload on top? Does that work? I will give that path a try and see what I can discover. Again, it will be later today or tomorrow before I can get to this. This is why I asked what should the board code actually look like. As the omap3_evm is ahead of some other OMAP34XX boards currently, a good working example would be helpful. If omap3_evm becomes the example, let's make it a good one. Derald _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot