On Fri, Jun 01, 2007 at 11:04:03AM -0700, Jim Seymour wrote:
> This patch does two things:
> (.../packages/devs/serial/arm/at91/current/src/at91_serial.c)
> 
> 1) It eliminates a CYG_FAIL call when the serial receiver fills up.
> 2) It eliminates a compiler warning when setting the "end" variable

Looks good.

I think this is your first patch?

It is simple enough that we don't need a copyright assignment.  

Please could you provide a ChangeLog entry as part of the patch.
Also, please could you make your patch relative to the packages
directory, not deep down inside the tree. Then there is less chance of
confusion as to where the patch should be applied. With this
particular patch and the previous discussion, it is not a problem, but
sometimes it can take me a while to figure out where to apply a patch.

          Thanks
                    Andrew

Reply via email to