Hi Peter,
On 09/08/11 00:47, Peter Turczak wrote:
Hi Greg,
thank you for the hint with scripts/checkpatch.pl, it helped a lot.
On Aug 8, 2011, at 7:03 AM, Greg Ungerer wrote:
> Overall it looks right. Just a couple of minor nits to clean up:
> Lets simplify the logic here a little and do:
Done.
>
> There is no spi-mcf520x.c in this patch...
> Is there supposed to be?
This is a platform definition file for my hardware. This will be be released
along with the hardware. Just forget about it..
>
> Can you regenerate your patch, and add a Signed-off-by?
Please find attached the regenerated patch.
Great, that looks good thanks.
Applied to the m68knommu git tree.
Regards
Greg
------------------------------------------------------------------------
Greg Ungerer -- Principal Engineer EMAIL: g...@snapgear.com
SnapGear Group, McAfee PHONE: +61 7 3435 2888
8 Gardner Close FAX: +61 7 3217 5323
Milton, QLD, 4064, Australia WEB: http://www.SnapGear.com
_______________________________________________
uClinux-dev mailing list
uClinux-dev@uclinux.org
http://mailman.uclinux.org/mailman/listinfo/uclinux-dev
This message was resent by uclinux-dev@uclinux.org
To unsubscribe see:
http://mailman.uclinux.org/mailman/options/uclinux-dev
