On 9/3/19 2:42 PM, Andy Shevchenko wrote: > Instead of using legacy platform data, switch to use device properties. > For clock frequency we are using well established clock-frequency property. > > Users, two for now, are also converted here. [...] > diff --git a/drivers/net/can/spi/mcp251x.c b/drivers/net/can/spi/mcp251x.c > index 6ee0ea51399a..3a4d7089dc7c 100644 > --- a/drivers/net/can/spi/mcp251x.c > +++ b/drivers/net/can/spi/mcp251x.c > @@ -20,29 +20,26 @@ > * > * Your platform definition file should specify something like: > * > - * static struct mcp251x_platform_data mcp251x_info = { > - * .oscillator_frequency = 8000000, > + * static const struct property_entry mpc251x_properties[] = { > + * PROPERTY_ENTRY_U32("clock-frequency", 8000000), > + * {} > * }; > * > * static struct spi_board_info spi_board_info[] = { > * { > * .modalias = "mcp2510", > * // "mcp2515" or "mcp25625" depending on your controller > - * .platform_data = &mcp251x_info, > + * .properties = &mcp251x_properties, > * .irq = IRQ_EINT13, > * .max_speed_hz = 2*1000*1000, > * .chip_select = 2, > * }, > * }; > - * > - * Please see mcp251x.h for a description of the fields in > - * struct mcp251x_platform_data. > */ I've removed this section already in this patch: https://git.kernel.org/pub/scm/linux/kernel/git/mkl/linux-can-next.git/commit/?h=linux-can-next-for-5.4-20190903&id=f6cae800bfdb6711f0d45af98643a944998be6f2 ...I've dropped that hunk. regards, Marc -- Pengutronix e.K. | Marc Kleine-Budde | Industrial Linux Solutions | Phone: +49-231-2826-924 | Vertretung West/Dortmund | Fax: +49-5121-206917-5555 | Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de |