svn commit: r323362 - in head/sys: conf dev/neta

[Marcin Wojtas]

2017-09-11 11:38 GMT+02:00 Andrew Turner <andrew at fubar.geek.nz>:
>
>> On 9 Sep 2017, at 12:54, Marcin Wojtas <mw at FreeBSD.org> wrote:
>>
>> Author: mw
>> Date: Sat Sep  9 11:54:04 2017
>> New Revision: 323362
>> URL: https://svnweb.freebsd.org/changeset/base/323362
>>
>> Log:
>>  Add support for Armada 3700 in the NETA driver
>>
>>  This patch enables using NETA driver on Marvell Armada 3700 SoC
>>  by introducing new compatible string, modifying clock source
>>  obtaining and also excluding unnecessary parts.
>>  The driver is added as a build option for arm64 platforms as well.
>
> I’m getting the following when trying to build with gcc (I have initial patches to build arm64 with gcc form ports).
>
> Andrew
>
> --- if_mvneta.o ---
> /jenkins/workspace/FreeBSD-arm64-head-gcc/freebsd-head/sys/dev/neta/if_mvneta.c:101:1: error: function declaration isn't a prototype [-Werror=strict-prototypes]
>  mvneta_get_clk()
>  ^~~~~~~~~~~~~~
> /jenkins/workspace/FreeBSD-arm64-head-gcc/freebsd-head/sys/dev/neta/if_mvneta.c: In function 'mvneta_detach':
> /jenkins/workspace/FreeBSD-arm64-head-gcc/freebsd-head/sys/dev/neta/if_mvneta.c:834:16: warning: variable 'ifp' set but not used [-Wunused-but-set-variable]
>   struct ifnet *ifp;
>                 ^~~
>

Can you apply below and confirm it helps?
https://pastebin.com/8kfdnMz3

If yes, I'll issue a commit with fix.

Marcin