Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 23 Jul 2019 16:15:21 +0000 (UTC)
From:      Oskar Holmlund <oskar.holmlund@yahoo.com>
To:        John-Mark Gurney <jmg@funkthat.com>,  Emmanuel Vadot <manu@bidouilliste.com>
Cc:        arm@FreeBSD.org, Ian Lepore <ian@freebsd.org>
Subject:   Re: 13-CURRENT snapshot 20190718 r350103 doesn't boot on BeagleBone White
Message-ID:  <1962999297.7816997.1563898521984@mail.yahoo.com>
In-Reply-To: <20190723002116.75493451127739cf50b4077d@bidouilliste.com>
References:  <20190721180510.GQ2342@funkthat.com> <415c9b4760029235cd62bf95a35a736f7566cb9d.camel@freebsd.org> <20190721205557.GR2342@funkthat.com> <20190722102652.abde19a9fb609451cb618fde@bidouilliste.com> <20190722171251.GU2342@funkthat.com> <20190723002116.75493451127739cf50b4077d@bidouilliste.com>

next in thread | previous in thread | raw e-mail | index | archive | help
2019-07-23 00:21 skrev Emmanuel Vadot:
> On Mon, 22 Jul 2019 10:12:51 -0700
> John-Mark Gurney <jmg@funkthat.com> wrote:
>=20
>> Emmanuel Vadot wrote this message on Mon, Jul 22, 2019 at 10:26 +0200:
>> > On Sun, 21 Jul 2019 13:55:57 -0700
>> > John-Mark Gurney <jmg@funkthat.com> wrote:
>> >
>> > > Ian Lepore wrote this message on Sun, Jul 21, 2019 at 12:31 -0600:
>> > > > On Sun, 2019-07-21 at 11:05 -0700, John-Mark Gurney wrote:
>> > > > > The microSD card that I was using on my BeagleBone white got
>> > > > > corrupted,
>> > > > > so I decided to update to the latest version.=C2=A0 The latest s=
napshot
>> > > > > fails
>> > > > > to boot.=C2=A0 It loads the kernel, but then when starting the k=
ernel, it
>> > > > > hangs, and eventually it will reset.
>> > > > >
>> > > > > The latest 12 snapshot boots fine: BEAGLEBONE-20190718-r350087.
>> > > > >
>> > > > > Any ideas?=C2=A0 I tried all three available 13 snaps, and they =
all behave
>> > > > > the same.
>> > > > >
>> > > >
>> > > > This happened with the latest DTS import (which was months ago).=
=C2=A0 A
>> > > > couple people have speculated that we just need a trivial do-nothi=
ng
>> > > > driver for the new ti,sysc device, but when I tried that a couple
>> > > > months ago it didn't work, so instead I just reverted sys/dts to t=
he
>> > > > old source and got on with what I needed to do.
>> > >
>> > > Can we revert the dts in the tree then?=C2=A0 Doesn't help when we k=
now
>> > > the fix, but don't apply it...
>> >
>> >=C2=A0 That would be a pain for the next updates.
>>
>> Well, IMO, better than leaving a platform broken for months...
>>
>> > > Or add an overlay that undoes the changes?
>> > >
>> > > I can do some testing...
>> >
>> >=C2=A0 Could be possible but that will probably break in a few updates =
of the
>> > DTS files.
>> >
>> >=C2=A0 We need a TI maintainer that's all.
>>
>> I'd recommend we disconnect the builds then or something so that people
>> know not even to bother to try the images instead of wasting hours of
>> time trying to figure out what's wrong w/ the images...
>>
>> At least then I'd post where's the images, and you would have replied
>> that things aren't working...
>>
>> > > > This is just the latest in a years-long string of breakages becaus=
e the
>> > > > linux TI folks just never stop tinkering with their device-tree so=
urce.
>> > > > I'm sure they're doing it because it gets them some benefits, but =
for
>> > > > us the changes add no value and have a high maintenance cost.=C2=
=A0 A hang
>> > > > before the copyright banner appears is especially painful to debug
>> > > > (doubly so because there's no existing EARLY_PRINTF support in the=
 ti
>> > > > code).
>> > >
>> > > [...]
>>
>> --
>>=C2=A0=C2=A0 John-Mark Gurney=C2=A0=C2=A0 =C2=A0=C2=A0=C2=A0 =C2=A0=C2=A0=
=C2=A0 =C2=A0=C2=A0=C2=A0 =C2=A0Voice: +1 415 225 5579
>>
>>=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 "All that I will do, has been done, All th=
at I have, has not."
>=20
>=C2=A0 So I've unbroken the BBB.
>=20

I appreciate your work, thank you!

>=C2=A0 I've made two commits :
>=C2=A0 r350229 ([1]) changes the hwmod driver to lookup the property in th=
e
> parent node as the dts changed that.
>=C2=A0 r350230 ([2]) adds a new driver for the ti,sysc bus. It's a simpleb=
us
> like bus and for now we only threat it like if it was.
>=20
>=C2=A0 But this is not enough to boot on BBB for now with the latest DTS.
>=C2=A0 I've put on a github branch ([3]) two other commits.
>=20
>=C2=A0 The first one correct the region of uart0 which isn't correct, I gu=
ess linux
>=C2=A0 doesn't care about this as much as we do. Since this patches the DT=
S I want
>=C2=A0 to start the process of upstreaming first before commiting this pat=
ch into
>=C2=A0 our tree. I also want to update the DTS to Linux 5.2 since I want t=
o merge
>=C2=A0 those for 12.1 .

Is there any strategy for device tree imports from Linux?
In the upcoming 13 i dont mind running the latest and greatest. But in 11.x=
 / 12.x?=20

>=C2=A0 The second one take care of a problem in ofw_reg_to_paddr. Since we=
 have now
>=C2=A0 a lot of region in the ti.sysc drivers, using only 32 pcells for th=
e regions
>=C2=A0 isn't enough. I've temporary bumped this value to 64 which is enoug=
h
> for booting
>=C2=A0 on the BBB but we need a cleaner solution. I'll look into it soon-i=
sh.
>=20
>=C2=A0 So right now if you want to run current on BBB please take update y=
ou
> source tree
>=C2=A0 and take the two patches from my github branch.
>=20
>=C2=A0 Note that I think that this is the last time that I fix TI related =
problems
>=C2=A0 in the tree. I've spent too much time fixing BBB and Pandaboard dur=
ing the
>=C2=A0 last 12 months and I don't even use or care about those boards. If =
someone
>=C2=A0 wants to keep them alive please invest time or money into this.
>=20
Again, Thank you for your work. Yes I'm interested in keeping the TI SoC al=
ive.=20
I will put up the resources to do periodic tests and bugfixes for the 12-st=
able for AM335x (and OSD3358).
If we can agree on a baseline of tests we can perform the test on other boa=
rds aswell.=C2=A0 Maybe based on proposal from Mark Linimon?

>=C2=A0 Thanks to ian@ for helping me with this issue.
>=20
> [1] :
> https://svnweb.freebsd.org/base/head/sys/arm/ti/ti_hwmods.c?revision=3D35=
0229&view=3Dmarkup
> [2] :
> https://svnweb.freebsd.org/base/head/sys/arm/ti/ti_sysc.c?revision=3D3502=
30&view=3Dmarkup
> [3] : https://github.com/evadot/freebsd/tree/bbb

--=20
B=C3=A4sta H=C3=A4lsningar
Oskar Holmlund
Tel 070-3220292



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1962999297.7816997.1563898521984>