Skip to content

Commit

Permalink
drm/ast: Rework definition of I/O read and write helpers
Browse files Browse the repository at this point in the history
Ast defines a number of I/O helpers for accessing hardware. Only 4 of
the many generated functions are actually used. Replace the respective
generator macros with those 4 functions. No functional changes.

Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230221155745.27484-3-tzimmermann@suse.de
  • Loading branch information
Thomas Zimmermann committed Mar 1, 2023
1 parent a384108 commit faf67f6
Showing 1 changed file with 14 additions and 34 deletions.
48 changes: 14 additions & 34 deletions drivers/gpu/drm/ast/ast_drv.h
Original file line number Diff line number Diff line change
Expand Up @@ -238,45 +238,25 @@ struct ast_private *ast_device_create(const struct drm_driver *drv,
#define AST_IO_VGACRCB_HWC_ENABLED BIT(1)
#define AST_IO_VGACRCB_HWC_16BPP BIT(0) /* set: ARGB4444, cleared: 2bpp palette */

#define __ast_read(x) \
static inline u##x ast_read##x(struct ast_private *ast, u32 reg) { \
u##x val = 0;\
val = ioread##x(ast->regs + reg); \
return val;\
static inline u32 ast_read32(struct ast_private *ast, u32 reg)
{
return ioread32(ast->regs + reg);
}

__ast_read(8);
__ast_read(16);
__ast_read(32)

#define __ast_io_read(x) \
static inline u##x ast_io_read##x(struct ast_private *ast, u32 reg) { \
u##x val = 0;\
val = ioread##x(ast->ioregs + reg); \
return val;\
static inline void ast_write32(struct ast_private *ast, u32 reg, u32 val)
{
iowrite32(val, ast->regs + reg);
}

__ast_io_read(8);
__ast_io_read(16);
__ast_io_read(32);

#define __ast_write(x) \
static inline void ast_write##x(struct ast_private *ast, u32 reg, u##x val) {\
iowrite##x(val, ast->regs + reg);\
}

__ast_write(8);
__ast_write(16);
__ast_write(32);

#define __ast_io_write(x) \
static inline void ast_io_write##x(struct ast_private *ast, u32 reg, u##x val) {\
iowrite##x(val, ast->ioregs + reg);\
}
static inline u8 ast_io_read8(struct ast_private *ast, u32 reg)
{
return ioread8(ast->ioregs + reg);
}

__ast_io_write(8);
__ast_io_write(16);
#undef __ast_io_write
static inline void ast_io_write8(struct ast_private *ast, u32 reg, u8 val)
{
iowrite8(val, ast->ioregs + reg);
}

static inline void ast_set_index_reg(struct ast_private *ast,
uint32_t base, uint8_t index,
Expand Down

0 comments on commit faf67f6

Please sign in to comment.