* [PATCH] drm/radeon: Prefer 'unsigned int' to bare use of 'unsigned' @ 2023-07-28 14:35 孙冉 2023-07-29 14:12 ` Bagas Sanjaya 0 siblings, 1 reply; 3+ messages in thread From: 孙冉 @ 2023-07-28 14:35 UTC (permalink / raw) To: airlied, daniel, alexander.deucher; +Cc: amd-gfx, dri-devel, linux-kernel WARNING: Prefer 'unsigned int' to bare use of 'unsigned' Signed-off-by: Ran Sun <sunran001@208suo.com> --- drivers/gpu/drm/radeon/radeon_object.h | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_object.h b/drivers/gpu/drm/radeon/radeon_object.h index 39cc87a59a9a..9b55a7103cfd 100644 --- a/drivers/gpu/drm/radeon/radeon_object.h +++ b/drivers/gpu/drm/radeon/radeon_object.h @@ -37,7 +37,7 @@ * * Returns corresponding domain of the ttm mem_type */ -static inline unsigned radeon_mem_type_to_domain(u32 mem_type) +static inline unsigned int radeon_mem_type_to_domain(u32 mem_type) { switch (mem_type) { case TTM_PL_VRAM: @@ -112,12 +112,12 @@ static inline unsigned long radeon_bo_size(struct radeon_bo *bo) return bo->tbo.base.size; } -static inline unsigned radeon_bo_ngpu_pages(struct radeon_bo *bo) +static inline unsigned int radeon_bo_ngpu_pages(struct radeon_bo *bo) { return bo->tbo.base.size / RADEON_GPU_PAGE_SIZE; } -static inline unsigned radeon_bo_gpu_page_alignment(struct radeon_bo *bo) +static inline unsigned int radeon_bo_gpu_page_alignment(struct radeon_bo *bo) { return (bo->tbo.page_alignment << PAGE_SHIFT) / RADEON_GPU_PAGE_SIZE; } @@ -189,7 +189,7 @@ static inline void *radeon_sa_bo_cpu_addr(struct drm_suballoc *sa_bo) extern int radeon_sa_bo_manager_init(struct radeon_device *rdev, struct radeon_sa_manager *sa_manager, - unsigned size, u32 align, u32 domain, + unsigned int size, u32 align, u32 domain, u32 flags); extern void radeon_sa_bo_manager_fini(struct radeon_device *rdev, struct radeon_sa_manager *sa_manager); -- 2.17.1 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] drm/radeon: Prefer 'unsigned int' to bare use of 'unsigned' 2023-07-28 14:35 [PATCH] drm/radeon: Prefer 'unsigned int' to bare use of 'unsigned' 孙冉 @ 2023-07-29 14:12 ` Bagas Sanjaya 2023-07-29 19:33 ` Nathan Chancellor 0 siblings, 1 reply; 3+ messages in thread From: Bagas Sanjaya @ 2023-07-29 14:12 UTC (permalink / raw) To: 孙冉, airlied, daniel, alexander.deucher Cc: amd-gfx, dri-devel, linux-kernel [-- Attachment #1: Type: text/plain, Size: 2192 bytes --] On Fri, Jul 28, 2023 at 10:35:19PM +0800, 孙冉 wrote: > WARNING: Prefer 'unsigned int' to bare use of 'unsigned' > > Signed-off-by: Ran Sun <sunran001@208suo.com> Your From: address != SoB identity > --- > drivers/gpu/drm/radeon/radeon_object.h | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/radeon/radeon_object.h b/drivers/gpu/drm/radeon/radeon_object.h > index 39cc87a59a9a..9b55a7103cfd 100644 > --- a/drivers/gpu/drm/radeon/radeon_object.h > +++ b/drivers/gpu/drm/radeon/radeon_object.h > @@ -37,7 +37,7 @@ > * > * Returns corresponding domain of the ttm mem_type > */ > -static inline unsigned radeon_mem_type_to_domain(u32 mem_type) > +static inline unsigned int radeon_mem_type_to_domain(u32 mem_type) > { > switch (mem_type) { > case TTM_PL_VRAM: > @@ -112,12 +112,12 @@ static inline unsigned long radeon_bo_size(struct radeon_bo *bo) > return bo->tbo.base.size; > } > > -static inline unsigned radeon_bo_ngpu_pages(struct radeon_bo *bo) > +static inline unsigned int radeon_bo_ngpu_pages(struct radeon_bo *bo) > { > return bo->tbo.base.size / RADEON_GPU_PAGE_SIZE; > } > > -static inline unsigned radeon_bo_gpu_page_alignment(struct radeon_bo *bo) > +static inline unsigned int radeon_bo_gpu_page_alignment(struct radeon_bo *bo) > { > return (bo->tbo.page_alignment << PAGE_SHIFT) / RADEON_GPU_PAGE_SIZE; > } > @@ -189,7 +189,7 @@ static inline void *radeon_sa_bo_cpu_addr(struct drm_suballoc *sa_bo) > > extern int radeon_sa_bo_manager_init(struct radeon_device *rdev, > struct radeon_sa_manager *sa_manager, > - unsigned size, u32 align, u32 domain, > + unsigned int size, u32 align, u32 domain, > u32 flags); > extern void radeon_sa_bo_manager_fini(struct radeon_device *rdev, > struct radeon_sa_manager *sa_manager); The patch is whitespace-corrupted. Use git-send-email(1) to submit patches. Also, your patch is also MIME-encoded, hence the corruption. To Alex: Please don't apply this patch due to reasons above. Thanks. -- An old man doll... just what I always wanted! - Clara [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] drm/radeon: Prefer 'unsigned int' to bare use of 'unsigned' 2023-07-29 14:12 ` Bagas Sanjaya @ 2023-07-29 19:33 ` Nathan Chancellor 0 siblings, 0 replies; 3+ messages in thread From: Nathan Chancellor @ 2023-07-29 19:33 UTC (permalink / raw) To: Bagas Sanjaya Cc: 孙冉, airlied, daniel, alexander.deucher, amd-gfx, dri-devel, linux-kernel On Sat, Jul 29, 2023 at 09:12:05PM +0700, Bagas Sanjaya wrote: > On Fri, Jul 28, 2023 at 10:35:19PM +0800, 孙冉 wrote: > > WARNING: Prefer 'unsigned int' to bare use of 'unsigned' > > > > Signed-off-by: Ran Sun <sunran001@208suo.com> > > Your From: address != SoB identity While the comment below is a completely valid complaint, I think this comment is being rather pedantic. Google Translate will tell you that 孙冉 is Sun Ran, so while the name does not strictly match, it is clearly the same... > > --- > > drivers/gpu/drm/radeon/radeon_object.h | 8 ++++---- > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > > diff --git a/drivers/gpu/drm/radeon/radeon_object.h b/drivers/gpu/drm/radeon/radeon_object.h > > index 39cc87a59a9a..9b55a7103cfd 100644 > > --- a/drivers/gpu/drm/radeon/radeon_object.h > > +++ b/drivers/gpu/drm/radeon/radeon_object.h > > @@ -37,7 +37,7 @@ > > * > > * Returns corresponding domain of the ttm mem_type > > */ > > -static inline unsigned radeon_mem_type_to_domain(u32 mem_type) > > +static inline unsigned int radeon_mem_type_to_domain(u32 mem_type) > > { > > switch (mem_type) { > > case TTM_PL_VRAM: > > @@ -112,12 +112,12 @@ static inline unsigned long radeon_bo_size(struct radeon_bo *bo) > > return bo->tbo.base.size; > > } > > > > -static inline unsigned radeon_bo_ngpu_pages(struct radeon_bo *bo) > > +static inline unsigned int radeon_bo_ngpu_pages(struct radeon_bo *bo) > > { > > return bo->tbo.base.size / RADEON_GPU_PAGE_SIZE; > > } > > > > -static inline unsigned radeon_bo_gpu_page_alignment(struct radeon_bo *bo) > > +static inline unsigned int radeon_bo_gpu_page_alignment(struct radeon_bo *bo) > > { > > return (bo->tbo.page_alignment << PAGE_SHIFT) / RADEON_GPU_PAGE_SIZE; > > } > > @@ -189,7 +189,7 @@ static inline void *radeon_sa_bo_cpu_addr(struct drm_suballoc *sa_bo) > > > > extern int radeon_sa_bo_manager_init(struct radeon_device *rdev, > > struct radeon_sa_manager *sa_manager, > > - unsigned size, u32 align, u32 domain, > > + unsigned int size, u32 align, u32 domain, > > u32 flags); > > extern void radeon_sa_bo_manager_fini(struct radeon_device *rdev, > > struct radeon_sa_manager *sa_manager); > > The patch is whitespace-corrupted. Use git-send-email(1) to submit patches. > Also, your patch is also MIME-encoded, hence the corruption. > > To Alex: Please don't apply this patch due to reasons above. > > Thanks. > > -- > An old man doll... just what I always wanted! - Clara ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-07-29 19:33 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-07-28 14:35 [PATCH] drm/radeon: Prefer 'unsigned int' to bare use of 'unsigned' 孙冉 2023-07-29 14:12 ` Bagas Sanjaya 2023-07-29 19:33 ` Nathan Chancellor
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox