[Intel-xe] [PATCH v2 2/4] fixup! drm/xe/display: Implement display support
Rodrigo Vivi
rodrigo.vivi at intel.com
Tue May 9 15:26:56 UTC 2023
On Mon, May 08, 2023 at 03:53:20PM -0700, Lucas De Marchi wrote:
> WARNING: This should only be squashed when the display implementation
> moves above commit "drm/xe/mmio: Use struct xe_reg".
I wonder if we should then try to move this patch under the display
instead waiting for the next round of moving the display up...
Also, could we change the subject from fixup to future-fixup so the
a git autosquash doesn't try to move this before we are ready?
>
> With the move of display above xe_reg conversion in xe_mmio,
> it should use the new types everywhere.
>
> Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
> Acked-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
> ---
> .../drm/xe/compat-i915-headers/intel_uncore.h | 103 +++++++++++++-----
> 1 file changed, 74 insertions(+), 29 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/compat-i915-headers/intel_uncore.h b/drivers/gpu/drm/xe/compat-i915-headers/intel_uncore.h
> index 90d79290a211..14f195fe275d 100644
> --- a/drivers/gpu/drm/xe/compat-i915-headers/intel_uncore.h
> +++ b/drivers/gpu/drm/xe/compat-i915-headers/intel_uncore.h
> @@ -17,82 +17,127 @@ static inline struct xe_gt *__fake_uncore_to_gt(struct fake_uncore *uncore)
> return to_gt(xe);
> }
>
> -static inline u32 intel_uncore_read(struct fake_uncore *uncore, i915_reg_t reg)
> +static inline u32 intel_uncore_read(struct fake_uncore *uncore,
> + i915_reg_t i915_reg)
> {
> - return xe_mmio_read32(__fake_uncore_to_gt(uncore), reg.reg);
> + struct xe_reg reg = XE_REG(i915_mmio_reg_offset(i915_reg));
> +
> + return xe_mmio_read32(__fake_uncore_to_gt(uncore), reg);
> }
>
> -static inline u32 intel_uncore_read8(struct fake_uncore *uncore, i915_reg_t reg)
> +static inline u32 intel_uncore_read8(struct fake_uncore *uncore,
> + i915_reg_t i915_reg)
> {
> - return xe_mmio_read8(__fake_uncore_to_gt(uncore), reg.reg);
> + struct xe_reg reg = XE_REG(i915_mmio_reg_offset(i915_reg));
> +
> + return xe_mmio_read8(__fake_uncore_to_gt(uncore), reg);
> }
>
> -static inline u64 intel_uncore_read64_2x32(struct fake_uncore *uncore, i915_reg_t lower_reg, i915_reg_t upper_reg)
> +static inline u64
> +intel_uncore_read64_2x32(struct fake_uncore *uncore,
> + i915_reg_t i915_lower_reg, i915_reg_t i915_upper_reg)
> {
> + struct xe_reg lower_reg = XE_REG(i915_mmio_reg_offset(i915_lower_reg));
> + struct xe_reg upper_reg = XE_REG(i915_mmio_reg_offset(i915_upper_reg));
> u32 upper, lower, old_upper;
> int loop = 0;
>
> - upper = xe_mmio_read32(__fake_uncore_to_gt(uncore), upper_reg.reg);
> + upper = xe_mmio_read32(__fake_uncore_to_gt(uncore), upper_reg);
> do {
> old_upper = upper;
> - lower = xe_mmio_read32(__fake_uncore_to_gt(uncore), lower_reg.reg);
> - upper = xe_mmio_read32(__fake_uncore_to_gt(uncore), upper_reg.reg);
> + lower = xe_mmio_read32(__fake_uncore_to_gt(uncore), lower_reg);
> + upper = xe_mmio_read32(__fake_uncore_to_gt(uncore), upper_reg);
> } while (upper != old_upper && loop++ < 2);
>
> return (u64)upper << 32 | lower;
> }
>
> -static inline void intel_uncore_posting_read(struct fake_uncore *uncore, i915_reg_t reg)
> +static inline void intel_uncore_posting_read(struct fake_uncore *uncore,
> + i915_reg_t i915_reg)
> {
> - xe_mmio_read32(__fake_uncore_to_gt(uncore), reg.reg);
> + struct xe_reg reg = XE_REG(i915_mmio_reg_offset(i915_reg));
> +
> + xe_mmio_read32(__fake_uncore_to_gt(uncore), reg);
> }
>
> -static inline void intel_uncore_write(struct fake_uncore *uncore, i915_reg_t reg, u32 val)
> +static inline void intel_uncore_write(struct fake_uncore *uncore,
> + i915_reg_t i915_reg, u32 val)
> {
> - xe_mmio_write32(__fake_uncore_to_gt(uncore), reg.reg, val);
> + struct xe_reg reg = XE_REG(i915_mmio_reg_offset(i915_reg));
> +
> + xe_mmio_write32(__fake_uncore_to_gt(uncore), reg, val);
> }
>
> -static inline u32 intel_uncore_rmw(struct fake_uncore *uncore, i915_reg_t reg, u32 clear, u32 set)
> +static inline u32 intel_uncore_rmw(struct fake_uncore *uncore,
> + i915_reg_t i915_reg, u32 clear, u32 set)
> {
> - return xe_mmio_rmw32(__fake_uncore_to_gt(uncore), reg.reg, clear, set);
> + struct xe_reg reg = XE_REG(i915_mmio_reg_offset(i915_reg));
> +
> + return xe_mmio_rmw32(__fake_uncore_to_gt(uncore), reg, clear, set);
> }
>
> -static inline int intel_wait_for_register(struct fake_uncore *uncore, i915_reg_t reg, u32 mask, u32 value, unsigned int timeout)
> +static inline int intel_wait_for_register(struct fake_uncore *uncore,
> + i915_reg_t i915_reg, u32 mask,
> + u32 value, unsigned int timeout)
> {
> - return xe_mmio_wait32(__fake_uncore_to_gt(uncore), reg.reg, value, mask, timeout * USEC_PER_MSEC, NULL, false);
> + struct xe_reg reg = XE_REG(i915_mmio_reg_offset(i915_reg));
> +
> + return xe_mmio_wait32(__fake_uncore_to_gt(uncore), reg, value, mask,
> + timeout * USEC_PER_MSEC, NULL, false);
> }
>
> -static inline int intel_wait_for_register_fw(struct fake_uncore *uncore, i915_reg_t reg, u32 mask, u32 value, unsigned int timeout)
> +static inline int intel_wait_for_register_fw(struct fake_uncore *uncore,
> + i915_reg_t i915_reg, u32 mask,
> + u32 value, unsigned int timeout)
> {
> - return xe_mmio_wait32(__fake_uncore_to_gt(uncore), reg.reg, value, mask, timeout * USEC_PER_MSEC, NULL, false);
> + struct xe_reg reg = XE_REG(i915_mmio_reg_offset(i915_reg));
> +
> + return xe_mmio_wait32(__fake_uncore_to_gt(uncore), reg, value, mask,
> + timeout * USEC_PER_MSEC, NULL, false);
> }
>
> -static inline int __intel_wait_for_register(struct fake_uncore *uncore, i915_reg_t reg, u32 mask, u32 value,
> - unsigned int fast_timeout_us, unsigned int slow_timeout_ms, u32 *out_value)
> +static inline int
> +__intel_wait_for_register(struct fake_uncore *uncore, i915_reg_t i915_reg,
> + u32 mask, u32 value, unsigned int fast_timeout_us,
> + unsigned int slow_timeout_ms, u32 *out_value)
> {
> - return xe_mmio_wait32(__fake_uncore_to_gt(uncore), reg.reg, value, mask,
> + struct xe_reg reg = XE_REG(i915_mmio_reg_offset(i915_reg));
> +
> + return xe_mmio_wait32(__fake_uncore_to_gt(uncore), reg, value, mask,
> fast_timeout_us + 1000 * slow_timeout_ms,
> out_value, false);
> }
>
> -static inline u32 intel_uncore_read_fw(struct fake_uncore *uncore, i915_reg_t reg)
> +static inline u32 intel_uncore_read_fw(struct fake_uncore *uncore,
> + i915_reg_t i915_reg)
> {
> - return xe_mmio_read32(__fake_uncore_to_gt(uncore), reg.reg);
> + struct xe_reg reg = XE_REG(i915_mmio_reg_offset(i915_reg));
> +
> + return xe_mmio_read32(__fake_uncore_to_gt(uncore), reg);
> }
>
> -static inline void intel_uncore_write_fw(struct fake_uncore *uncore, i915_reg_t reg, u32 val)
> +static inline void intel_uncore_write_fw(struct fake_uncore *uncore,
> + i915_reg_t i915_reg, u32 val)
> {
> - xe_mmio_write32(__fake_uncore_to_gt(uncore), reg.reg, val);
> + struct xe_reg reg = XE_REG(i915_mmio_reg_offset(i915_reg));
> +
> + xe_mmio_write32(__fake_uncore_to_gt(uncore), reg, val);
> }
>
> -static inline u32 intel_uncore_read_notrace(struct fake_uncore *uncore, i915_reg_t reg)
> +static inline u32 intel_uncore_read_notrace(struct fake_uncore *uncore,
> + i915_reg_t i915_reg)
> {
> - return xe_mmio_read32(__fake_uncore_to_gt(uncore), reg.reg);
> + struct xe_reg reg = XE_REG(i915_mmio_reg_offset(i915_reg));
> +
> + return xe_mmio_read32(__fake_uncore_to_gt(uncore), reg);
> }
>
> -static inline void intel_uncore_write_notrace(struct fake_uncore *uncore, i915_reg_t reg, u32 val)
> +static inline void intel_uncore_write_notrace(struct fake_uncore *uncore,
> + i915_reg_t i915_reg, u32 val)
> {
> - xe_mmio_write32(__fake_uncore_to_gt(uncore), reg.reg, val);
> + struct xe_reg reg = XE_REG(i915_mmio_reg_offset(i915_reg));
> +
> + xe_mmio_write32(__fake_uncore_to_gt(uncore), reg, val);
> }
>
> #endif /* __INTEL_UNCORE_H__ */
> --
> 2.40.1
>
More information about the Intel-xe
mailing list