From owner-svn-src-head@FreeBSD.ORG Mon Sep 2 17:10:40 2013 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id 589E9873; Mon, 2 Sep 2013 17:10:40 +0000 (UTC) (envelope-from rpaulo@FreeBSD.org) Received: from felyko.com (felyko.com [IPv6:2607:f2f8:a528::3:1337:ca7]) by mx1.freebsd.org (Postfix) with ESMTP id 41F92230F; Mon, 2 Sep 2013 17:10:40 +0000 (UTC) Received: from [IPv6:2601:9:4d00:119:fc2d:d8a4:158d:d65] (unknown [IPv6:2601:9:4d00:119:fc2d:d8a4:158d:d65]) (using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by felyko.com (Postfix) with ESMTPSA id 5213339821; Mon, 2 Sep 2013 10:10:39 -0700 (PDT) Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 6.5 \(1508\)) Subject: Re: svn commit: r255130 - in head/sys: arm/conf arm/freescale/imx boot/fdt/dts From: Rui Paulo In-Reply-To: <20130902112652.870eff18955c9b28fee978df@ddteam.net> Date: Mon, 2 Sep 2013 10:10:39 -0700 Content-Transfer-Encoding: quoted-printable Message-Id: References: <201309012015.r81KFaa9056648@svn.freebsd.org> <20130902112652.870eff18955c9b28fee978df@ddteam.net> To: Aleksandr Rybalko X-Mailer: Apple Mail (2.1508) Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 02 Sep 2013 17:10:40 -0000 On 2 Sep 2013, at 01:26, Aleksandr Rybalko wrote: > Hello Rui! >=20 > On Sun, 1 Sep 2013 20:15:36 +0000 (UTC) > Rui Paulo wrote: >=20 >> Author: rpaulo >> Date: Sun Sep 1 20:15:35 2013 >> New Revision: 255130 >> URL: http://svnweb.freebsd.org/changeset/base/255130 >>=20 >> Log: >> Initial support for the Digi ConnectCore(c) i.MX53 / Wi-i.MX53 = boards. >>=20 >> There are many drivers missing, but we can reach single user mode = now. >>=20 >> Hardware graciously donated by Douglas Beattie. >>=20 >> Added: >> head/sys/arm/conf/DIGI-CCWMX53 (contents, props changed) >> head/sys/arm/freescale/imx/files.imx53 (contents, props changed) >> head/sys/arm/freescale/imx/imx53_machdep.c (contents, props = changed) >> head/sys/arm/freescale/imx/std.imx53 (contents, props changed) >> head/sys/boot/fdt/dts/digi-ccwmx53.dts (contents, props changed) >> head/sys/boot/fdt/dts/imx53x.dtsi (contents, props changed) >> Modified: >> head/sys/arm/freescale/imx/imx51_ccm.c >> head/sys/arm/freescale/imx/imx51_gpio.c >> head/sys/arm/freescale/imx/imx51_iomux.c >> head/sys/arm/freescale/imx/imx51_machdep.c >> head/sys/arm/freescale/imx/imx_gpt.c >> head/sys/arm/freescale/imx/imx_wdog.c >>=20 >=20 > [[cut]] >=20 >> Modified: head/sys/arm/freescale/imx/imx51_machdep.c >> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >> --- head/sys/arm/freescale/imx/imx51_machdep.c Sun Sep 1 = 19:59:54 2013 (r255129) >> +++ head/sys/arm/freescale/imx/imx51_machdep.c Sun Sep 1 = 20:15:35 2013 (r255130) >> @@ -107,7 +107,7 @@ platform_devmap_init(void) >> * Map segment where UART1 and UART2 located. >> */ >> fdt_devmap[0].pd_va =3D IMX51_DEV_VIRT_BASE + 0x03f00000; >> - fdt_devmap[0].pd_pa =3D 0x73f00000; >> + fdt_devmap[0].pd_pa =3D 0x53f00000; >=20 > I hope it is just mistake, is it? Yes, fixed. >=20 >> fdt_devmap[0].pd_size =3D 0x00100000; >> fdt_devmap[0].pd_prot =3D VM_PROT_READ | VM_PROT_WRITE; >> fdt_devmap[0].pd_cache =3D PTE_NOCACHE; >>=20 >=20 > [[cut]] >=20 > And, do you board have own FDT blob? > If it so, then better to try to use it. Otherwise, it is better to > unify compatible property values, instead of do test for growing list. The FDT blob is not relevant to this since console.c is not using any = FDT functions. This is just a debugging aid for the platform bring up = until I get uart_dev_imx.c working correctly. That said, eventually this will go away. The i.MX51 port you did has the = same problem, anyway. -- Rui Paulo