Re: [Intel-gfx] [PATCH v2 0/4] Simplify uncore register updates

[Andi Shyti <andi.shyti@linux.intel.com>]

Hi Andrzej,

On Thu, Oct 06, 2022 at 06:31:56PM +0200, Andrzej Hajda wrote:
> This patchset should not modify behaviour of the code (except patch 3).
> It just replaces sequence of uncore read/modify/write with single call.
> Moreover it replaces nested pointers with alias if there is one.
> All patches except 3rd, were generated using coccinelle (quite ugly)
> and adjusted in some cases by hand.
> 
> v2:
> - addressed comments by Andi, Jani and lkp,
> - despite Jani objections Ive left 1st patch with hope he will change his mind,
>   if not it can be safely ignored :),
> - converted few more read/modifiy/write sequences in last patch,
> - style adjustements to make checkpatch happy.
> 
> Regards
> Andrzej
> 
> 
> Andrzej Hajda (4):
>   drm/i915/display: Use drm_device alias if defined
>   drm/i915/display: Use intel_uncore alias if defined
>   drm/i915: make intel_uncore_rmw() write unconditionally
>   drm/i915: use proper helper for register updates

Pushed in drm-intel-next

Thanks,
Andi

>  drivers/gpu/drm/i915/display/icl_dsi.c        |   4 +-
>  drivers/gpu/drm/i915/display/intel_display.c  |   3 +-
>  .../drm/i915/display/intel_display_debugfs.c  |   2 +-
>  drivers/gpu/drm/i915/display/intel_hotplug.c  |   2 +-
>  drivers/gpu/drm/i915/display/intel_lvds.c     |  13 +-
>  drivers/gpu/drm/i915/display/intel_opregion.c |   2 +-
>  drivers/gpu/drm/i915/display/intel_tc.c       |   9 +-
>  drivers/gpu/drm/i915/display/vlv_dsi.c        |   2 +-
>  drivers/gpu/drm/i915/i915_irq.c               | 237 +++++++-----------
>  drivers/gpu/drm/i915/intel_pm.c               |  60 ++---
>  drivers/gpu/drm/i915/intel_uncore.h           |   8 +-
>  drivers/gpu/drm/i915/vlv_suspend.c            |  28 +--
>  12 files changed, 133 insertions(+), 237 deletions(-)
> 
> -- 
> 2.34.1