lkml.org 
[lkml]   [2021]   [Jul]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [RFC v2 2/2] drm/amd/display: Use PPC FPU functions
From
Date
Am 21.07.21 um 06:48 schrieb Anson Jacob:
> Use kernel_fpu_begin & kernel_fpu_end for PPC
>
> Depends on "ppc/fpu: Add generic FPU api similar to x86"
>
> v2:
> - Got rid of macro switch for PPC as header file with same
> name as x86 is added by previous patch in the series
>
> Signed-off-by: Anson Jacob <Anson.Jacob@amd.com>
> CC: Christoph Hellwig <hch@infradead.org>
> CC: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com>
> CC: Harry Wentland <harry.wentland@amd.com>
> CC: Christian König <christian.koenig@amd.com>

Looks good in general, but question is what about other architectures
like ARM?

Regards,
Christian.

> ---
> drivers/gpu/drm/amd/display/dc/os_types.h | 29 -----------------------
> 1 file changed, 29 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/os_types.h b/drivers/gpu/drm/amd/display/dc/os_types.h
> index 126c2f3a4dd3..47ef434f93d8 100644
> --- a/drivers/gpu/drm/amd/display/dc/os_types.h
> +++ b/drivers/gpu/drm/amd/display/dc/os_types.h
> @@ -51,38 +51,9 @@
> #define dm_error(fmt, ...) DRM_ERROR(fmt, ##__VA_ARGS__)
>
> #if defined(CONFIG_DRM_AMD_DC_DCN)
> -#if defined(CONFIG_X86)
> #include <asm/fpu/api.h>
> #define DC_FP_START() kernel_fpu_begin()
> #define DC_FP_END() kernel_fpu_end()
> -#elif defined(CONFIG_PPC64)
> -#include <asm/switch_to.h>
> -#include <asm/cputable.h>
> -#define DC_FP_START() { \
> - if (cpu_has_feature(CPU_FTR_VSX_COMP)) { \
> - preempt_disable(); \
> - enable_kernel_vsx(); \
> - } else if (cpu_has_feature(CPU_FTR_ALTIVEC_COMP)) { \
> - preempt_disable(); \
> - enable_kernel_altivec(); \
> - } else if (!cpu_has_feature(CPU_FTR_FPU_UNAVAILABLE)) { \
> - preempt_disable(); \
> - enable_kernel_fp(); \
> - } \
> -}
> -#define DC_FP_END() { \
> - if (cpu_has_feature(CPU_FTR_VSX_COMP)) { \
> - disable_kernel_vsx(); \
> - preempt_enable(); \
> - } else if (cpu_has_feature(CPU_FTR_ALTIVEC_COMP)) { \
> - disable_kernel_altivec(); \
> - preempt_enable(); \
> - } else if (!cpu_has_feature(CPU_FTR_FPU_UNAVAILABLE)) { \
> - disable_kernel_fp(); \
> - preempt_enable(); \
> - } \
> -}
> -#endif
> #endif
>
> /*

\
 
 \ /
  Last update: 2021-07-21 08:31    [W:0.086 / U:0.984 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site