> -----Original Message-----
> From: Mathieu Poirier <[email protected]>
> Sent: Wednesday, November 26, 2025 12:54 PM
> To: Shenwei Wang <[email protected]>
> Cc: Bjorn Andersson <[email protected]>; Rob Herring <[email protected]>;
> Krzysztof Kozlowski <[email protected]>; Conor Dooley
> <[email protected]>; Shawn Guo <[email protected]>; Sascha Hauer
> <[email protected]>; Jonathan Corbet <[email protected]>; Linus Walleij
> <[email protected]>; Bartosz Golaszewski <[email protected]>; Pengutronix
> Kernel Team <[email protected]>; Fabio Estevam <[email protected]>;
> Peng Fan <[email protected]>; [email protected];
> [email protected]; [email protected]; linux-arm-
> [email protected]; [email protected]; linux-
> [email protected]; dl-linux-imx <[email protected]>
> Subject: [EXT] Re: [PATCH v5 2/5] remoteproc: imx_rproc: Populate devices
> under "rpmsg" subnode
> > +
> > + drvdata = dev_get_drvdata(&rpdev->dev);
> > + if (drvdata && drvdata->rx_callback)
> > + return drvdata->rx_callback(rpdev, data, len, priv,
> > + src);
> > +
> > + return 0;
> > +}
> > +
> > +static void imx_rpmsg_endpoint_remove(struct rpmsg_device *rpdev) {
> > + of_platform_depopulate(&rpdev->dev);
> > +}
> > +
> > +static int imx_rpmsg_endpoint_probe(struct rpmsg_device *rpdev) {
>
> Where does the rpmsg_device come from? Usually there is a call to
> rpmsg_register_device() and I don't see it anywhere in this patchset. I also
> don't
> see a link to the remote processor. I can't continue with this set for as
> long as I
> don't have this information.
>
It is in the function below named imx_of_rpmsg_register_rpdriver.
Thanks,
Shenwei
> > +
> > +static int imx_of_rpmsg_register_rpdriver(struct device_node *channel,
> > + struct device *dev, int idx) {
> > + struct imx_rpmsg_driver_data *driver_data;
> > + struct imx_rpmsg_driver *rp_driver;
> > + struct rpmsg_device_id *rpdev_id;
> > +
> > + rpdev_id = devm_kzalloc(dev, sizeof(*rpdev_id) * 2, GFP_KERNEL);
> > + if (!rpdev_id)
> > + return -ENOMEM;
> > +
> > + strscpy(rpdev_id[0].name, channel_device_map[idx][0],
> > + RPMSG_NAME_SIZE);
> > +
> > + rp_driver = devm_kzalloc(dev, sizeof(*rp_driver), GFP_KERNEL);
> > + if (!rp_driver)
> > + return -ENOMEM;
> > +
> > + driver_data = devm_kzalloc(dev, sizeof(*driver_data), GFP_KERNEL);
> > + if (!driver_data)
> > + return -ENOMEM;
> > +
> > + driver_data->rproc_name = dev->of_node->name;
> > + driver_data->channel_node = channel;
> > + driver_data->map_idx = idx;
> > +
> > + rp_driver->rpdrv.drv.name = channel_device_map[idx][0];
> > + rp_driver->rpdrv.id_table = rpdev_id;
> > + rp_driver->rpdrv.probe = imx_rpmsg_endpoint_probe;
> > + rp_driver->rpdrv.remove = imx_rpmsg_endpoint_remove;
> > + rp_driver->rpdrv.callback = imx_rpmsg_endpoint_cb;
> > + rp_driver->driver_data = driver_data;
> > +
> > + register_rpmsg_driver(&rp_driver->rpdrv);
> > +
> > + return 0;
> > +}
