mbox series

[v3,0/3] drm/i915/xe3lpd: Update bandwidth parameters

Message ID 20250307-xe3lpd-bandwidth-update-v3-0-58bbe81f65bf@intel.com (mailing list archive)
Headers show
Series drm/i915/xe3lpd: Update bandwidth parameters | expand

Message

Gustavo Sousa March 7, 2025, 7:25 p.m. UTC
Bandwidth parameters for Xe3_LPD have been updated with respect to
previous display releases. Encode them into xe3lpd_sa_info and use that
new struct.

Since we are touching intel_bw.c, also take the opportunity convert it
to intel_display.

Changes in v2:
  - Fix typo in patch #2.

Changes in v3:
- Squash patches #1 and #2 into a single one and modify it to convert
  intel_bw.c internally to intel_display (new patch subject is
  "drm/i915/display: Convert intel_bw.c internally to intel_display").
- Add a new patch #2 to convert intel_bw.c externally to intel_display.
- Link to v2: https://lore.kernel.org/r/20250217153550.43909-1-gustavo.sousa@intel.com

---
Gustavo Sousa (3):
      drm/i915/display: Convert intel_bw.c internally to intel_display
      drm/i915/display: Convert intel_bw.c externally to intel_display
      drm/i915/xe3lpd: Update bandwidth parameters

 drivers/gpu/drm/i915/display/intel_bw.c            | 440 +++++++++++----------
 drivers/gpu/drm/i915/display/intel_bw.h            |   9 +-
 drivers/gpu/drm/i915/display/intel_cdclk.c         |   3 +-
 .../gpu/drm/i915/display/intel_display_driver.c    |   2 +-
 drivers/gpu/drm/i915/display/skl_watermark.c       |  10 +-
 drivers/gpu/drm/i915/i915_driver.c                 |   2 +-
 drivers/gpu/drm/xe/display/xe_display.c            |   2 +-
 7 files changed, 245 insertions(+), 223 deletions(-)
---
base-commit: f811577f424491a57b1e8669bde62998227d6907
change-id: 20250228-xe3lpd-bandwidth-update-f011599c0c3e

Best regards,

Comments

Jani Nikula March 10, 2025, 1:12 p.m. UTC | #1
On Fri, 07 Mar 2025, Gustavo Sousa <gustavo.sousa@intel.com> wrote:
> Bandwidth parameters for Xe3_LPD have been updated with respect to
> previous display releases. Encode them into xe3lpd_sa_info and use that
> new struct.
>
> Since we are touching intel_bw.c, also take the opportunity convert it
> to intel_display.

In case you didn't notice already, this series and [1] from Ville will
conflict badly with each other. Please collaborate to sort out which
series to merge first.

Thanks,
Jani.


[1] https://lore.kernel.org/r/20250307180139.15744-1-ville.syrjala@linux.intel.com


>
> Changes in v2:
>   - Fix typo in patch #2.
>
> Changes in v3:
> - Squash patches #1 and #2 into a single one and modify it to convert
>   intel_bw.c internally to intel_display (new patch subject is
>   "drm/i915/display: Convert intel_bw.c internally to intel_display").
> - Add a new patch #2 to convert intel_bw.c externally to intel_display.
> - Link to v2: https://lore.kernel.org/r/20250217153550.43909-1-gustavo.sousa@intel.com
>
> ---
> Gustavo Sousa (3):
>       drm/i915/display: Convert intel_bw.c internally to intel_display
>       drm/i915/display: Convert intel_bw.c externally to intel_display
>       drm/i915/xe3lpd: Update bandwidth parameters
>
>  drivers/gpu/drm/i915/display/intel_bw.c            | 440 +++++++++++----------
>  drivers/gpu/drm/i915/display/intel_bw.h            |   9 +-
>  drivers/gpu/drm/i915/display/intel_cdclk.c         |   3 +-
>  .../gpu/drm/i915/display/intel_display_driver.c    |   2 +-
>  drivers/gpu/drm/i915/display/skl_watermark.c       |  10 +-
>  drivers/gpu/drm/i915/i915_driver.c                 |   2 +-
>  drivers/gpu/drm/xe/display/xe_display.c            |   2 +-
>  7 files changed, 245 insertions(+), 223 deletions(-)
> ---
> base-commit: f811577f424491a57b1e8669bde62998227d6907
> change-id: 20250228-xe3lpd-bandwidth-update-f011599c0c3e
>
> Best regards,
Ville Syrjälä March 10, 2025, 4:37 p.m. UTC | #2
On Mon, Mar 10, 2025 at 03:12:43PM +0200, Jani Nikula wrote:
> On Fri, 07 Mar 2025, Gustavo Sousa <gustavo.sousa@intel.com> wrote:
> > Bandwidth parameters for Xe3_LPD have been updated with respect to
> > previous display releases. Encode them into xe3lpd_sa_info and use that
> > new struct.
> >
> > Since we are touching intel_bw.c, also take the opportunity convert it
> > to intel_display.
> 
> In case you didn't notice already, this series and [1] from Ville will
> conflict badly with each other. Please collaborate to sort out which
> series to merge first.

