Messages in this thread | | | From | Bartosz Golaszewski <> | Date | Mon, 24 Sep 2018 13:44:19 +0200 | Subject | Re: [PATCH v3 4/4] clk: pmc-atom: use devm_kstrdup_const() |
| |
pon., 24 wrz 2018 o 13:23 Andy Shevchenko <andriy.shevchenko@linux.intel.com> napisał(a): > > On Mon, Sep 24, 2018 at 12:11:50PM +0200, Bartosz Golaszewski wrote: > > Use devm_kstrdup_const() in the pmc-atom driver. This mostly serves as > > an example of how to use this new routine to shrink driver code. > > > > While we're at it: replace a call to kcalloc() with devm_kcalloc(). > > > @@ -352,8 +344,6 @@ static int plt_clk_probe(struct platform_device *pdev) > > goto err_drop_mclk; > > } > > > > - plt_clk_free_parent_names_loop(parent_names, data->nparents); > > - > > platform_set_drvdata(pdev, data); > > return 0; > > I don't think this is a good example. > > You changed a behaviour here in the way that you keep all chunks of memory > (even small enough for pointers) during entire life time of the driver, which > pretty likely would be forever till next boot. > > In the original case the memory was freed immediately in probe either it fails > or returns with success. > > NAK, sorry. > >
I see.
I'd like to still merge patches 1-3 and then I'd come up with better examples for the next release cycle once these are in?
Bart
| |