From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Mon, 22 Feb 2016 07:44:32 +0100 Subject: [U-Boot] ls1021aqds_nor_lpuart_ boards In-Reply-To: <56CA7E32.1080108@toradex.com> References: <56C9A017.6020809@denx.de> <56CA7E32.1080108@toradex.com> Message-ID: <56CAAE50.1010902@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi, On 22/02/2016 04:19, Bhuvanchandra DV wrote: > Hi Stefano, > > Adding Bin Meng. > > On 02/21/2016 05:01 PM, Stefano Babic wrote: >> Hi Bhuvanchandra, >> >> your applied patch: >> >> commit 5160def267739518950e8e3bb78e0e6c3941cb3e >> Author: Bhuvanchandra DV >> Date: Wed Jan 27 10:31:52 2016 +0530 >> >> dm: lpuart: Drop the legacy code >> >> braks ls1021aqds_*nor_lpuart. Tom has gently merged my PR, but this >> should be fixed. In fact, CONFIG_DM and CONFIG_DM_SERIAL are not set for >> these boards (setting them, build is fine): >> >> For example, for ls1021aqds_nor_lpuart: >> diff --git a/configs/ls1021aqds_nor_lpuart_defconfig >> b/configs/ls1021aqds_nor_lpuart_defconfig >> index 1186af2..d8d77c2 100644 >> --- a/configs/ls1021aqds_nor_lpuart_defconfig >> +++ b/configs/ls1021aqds_nor_lpuart_defconfig >> @@ -2,6 +2,8 @@ CONFIG_ARM=y >> CONFIG_TARGET_LS1021AQDS=y >> CONFIG_SYS_EXTRA_OPTIONS="LPUART" >> # CONFIG_CMD_SETEXPR is not set >> +CONFIG_DM=y >> +CONFIG_DM_SERIAL=y >> CONFIG_NETDEVICES=y >> CONFIG_E1000=y >> CONFIG_FSL_LPUART=y >> >> Can you provide a formal patch and resubmit ? I have no way to test the >> result on the board. > > I also don't have this h/w to test at my end, may be Bin can help here. Fabio has already added Alison who is board maintainer - the one could have the hw for test. Best regards, Stefano -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de =====================================================================