From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Warren Date: Wed, 22 Jul 2015 12:04:37 -0600 Subject: [U-Boot] [PATCH V2 6/6] T210: Add support for 64-bit T210-based P2571 board In-Reply-To: <1437421844-5157-7-git-send-email-twarren@nvidia.com> References: <1437421844-5157-1-git-send-email-twarren@nvidia.com> <1437421844-5157-7-git-send-email-twarren@nvidia.com> Message-ID: <55AFDB35.8080901@wwwdotorg.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 07/20/2015 01:50 PM, Tom Warren wrote: > Based on Venice2, incorporates Stephen Warren's > latest P2571 pinmux table. > > With Thierry Reding's 64-bit build fixes, this > will build and and boot in 64-bit on my P2571 > (when used with a 32-bit AVP loader). > diff --git a/include/configs/venice2.h b/include/configs/p2571.h > /* USB Host support */ > #define CONFIG_USB_EHCI > #define CONFIG_USB_EHCI_TEGRA > -#define CONFIG_USB_MAX_CONTROLLER_COUNT 2 > +#define CONFIG_USB_MAX_CONTROLLER_COUNT 1 Why's that? > -/* USB networking support */ > -#define CONFIG_USB_HOST_ETHER > -#define CONFIG_USB_ETHER_ASIX > - > -/* General networking support */ > -#define CONFIG_CMD_DHCP I assume that's to solve some compile issue? If so, a FIXME/TODO comment (like you added for tegra-common-usb-gadget.h) would be better, so it's obvious we need to go back and re-enable it. > +#if defined(CONFIG_ARM64) > +#define COUNTER_FREQUENCY 12000000 > +#define CPU_RELEASE_ADDR 0x80000000 > +#endif CONFIG_ARM64 is always true now. According to the schematics, the crystal frequency is 38.4MHz. Where did the value of CPU_RELEASE_ADDR come from? It's odd that there are 4 CPUs but only 1 release address. > diff --git a/include/configs/tegra124-common.h b/include/configs/tegra210-common.h > -#define CONFIG_SYS_TEXT_BASE 0x80110000 > +#define CONFIG_SYS_TEXT_BASE 0x8010E000 It'd be best to keep that consistent with earlier chips.