I can rebase my stuff, so no worries.

> 
> Thanks,
> Jani.
> 
> 
> [1] https://lore.kernel.org/r/20250307180139.15744-1-ville.syrjala@linux.intel.com
> 
> 
> >
> > Changes in v2:
> >   - Fix typo in patch #2.
> >
> > Changes in v3:
> > - Squash patches #1 and #2 into a single one and modify it to convert
> >   intel_bw.c internally to intel_display (new patch subject is
> >   "drm/i915/display: Convert intel_bw.c internally to intel_display").
> > - Add a new patch #2 to convert intel_bw.c externally to intel_display.
> > - Link to v2: https://lore.kernel.org/r/20250217153550.43909-1-gustavo.sousa@intel.com
> >
> > ---
> > Gustavo Sousa (3):
> >       drm/i915/display: Convert intel_bw.c internally to intel_display
> >       drm/i915/display: Convert intel_bw.c externally to intel_display
> >       drm/i915/xe3lpd: Update bandwidth parameters
> >
> >  drivers/gpu/drm/i915/display/intel_bw.c            | 440 +++++++++++----------
> >  drivers/gpu/drm/i915/display/intel_bw.h            |   9 +-
> >  drivers/gpu/drm/i915/display/intel_cdclk.c         |   3 +-
> >  .../gpu/drm/i915/display/intel_display_driver.c    |   2 +-
> >  drivers/gpu/drm/i915/display/skl_watermark.c       |  10 +-
> >  drivers/gpu/drm/i915/i915_driver.c                 |   2 +-
> >  drivers/gpu/drm/xe/display/xe_display.c            |   2 +-
> >  7 files changed, 245 insertions(+), 223 deletions(-)
> > ---
> > base-commit: f811577f424491a57b1e8669bde62998227d6907
> > change-id: 20250228-xe3lpd-bandwidth-update-f011599c0c3e
> >
> > Best regards,
> 
> -- 
> Jani Nikula, Intel
Gustavo Sousa March 10, 2025, 7:01 p.m. UTC | #3
Quoting Ville Syrjälä (2025-03-10 13:37:15-03:00)
>On Mon, Mar 10, 2025 at 03:12:43PM +0200, Jani Nikula wrote:
>> On Fri, 07 Mar 2025, Gustavo Sousa <gustavo.sousa@intel.com> wrote:
>> > Bandwidth parameters for Xe3_LPD have been updated with respect to
>> > previous display releases. Encode them into xe3lpd_sa_info and use that
>> > new struct.
>> >
>> > Since we are touching intel_bw.c, also take the opportunity convert it
>> > to intel_display.
>> 
>> In case you didn't notice already, this series and [1] from Ville will
>> conflict badly with each other. Please collaborate to sort out which
>> series to merge first.

Thanks for the heads-up, Jani.

>
>I can rebase my stuff, so no worries.

Thanks, Ville.

--
Gustavo Sousa