Re: [PATCH v3 16/19] arch/sh: Implement with generic helpers
On Mon, 2023-04-17 at 14:56 +0200, Thomas Zimmermann wrote: > Replace the architecture's fbdev helpers with the generic > ones from . No functional changes. > > v2: > * use default implementation for fb_pgprotect() (Arnd) > > Signed-off-by: Thomas Zimmermann > Cc: Yoshinori Sato > Cc: Rich Felker > Cc: John Paul Adrian Glaubitz > --- > arch/sh/include/asm/fb.h | 15 +-- > 1 file changed, 1 insertion(+), 14 deletions(-) > > diff --git a/arch/sh/include/asm/fb.h b/arch/sh/include/asm/fb.h > index 9a0bca2686fd..19df13ee9ca7 100644 > --- a/arch/sh/include/asm/fb.h > +++ b/arch/sh/include/asm/fb.h > @@ -2,19 +2,6 @@ > #ifndef _ASM_FB_H_ > #define _ASM_FB_H_ > > -#include > -#include > -#include > - > -static inline void fb_pgprotect(struct file *file, struct vm_area_struct > *vma, > - unsigned long off) > -{ > - vma->vm_page_prot = pgprot_writecombine(vma->vm_page_prot); > -} > - > -static inline int fb_is_primary_device(struct fb_info *info) > -{ > - return 0; > -} > +#include > > #endif /* _ASM_FB_H_ */ Acked-by: John Paul Adrian Glaubitz -- .''`. John Paul Adrian Glaubitz : :' : Debian Developer `. `' Physicist `-GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913
Re: [PATCH v3 16/19] arch/sh: Implement with generic helpers
Hi Am 17.04.23 um 16:13 schrieb John Paul Adrian Glaubitz: Hi Thomas! On Mon, 2023-04-17 at 16:06 +0200, Thomas Zimmermann wrote: Hi Am 17.04.23 um 15:02 schrieb John Paul Adrian Glaubitz: Hi Thomas! On Mon, 2023-04-17 at 14:56 +0200, Thomas Zimmermann wrote: Replace the architecture's fbdev helpers with the generic ones from . No functional changes. v2: * use default implementation for fb_pgprotect() (Arnd) Signed-off-by: Thomas Zimmermann Cc: Yoshinori Sato Cc: Rich Felker Cc: John Paul Adrian Glaubitz --- arch/sh/include/asm/fb.h | 15 +-- 1 file changed, 1 insertion(+), 14 deletions(-) diff --git a/arch/sh/include/asm/fb.h b/arch/sh/include/asm/fb.h index 9a0bca2686fd..19df13ee9ca7 100644 --- a/arch/sh/include/asm/fb.h +++ b/arch/sh/include/asm/fb.h @@ -2,19 +2,6 @@ #ifndef _ASM_FB_H_ #define _ASM_FB_H_ -#include -#include -#include - -static inline void fb_pgprotect(struct file *file, struct vm_area_struct *vma, - unsigned long off) -{ - vma->vm_page_prot = pgprot_writecombine(vma->vm_page_prot); -} Looking at the macro in asm-generic/fb.h, fb_pgprotect() is being replaced with a no-op function. Is that intentional? Can you briefly explain the background for this change? Patch 01 of this patchset changes the generic fb_pgprotect() to set pgprot_writecombine(). So on SH, there should be no change at all. Ah, I missed that, thanks for the explanation. Let me check and Ack your patch Thanks! then. I assume you will be taking this patch as part of the whole series through your own tree? Yes, I'd merge it through drm-misc-next ideally. Best regards Thomas Thanks, Adrian -- Thomas Zimmermann Graphics Driver Developer SUSE Software Solutions Germany GmbH Maxfeldstr. 5, 90409 Nürnberg, Germany (HRB 36809, AG Nürnberg) Geschäftsführer: Ivo Totev OpenPGP_signature Description: OpenPGP digital signature
Re: [PATCH v3 16/19] arch/sh: Implement with generic helpers
Hi Thomas! On Mon, 2023-04-17 at 16:06 +0200, Thomas Zimmermann wrote: > Hi > > Am 17.04.23 um 15:02 schrieb John Paul Adrian Glaubitz: > > Hi Thomas! > > > > On Mon, 2023-04-17 at 14:56 +0200, Thomas Zimmermann wrote: > > > Replace the architecture's fbdev helpers with the generic > > > ones from . No functional changes. > > > > > > v2: > > > * use default implementation for fb_pgprotect() (Arnd) > > > > > > Signed-off-by: Thomas Zimmermann > > > Cc: Yoshinori Sato > > > Cc: Rich Felker > > > Cc: John Paul Adrian Glaubitz > > > --- > > > arch/sh/include/asm/fb.h | 15 +-- > > > 1 file changed, 1 insertion(+), 14 deletions(-) > > > > > > diff --git a/arch/sh/include/asm/fb.h b/arch/sh/include/asm/fb.h > > > index 9a0bca2686fd..19df13ee9ca7 100644 > > > --- a/arch/sh/include/asm/fb.h > > > +++ b/arch/sh/include/asm/fb.h > > > @@ -2,19 +2,6 @@ > > > #ifndef _ASM_FB_H_ > > > #define _ASM_FB_H_ > > > > > > -#include > > > -#include > > > -#include > > > - > > > -static inline void fb_pgprotect(struct file *file, struct vm_area_struct > > > *vma, > > > - unsigned long off) > > > -{ > > > - vma->vm_page_prot = pgprot_writecombine(vma->vm_page_prot); > > > -} > > > > Looking at the macro in asm-generic/fb.h, fb_pgprotect() is being replaced > > with > > a no-op function. Is that intentional? Can you briefly explain the > > background > > for this change? > > Patch 01 of this patchset changes the generic fb_pgprotect() to set > pgprot_writecombine(). So on SH, there should be no change at all. > Ah, I missed that, thanks for the explanation. Let me check and Ack your patch then. I assume you will be taking this patch as part of the whole series through your own tree? Thanks, Adrian -- .''`. John Paul Adrian Glaubitz : :' : Debian Developer `. `' Physicist `-GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913
Re: [PATCH v3 16/19] arch/sh: Implement with generic helpers
Am 17.04.23 um 16:06 schrieb Thomas Zimmermann: Hi Am 17.04.23 um 15:02 schrieb John Paul Adrian Glaubitz: Hi Thomas! On Mon, 2023-04-17 at 14:56 +0200, Thomas Zimmermann wrote: Replace the architecture's fbdev helpers with the generic ones from . No functional changes. v2: * use default implementation for fb_pgprotect() (Arnd) Signed-off-by: Thomas Zimmermann Cc: Yoshinori Sato Cc: Rich Felker Cc: John Paul Adrian Glaubitz --- arch/sh/include/asm/fb.h | 15 +-- 1 file changed, 1 insertion(+), 14 deletions(-) diff --git a/arch/sh/include/asm/fb.h b/arch/sh/include/asm/fb.h index 9a0bca2686fd..19df13ee9ca7 100644 --- a/arch/sh/include/asm/fb.h +++ b/arch/sh/include/asm/fb.h @@ -2,19 +2,6 @@ #ifndef _ASM_FB_H_ #define _ASM_FB_H_ -#include -#include -#include - -static inline void fb_pgprotect(struct file *file, struct vm_area_struct *vma, - unsigned long off) -{ - vma->vm_page_prot = pgprot_writecombine(vma->vm_page_prot); -} Looking at the macro in asm-generic/fb.h, fb_pgprotect() is being replaced with a no-op function. Is that intentional? Can you briefly explain the background for this change? Patch 01 of this patchset changes the generic fb_pgprotect() to set pgprot_writecombine(). So on SH, there should be no change at all. In case you didn't receive that patch: https://patchwork.freedesktop.org/patch/532493/?series=116157=3 Best regards Thomas -static inline int fb_is_primary_device(struct fb_info *info) -{ - return 0; -} +#include #endif /* _ASM_FB_H_ */ Thanks, Adrian -- Thomas Zimmermann Graphics Driver Developer SUSE Software Solutions Germany GmbH Maxfeldstr. 5, 90409 Nürnberg, Germany (HRB 36809, AG Nürnberg) Geschäftsführer: Ivo Totev OpenPGP_signature Description: OpenPGP digital signature
Re: [PATCH v3 16/19] arch/sh: Implement with generic helpers
Hi Am 17.04.23 um 15:02 schrieb John Paul Adrian Glaubitz: Hi Thomas! On Mon, 2023-04-17 at 14:56 +0200, Thomas Zimmermann wrote: Replace the architecture's fbdev helpers with the generic ones from . No functional changes. v2: * use default implementation for fb_pgprotect() (Arnd) Signed-off-by: Thomas Zimmermann Cc: Yoshinori Sato Cc: Rich Felker Cc: John Paul Adrian Glaubitz --- arch/sh/include/asm/fb.h | 15 +-- 1 file changed, 1 insertion(+), 14 deletions(-) diff --git a/arch/sh/include/asm/fb.h b/arch/sh/include/asm/fb.h index 9a0bca2686fd..19df13ee9ca7 100644 --- a/arch/sh/include/asm/fb.h +++ b/arch/sh/include/asm/fb.h @@ -2,19 +2,6 @@ #ifndef _ASM_FB_H_ #define _ASM_FB_H_ -#include -#include -#include - -static inline void fb_pgprotect(struct file *file, struct vm_area_struct *vma, - unsigned long off) -{ - vma->vm_page_prot = pgprot_writecombine(vma->vm_page_prot); -} Looking at the macro in asm-generic/fb.h, fb_pgprotect() is being replaced with a no-op function. Is that intentional? Can you briefly explain the background for this change? Patch 01 of this patchset changes the generic fb_pgprotect() to set pgprot_writecombine(). So on SH, there should be no change at all. Best regards Thomas -static inline int fb_is_primary_device(struct fb_info *info) -{ - return 0; -} +#include #endif /* _ASM_FB_H_ */ Thanks, Adrian -- Thomas Zimmermann Graphics Driver Developer SUSE Software Solutions Germany GmbH Maxfeldstr. 5, 90409 Nürnberg, Germany (HRB 36809, AG Nürnberg) Geschäftsführer: Ivo Totev OpenPGP_signature Description: OpenPGP digital signature
Re: [PATCH v3 16/19] arch/sh: Implement with generic helpers
Hi Thomas! On Mon, 2023-04-17 at 14:56 +0200, Thomas Zimmermann wrote: > Replace the architecture's fbdev helpers with the generic > ones from . No functional changes. > > v2: > * use default implementation for fb_pgprotect() (Arnd) > > Signed-off-by: Thomas Zimmermann > Cc: Yoshinori Sato > Cc: Rich Felker > Cc: John Paul Adrian Glaubitz > --- > arch/sh/include/asm/fb.h | 15 +-- > 1 file changed, 1 insertion(+), 14 deletions(-) > > diff --git a/arch/sh/include/asm/fb.h b/arch/sh/include/asm/fb.h > index 9a0bca2686fd..19df13ee9ca7 100644 > --- a/arch/sh/include/asm/fb.h > +++ b/arch/sh/include/asm/fb.h > @@ -2,19 +2,6 @@ > #ifndef _ASM_FB_H_ > #define _ASM_FB_H_ > > -#include > -#include > -#include > - > -static inline void fb_pgprotect(struct file *file, struct vm_area_struct > *vma, > - unsigned long off) > -{ > - vma->vm_page_prot = pgprot_writecombine(vma->vm_page_prot); > -} Looking at the macro in asm-generic/fb.h, fb_pgprotect() is being replaced with a no-op function. Is that intentional? Can you briefly explain the background for this change? > -static inline int fb_is_primary_device(struct fb_info *info) > -{ > - return 0; > -} > +#include > > #endif /* _ASM_FB_H_ */ Thanks, Adrian -- .''`. John Paul Adrian Glaubitz : :' : Debian Developer `. `' Physicist `-GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913
[PATCH v3 16/19] arch/sh: Implement with generic helpers
Replace the architecture's fbdev helpers with the generic ones from . No functional changes. v2: * use default implementation for fb_pgprotect() (Arnd) Signed-off-by: Thomas Zimmermann Cc: Yoshinori Sato Cc: Rich Felker Cc: John Paul Adrian Glaubitz --- arch/sh/include/asm/fb.h | 15 +-- 1 file changed, 1 insertion(+), 14 deletions(-) diff --git a/arch/sh/include/asm/fb.h b/arch/sh/include/asm/fb.h index 9a0bca2686fd..19df13ee9ca7 100644 --- a/arch/sh/include/asm/fb.h +++ b/arch/sh/include/asm/fb.h @@ -2,19 +2,6 @@ #ifndef _ASM_FB_H_ #define _ASM_FB_H_ -#include -#include -#include - -static inline void fb_pgprotect(struct file *file, struct vm_area_struct *vma, - unsigned long off) -{ - vma->vm_page_prot = pgprot_writecombine(vma->vm_page_prot); -} - -static inline int fb_is_primary_device(struct fb_info *info) -{ - return 0; -} +#include #endif /* _ASM_FB_H_ */ -- 2.40.0