Commit a4583c1d authored by Tang Bin's avatar Tang Bin Committed by Marc Kleine-Budde
Browse files

can: mscan: mpc5xxx_can: mpc5xxx_can_probe(): use of_device_get_match_data to simplify code 

Retrieve OF match data, it's better and cleaner to use
'of_device_get_match_data' over 'of_match_device'.

Link: https://lore.kernel.org/r/20210823113338.3568-4-tangbin@cmss.chinamobile.com


Signed-off-by: default avatarTang Bin <tangbin@cmss.chinamobile.com>
Signed-off-by: default avatarMarc Kleine-Budde <mkl@pengutronix.de>
parent 1d38ec49

drivers/net/can/mscan/mpc5xxx_can.c

+2 −4
Original line number Original line Diff line number Diff line
@@ -279,7 +279,6 @@ static u32 mpc512x_can_get_clock(struct platform_device *ofdev, 
static const struct of_device_id mpc5xxx_can_table[];

static const struct of_device_id mpc5xxx_can_table[];

static int mpc5xxx_can_probe(struct platform_device *ofdev)

static int mpc5xxx_can_probe(struct platform_device *ofdev)

{

{

	const struct of_device_id *match;

	const struct mpc5xxx_can_data *data;

	const struct mpc5xxx_can_data *data;

	struct device_node *np = ofdev->dev.of_node;

	struct device_node *np = ofdev->dev.of_node;

	struct net_device *dev;

	struct net_device *dev;
@@ -289,10 +288,9 @@ static int mpc5xxx_can_probe(struct platform_device *ofdev) 
	int irq, mscan_clksrc = 0;

	int irq, mscan_clksrc = 0;

	int err = -ENOMEM;

	int err = -ENOMEM;





	match = of_match_device(mpc5xxx_can_table, &ofdev->dev);

	data = of_device_get_match_data(&ofdev->dev);

	if (!match)

	if (!data)

		return -EINVAL;

		return -EINVAL;

	data = match->data;





	base = of_iomap(np, 0);

	base = of_iomap(np, 0);

	if (!base) {

	if (!base) {