[PATCH] pwm: lpc32xx: remove handling of PWM channels

[Uwe Kleine-König]

Hello Thierry,

On Mon, Jul 24, 2023 at 04:00:32PM +0200, Uwe Kleine-König wrote:
> As noted by Dan Carpenter in
> https://lore.kernel.org/linux-pwm/919cac5d-491e-4534-baed-bf813181192d@moroto.mountain
> lpc32xx->chip.pwms is NULL before devm_pwmchip_add() is called so this
> patch fixes a null pointer exception.
> 
> Maybe add the following to the commit log:
> 
> 	Even though I wasn't aware at the time when I forward ported that patch,
> 	this fixes a null pointer dereference as lpc32xx->chip.pwms is NULL
> 	before devm_pwmchip_add() is called.
> 
> 	Reported-by: Dan Carpenter <dan.carpenter at linaro.org>
> 	Fixes: 3d2813fb17e5 ("pwm: lpc32xx: Don't modify HW state in .probe() after the PWM chip was registered")

You (probably with b4's help) picked up the Fixes line. The description
and also the Reported-by was not picked up, though.

IMHO adding the missed bits would be beneficial.

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20230728/5af4dd6a/attachment-0001.sig>