Re: [PATCH v9 1/7] nvmem: core: Create all cells before adding the nvmem device

[Miquel Raynal <miquel.raynal@bootlin.com>]

Hi Srinivas,

srinivas.kandagatla@linaro.org wrote on Fri, 11 Aug 2023 13:26:24 +0100:

> On 11/08/2023 13:11, Miquel Raynal wrote:
> 
> >>
> >>  
> > 
> > nvmem_register() calls device_initialize() and later device_add(),
> > which is exactly the content of device_register(). Upon error
> > after device_add(), we currently call device_put(), whereas
> > device_unregister would call both device_del() and device_put().
> > 
> > I would expect device_del() to be first called upon error before
> > device_put() *after* device_add() has succeded, no?  
> 
> That is correct afaiu, if device_add is succeed we need to call device_del(). As the patch now moved the device_add to end of function we really do not need device_del() in err path.

Right, I'm looking at the end of the series where I need to add
device_del() in the error path because something gets added after
device_add(). So we are aligned, thanks for the feedback.

> >>> I also see the layout_np below should be freed before jumping in the
> >>> error section.  
> >>
> >> you mean missing of_node_put()?  
> > 
> > Yes, I need to call of_node_put() before jumping into the error path.  
> 
> Are we not already doing it in nvmem_layout_get() and nvmem_add_cells_from_fixed_layout() ?

We perform the layout_get for two reasons:
- knowing if there is a layout
- using the layout
Here we are in the first case, and we don't want to retain a reference
from here. Only in the second case.

Thanks,
Miquèl