Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 2 Sep 2013 10:10:39 -0700
From:      Rui Paulo <rpaulo@FreeBSD.org>
To:        Aleksandr Rybalko <ray@ddteam.net>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r255130 - in head/sys: arm/conf arm/freescale/imx boot/fdt/dts
Message-ID:  <E275B036-8B04-4EFB-92CC-9C94C3D88F53@FreeBSD.org>
In-Reply-To: <20130902112652.870eff18955c9b28fee978df@ddteam.net>
References:  <201309012015.r81KFaa9056648@svn.freebsd.org> <20130902112652.870eff18955c9b28fee978df@ddteam.net>

next in thread | previous in thread | raw e-mail | index | archive | help
On 2 Sep 2013, at 01:26, Aleksandr Rybalko <ray@ddteam.net> wrote:

> Hello Rui!
>=20
> On Sun, 1 Sep 2013 20:15:36 +0000 (UTC)
> Rui Paulo <rpaulo@FreeBSD.org> 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




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?E275B036-8B04-4EFB-92CC-9C94C3D88F53>