Fix misleading indentation warning:

usr.sbin/lmcconfig/lmcconfig.c:941:39: error: misleading indentation; statement is not part of the previous 'if' [-Werror,-Wmisleading-indentation]
return 64.0;
^
usr.sbin/lmcconfig/lmcconfig.c:940:3: note: previous statement is here
if ((vga >= 0x3A) && (vga < 0x3F)) return 50.0 + 2.80 * (vga - 0x3A);
^

Direct commit to stable/11, since lmcconfig has been removed from
FreeBSD 12 and later.

Copy head@r302406 to stable/11 as part of the 11.0-RELEASE cycle.
Prune svn:mergeinfo from the new branch, as nothing has been merged
here.

Additional commits post-branch will follow.

Approved by: re (implicit)
Sponsored by: The FreeBSD Foundation

Use strlcpy() instead of strncpy() when copying ifname to ensure
that it is NUL terminated. Additional NUL padding is not required
for short names.

Reported by: Coverity
CID: 974852
MFC after: 1 week

Cleanup unnecessary semicolons from utilities we all love.

Use NULL instead of 0 for pointers.

fopen(3) will return NULL in case it can't open the STREAM.

Rework time handling.

After I made the previous commit, I noticed the code does some things it
shouldn't. It casts a struct timeval to a time_t, assuming tv_sec is the
first member. Also, we are not interested in microseconds, so it is
better to just call time(NULL).

MFC after: 1 month

Remove unneeded struct timezone.

We're only interested in a timestamp -- not the timezone.

lmcconfig(8): bump to WARNS=3, fix typo in usage while here

Slight style(9) improvments
- function definitions
- sort #include

Fix type used to hold fgetc(3) output, this should help arm and powerpc
which have char == unsigned char

Remove useless conditional.

`vga' cannot be less than 0x3f when reaching the check.

Coverity Prevent: CID 5196

Include string.h for strncpy prototype
Include strings.h for bcopy and bzero prototype

Add the lmcconfig tool for controlling the lmc driver. Add man pages and
glue.

Submitted by: David Boggs