svn commit: r40942 - head/en_US.ISO8859-1/books/arch-handbook/driverbasics

[Benjamin Kaduk]

On Mon, 11 Feb 2013, Eitan Adler wrote:

>
> Modified: head/en_US.ISO8859-1/books/arch-handbook/driverbasics/chapter.xml
> ==============================================================================
> --- head/en_US.ISO8859-1/books/arch-handbook/driverbasics/chapter.xml	Mon Feb 11 13:46:24 2013	(r40941)
> +++ head/en_US.ISO8859-1/books/arch-handbook/driverbasics/chapter.xml	Mon Feb 11 13:46:26 2013	(r40942)
> @@ -135,17 +134,17 @@ DECLARE_MODULE(skeleton, skel_mod, SI_SU
>     <sect2>
>       <title>Makefile</title>
>
> -      <para>FreeBSD provides a makefile include that you can use to
> -	quickly compile your kernel addition.</para>
> +      <para>FreeBSD provides a makefile include to
> +	quickly compile a kernel addition.</para>

This sentence does not seem right -- the user compiles the module, not the 
include file.  I think these .mk files are referred to as "system 
makefiles", anyway.  The best I can do on short notice is:
"FreeBSD provides a system makefile to simplify compiling a kernel 
module."

>       <programlisting>SRCS=skeleton.c
> KMOD=skeleton
>
> @@ -201,8 +200,8 @@ KMOD=skeleton
>       the source tree.</para>
>
>     <para>This simple example pseudo-device remembers whatever values
> -      you write to it and can then supply them back to you when you
> -      read from it.</para>
> +      that are written to it and can then echo them back when
> +      read.</para>

"whatever values that are written to it" doesn't seem right; "whatever 
values are written to it" feels much better.

-Ben

>     <example>
>       <title>Example of a Sample Echo Pseudo-Device Driver for