Hi Andy, On Wed, Mar 8, 2023 at 11:29 PM Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote: > > On Wed, Mar 08, 2023 at 06:20:14PM +0800, Pin-yen Lin wrote: > > On Mon, Mar 6, 2023 at 7:49 PM Andy Shevchenko > > > On Fri, Mar 03, 2023 at 10:33:43PM +0800, Pin-yen Lin wrote: > > ... > > > > > + switch_desc->typec_ports = devm_kcalloc(dev, switch_desc->num_typec_switches, > > > > + sizeof(struct drm_dp_typec_port_data), > > > > + GFP_KERNEL); > > > > + if (!switch_desc->typec_ports) > > > > + return -ENOMEM; > > > > > > How often this function _can_ be called during the runtime? > > > If it's _possible_ to call it infinite times, consider *not* using devm. > > > > I would expect this function to be only called during driver probing, > > and this is the case for the current users in this series. So I think > > this is only called once if EPROBDE_DEFER doesn't count. > > No, deferred probe is not counted. > > Can you add a word somewhere in the comment(?) to make this clear? Sure. I'll add some note in the comments about this. > > -- > With Best Regards, > Andy Shevchenko > > Thanks and regards, Pin-yen