Re: [PATCH] drm/i915/hwmon: Get rid of devm

2024-04-18 Thread Dixit, Ashutosh
On Thu, 18 Apr 2024 14:56:58 -0700, Andi Shyti wrote: > > > v2: Change commit message and other minor code changes > > v3: Cleanup from i915_hwmon_register on error (Armin Wolf) > > v4: Eliminate potential static analyzer warning (Rodrigo) > > Eliminate fetch_and_zero (Jani) > > v5: Restore pre

Re: [PATCH] drm/i915/hwmon: Get rid of devm

2024-04-18 Thread Andi Shyti
Hi Ashutosh, On Wed, Apr 17, 2024 at 07:56:46AM -0700, Ashutosh Dixit wrote: > When both hwmon and hwmon drvdata (on which hwmon depends) are device > managed resources, the expectation, on device unbind, is that hwmon will be > released before drvdata. However, in i915 there are two separate code

[PATCH] drm/i915/hwmon: Get rid of devm

2024-04-17 Thread Ashutosh Dixit
When both hwmon and hwmon drvdata (on which hwmon depends) are device managed resources, the expectation, on device unbind, is that hwmon will be released before drvdata. However, in i915 there are two separate code paths, which both release either drvdata or hwmon and either can be released before

[PATCH] drm/i915/hwmon: Get rid of devm

2024-04-15 Thread Ashutosh Dixit
When both hwmon and hwmon drvdata (on which hwmon depends) are device managed resources, the expectation, on device unbind, is that hwmon will be released before drvdata. However, in i915 there are two separate code paths, which both release either drvdata or hwmon and either can be released before