On 01/23/2014 08:23 PM, Michael Grzeschik wrote:
This patch adds basic support for the mv643xx gigabit ethernet stack. It is found on several marvell orion SoCs. The code is based on the kirkwood_egiga driver from u-boot and was renamed. It uses dma_alloc_coherent instead of xmalloc. The huge register representing struct was changed to register offset defines. The write and read macros got changed to direct writel and readl calls.
Nice patches! Unfortunately, I have no time to review and test them today, but I do have some remarks right now.
Signed-off-by: Michael Grzeschik <m.grzeschik@xxxxxxxxxxxxxx> --- drivers/net/Kconfig | 5 + drivers/net/Makefile | 1 + drivers/net/mv643xx.c | 714 ++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/net/mv643xx.h | 473 +++++++++++++++++++++++++++++++++
We really all hate "mv643xx" because it is a pain to say and write it. I guess barebox will never be run on systems with mv64xxx controllers but only Marvell Orion SoC. I'd be *very* happy if you do s/mv643xx/orion/g [...]
diff --git a/drivers/net/mv643xx.c b/drivers/net/mv643xx.c new file mode 100644 index 0000000..3d0bfdc --- /dev/null +++ b/drivers/net/mv643xx.c @@ -0,0 +1,714 @@ +/*
[...]
+ */ + +#include <common.h> +#include <malloc.h> +#include <net.h> +#include <init.h> +#include <driver.h> +#include <io.h> +#include <clock.h> +#include <xfuncs.h> +#include <linux/phy.h> +#include <linux/clk.h> +#include <linux/err.h> +#include <of_net.h> +#include <mach/dove-regs.h>
Please don't. The same driver will be used on Kirkwood and possibly orion5x, mv78x00 if they get supported. Have every register offset defined in here or "mv643xx.h" and get rid of the above. If you need some callback for memory windows, let's get it on now and create it in a way it is compatible with using this driver on the other SoCs. Also, we really have no plans for Dove, Kirkwood or any other Marvell SoC with !CONFIG_OF, so feel free to remove any reference to non-DT usage. BTW, how about sorting the #includes alphabetically? For the rest, I'll give it a go on Dove ASAP. Sebastian _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox
