Date: Wed, 4 Apr 2018 02:13:14 +0200 From: Marcin Wojtas <mw@semihalf.com> To: Emmanuel Vadot <manu@bidouilliste.com> Cc: Marcin Wojtas <mw@freebsd.org>, src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org, owner-src-committers@freebsd.org Subject: Re: svn commit: r331961 - in head/sys: arm/mv dts/arm Message-ID: <CAPv3WKd8yM8ud=AroNg-CUPbJvrzDxTB-P=_2KFpS=b=N2kQrA@mail.gmail.com> In-Reply-To: <5dea78be118fa5ee4b8cb77a1e26a6f0@megadrive.org> References: <201804032221.w33MLDTE019502@repo.freebsd.org> <5dea78be118fa5ee4b8cb77a1e26a6f0@megadrive.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Hi Manu, 2018-04-04 0:28 GMT+02:00 Emmanuel Vadot <manu@bidouilliste.com>: > On 2018-04-04 00:21, Marcin Wojtas wrote: >> >> Author: mw >> Date: Tue Apr 3 22:21:12 2018 >> New Revision: 331961 >> URL: https://svnweb.freebsd.org/changeset/base/331961 >> >> Log: >> Make Marvell AmadaXP timer driver more generic >> >> Store pointers to SoC specific functions in mv_timer_config structure >> and determine proper config in runtime based on compatible string from >> FDT. >> Compatible string for ArmadaXP timers is changed to match Linux FDT. >> Armada 38x uses generic Cortex-A9 timer and separate watchdog drivers, >> so >> it does not need to be supported by timer driver. >> >> Submitted by: Rafal Kozik <rk@semihalf.com> >> Reviewed by: manu >> Obtained from: Semihalf >> Sponsored by: Stormshield >> Differential Revision: https://reviews.freebsd.org/D14741 > > > Hello Marcin, > > I've only reviewed the DTS part. Right, forgot to add note - sorry. > Also you didn't address andrew@ comment, see > https://reviews.freebsd.org/D14741?id=40452#inline-89719 > I took the wrong patch version initially (it was fixed in phabricator long time ago). I applied this improvemnet in r331965. Thanks, Marcin
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAPv3WKd8yM8ud=AroNg-CUPbJvrzDxTB-P=_2KFpS=b=N2kQrA>