| Message ID | 20230912135050.17155-1-tzimmermann@suse.de (mailing list archive) |
|---|---|
| Headers | show |
| Series | ppc, fbdev: Clean up fbdev mmap helper | expand |
Ping for a review. I'd like to get at least the first two patches into the DRM git tree. The PPC patches could later be merged through another tree. Best regards Thomas Am 12.09.23 um 15:48 schrieb Thomas Zimmermann: > Clean up and rename fb_pgprotect() to work without struct file. Then > refactor the implementation for PowerPC. This change has been discussed > at [1] in the context of refactoring fbdev's mmap code. > > The first two patches update fbdev and replace fbdev's fb_pgprotect() > with pgprot_framebuffer() on all architectures. The new helper's stream- > lined interface enables more refactoring within fbdev's mmap > implementation. > > Patches 3 to 5 adapt PowerPC's internal interfaces to provide > phys_mem_access_prot() that works without struct file. Neither the > architecture code or fbdev helpers need the parameter. > > v4: > * fix commit message (Christophe) > v3: > * rename fb_pgrotect() to pgprot_framebuffer() (Arnd) > v2: > * reorder patches to simplify merging (Michael) > > [1] https://lore.kernel.org/linuxppc-dev/5501ba80-bdb0-6344-16b0-0466a950f82c@suse.com/ > > Thomas Zimmermann (5): > fbdev: Avoid file argument in fb_pgprotect() > fbdev: Replace fb_pgprotect() with pgprot_framebuffer() > arch/powerpc: Remove trailing whitespaces > arch/powerpc: Remove file parameter from phys_mem_access_prot code > arch/powerpc: Call internal __phys_mem_access_prot() in fbdev code > > arch/ia64/include/asm/fb.h | 15 +++++++-------- > arch/m68k/include/asm/fb.h | 19 ++++++++++--------- > arch/mips/include/asm/fb.h | 11 +++++------ > arch/powerpc/include/asm/book3s/pgtable.h | 10 ++++++++-- > arch/powerpc/include/asm/fb.h | 13 +++++-------- > arch/powerpc/include/asm/machdep.h | 13 ++++++------- > arch/powerpc/include/asm/nohash/pgtable.h | 10 ++++++++-- > arch/powerpc/include/asm/pci.h | 4 +--- > arch/powerpc/kernel/pci-common.c | 3 +-- > arch/powerpc/mm/mem.c | 8 ++++---- > arch/sparc/include/asm/fb.h | 15 +++++++++------ > arch/x86/include/asm/fb.h | 10 ++++++---- > arch/x86/video/fbdev.c | 15 ++++++++------- > drivers/video/fbdev/core/fb_chrdev.c | 3 ++- > include/asm-generic/fb.h | 12 ++++++------ > 15 files changed, 86 insertions(+), 75 deletions(-) >
On Tue, Sep 12, 2023, at 09:48, Thomas Zimmermann wrote: > Clean up and rename fb_pgprotect() to work without struct file. Then > refactor the implementation for PowerPC. This change has been discussed > at [1] in the context of refactoring fbdev's mmap code. > > The first two patches update fbdev and replace fbdev's fb_pgprotect() > with pgprot_framebuffer() on all architectures. The new helper's stream- > lined interface enables more refactoring within fbdev's mmap > implementation. > > Patches 3 to 5 adapt PowerPC's internal interfaces to provide > phys_mem_access_prot() that works without struct file. Neither the > architecture code or fbdev helpers need the parameter. > > v4: > * fix commit message (Christophe) > v3: > * rename fb_pgrotect() to pgprot_framebuffer() (Arnd) I had another look today, and everything look good to me now. Whole series Reviewed-by: Arnd Bergmann <arnd@arndb.de>