> -----Original Message-----
> From: Peng Fan (OSS) <[email protected]>
> Sent: Thursday, August 21, 2025 6:03 AM
> To: Shenwei Wang <[email protected]>
> Cc: Bjorn Andersson <[email protected]>; Mathieu Poirier
> <[email protected]>; Rob Herring <[email protected]>; Krzysztof
> Kozlowski <[email protected]>; Conor Dooley <[email protected]>; Shawn
> Guo <[email protected]>; Sascha Hauer <[email protected]>; Linus
> Walleij <[email protected]>; Bartosz Golaszewski <[email protected]>;
> Pengutronix Kernel Team <[email protected]>; Fabio Estevam
> <[email protected]>; Peng Fan <[email protected]>; linux-
> [email protected]; [email protected]; [email protected];
> [email protected]; [email protected];
> dl-linux-imx
> <[email protected]>
> Subject: Re: [PATCH 2/4] remoteproc: imx_rproc: Populate devices under
> "rpmsg" subnode
> >
> > #include "imx_rproc.h"
> >@@ -1084,6 +1088,126 @@ static int imx_rproc_sys_off_handler(struct
> sys_off_data *data)
> > return NOTIFY_DONE;
> > }
> >
>
> Just wonder could the changes be moved to drivers/rpmsg?
Since the DTS node is under imx_rproc, the most straightforward approach is to
implement the
feature here. Moving the implementation to drivers/rpmsg would introduce
additional complexity
and might be less direct.
Thanks,
Shenwei
>
> >+struct imx_rpmsg_driver {
> >+ struct rpmsg_driver rpdrv;
> >+ void *driver_data;
> >+};
> >+
> >+ i = drvdata->map_idx;
> >+ if (i >= ARRAY_SIZE(channel_device_map))
> >+ return -ENODEV;
> >+
> >+ auxdata = devm_kzalloc(dev, sizeof(*auxdata)*2, GFP_KERNEL);
> >+ if (!auxdata)
> >+ return -ENOMEM;
> >+
> >+ u8 major;
> >+ u8 minor;
> >+ u8 type;
> >+ u8 cmd;
> >+ u8 reserved[5];
> >+} __packed;
>
> A comment should be added to describe each member.
>
> Regards,
> Peng
