On Mon, Mar 2, 2015 at 10:44 AM, Alexandru Ardelean
<ardeleana...@gmail.com> wrote:
> So, on a powerpc system this works.
>
> static inline u64 b53_mac_array_to_u64(const u8 *u8_arr) {
> u64 mac = 0;
> u8 *cmac = (u8 *)&mac;
> memcpy(&cmac[2], u8_arr, 6);
> return mac;
> }
>
> I've done this approach initially, but there were some concerns afterwards
> regarding endianness.
> On my x86_64 system it looks ok, but I'm hoping you'd validate that this is
> endian-correct and would work on little endian targets.
> And then I'll move this in the port mirroring patch.
Please don't top post.
Hm, looking a the original patch, did you test this in little endian?
the shift looks wrong for there.
Same for the memcpy, you need to copy to cmac[0] in case of little
endian. Maybe it would be easier to just make the source mac u16
aligned, and then use ether_addr_copy().
Jonas
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
