mpr: fix copying of event_mask

[oshogbo]

Summary

Before the commit 6cc44223cb6717795afdac4348bbe7e2a968a07d the
field event_mask was fully copied to the EventMasks field.
After this commit the event_mask (uint8_t) is 4 times casted to
EventMask (uint32_t). Because of that 24 bits of each event_mask array
is lost.

This commits brings back simple copying of field, and after words
converting 32 bits field to the requested endian.

I don't think we need more sufisticated method, as the array is of size 4 (for 32 bits version).

[oshogbo]

Fix description

[imp]

sys/dev/mpr/mpr.c

2767

16 seems wrong here and below, especially since we use MPI2_EVENT_NOTIFY_EVENTMASK_WORDS for the loops, maybe an expression with that?

[oshogbo]

Use sizeof.

[oshogbo]

sys/dev/mpr/mpr.c

2767

Actually is ok because we are copying an array of 16 elements of unit8_t to uint32_t of elements 4.
However, you are right I shouldn't hardcore the values (its from original code) I just used converted it to sizeof.

[oshogbo]

@imp Have you had a chance to look into this?

[imp]

looks good. I'm torn on the other 16s since it's the other leg of the branch.
But the change itself looks good.

sys/dev/mpr/mpr.c

2761

the 16s here could also be changed, but that's starting to stray a bit from the remit of your change.