On Wed, Nov 10, 2021 at 09:49:49AM +0100, Uwe Kleine-König wrote: > Each devm allocations has an overhead of 24 bytes to store the related > struct devres_node additionally to the fragmentation of the allocator. > So allocating 16 struct lpc18xx_pwm_data (which only hold a single int) > adds quite some overhead. Instead put the per-channel data into the > driver data struct and allocate it in one go. > > Signed-off-by: Uwe Kleine-König > --- > drivers/pwm/pwm-lpc18xx-sct.c | 12 +++++------- > 1 file changed, 5 insertions(+), 7 deletions(-) This could've been merged with patch 3 to make it a bit more obvious why this is useful. Patch 2 itself is a bit half-baked without patch 3. But I'll apply these anyway. Thanks, Thierry