Re: [PATCH v2 0/3] pwm: tegra: Cleanups and .get_state()

[Thierry Reding]

On Fri, Sep 18, 2026 at 04:33:44PM +0200, Uwe Kleine-König wrote:
> Hello,
> 
> v1 of this series can be found at
> https://lore.kernel.org/cover.1784030076.git.ukleinek@xxxxxxxxxx.
> 
> Changes since then:
> 
>  - Reordered the patches to have dev_err_probe and dev first. Fixes a
>    build failure in the middle of v1. This way patch 2 -- which could be
>    considered a fix -- isn't before the cleanup in patch 1, but doing
>    patch 1 the old way first also feels strange.
> 
>  - add { } around blocks with a single statement if there is also a
>    comment.
> 
>  - fixed too many parenthesis in patch #3 (formerly #6).
> 
>  - dropped other patches as they reorder stuff in unwanted or at least
>    untested ways.
> 
> There was a concern in reply to patch #1 of the v1 series (now #2) from
> Mikko Perttunen. He wrote: 
> 
> > I feel like driver_override falls in the realm of 'root can mess with
> > the system as they feel like but if they don't know what they're doing
> > they get to keep the pieces'. So adding a check in every driver, or
> > in practice having a random mix of drivers with and without the check,
> > doesn't seem necessary to me.
> > 
> > If we actually want to check for this condition, could it be done
> > centrally instead? I.e. don't call probe if there's no match data and
> > the driver's match table implies it requires it.
> 
> It cannot be done reliably in the driver core, and IMHO even root
> shouldn't be able to trigger a NULL pointer exception. So I kept the
> check.

As I mentioned in a comment to the patch, I second Mikko's concern.
Adding validity checks for device data seems like one of those
boilerplate things we should be able to avoid. We never match by name in
the drivers and if driver_override is the only reason why the device
data might end up being NULL, then driver_override should be completely
disabled for this driver because it simply isn't going to work without
the match data (as evidenced by your patch returning an error code in
that case).

I'll take a look at adding a way for the core to let drivers opt-out of
driver_override if it doesn't make sense for them.

Thierry

Attachment: signature.asc
Description: PGP signature