[PATCH] net:fec: fixed unaligned access and stack corruption
Marco Felsch
m.felsch at pengutronix.de
Thu Jul 30 17:33:23 EDT 2020
Hi Ahmad,
On 20-07-30 23:23, Ahmad Fatoum wrote:
> Hello Marco,
>
> On 7/30/20 11:13 PM, Marco Felsch wrote:
> > Hi,
> >
> > sorry for jumping in.
>
> No worries,
>
> > On 20-07-11 07:28, Ahmad Fatoum wrote:
> >> Ah, I thought the drvdata is pre-populated in Linux. I would rename
> >> the new function to device_get_match_data then for alignment with
> >> Linux, with the difference that it returns either platform data or
> >> device tree driver_data as appropriate.
> >
> > Linux uses the of_device_get_match_data() [1] for of-based drivers.
> >
> > [1] https://elixir.bootlin.com/linux/v5.8-rc4/source/drivers/of/device.c#L189
>
> Yes, future barebox device_get_match_data should work with both of
> and platform data, that's why I'd drop the of_ prefix.
I think there is already a device_ wrapper function too but those are
only to differentiate between acpi and of functions. So +1 from my side
:)
Regards,
Marco
More information about the barebox
mailing list