Skip to content

Commit 6f3efad

Browse files
committed
dmaengine: sdxi: hw: Remove DSC_OPERATION_BYTES
Seems like unnecessary indirection and isn't used outside a couple descriptor layouts. Signed-off-by: Nathan Lynch <nathan.lynch@amd.com>
1 parent 424a71a commit 6f3efad

1 file changed

Lines changed: 2 additions & 8 deletions

File tree

  • drivers/dma/sdxi

drivers/dma/sdxi/hw.h

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -140,12 +140,6 @@ struct sdxi_cst_blk {
140140
} __packed;
141141
static_assert(sizeof(struct sdxi_cst_blk) == 32);
142142

143-
/*
144-
* Size of the "body" of each descriptor between the common opcode and
145-
* csb_ptr fields.
146-
*/
147-
#define DSC_OPERATION_BYTES 52
148-
149143
#define define_sdxi_dsc(tag_, name_, op_body_) \
150144
struct tag_ { \
151145
__le32 opcode; \
@@ -162,7 +156,7 @@ struct sdxi_desc {
162156
/* DSC_GENERIC - common header and footer */
163157
struct_group_tagged(sdxi_dsc_generic, generic,
164158
__le32 opcode;
165-
__u8 operation[DSC_OPERATION_BYTES];
159+
__u8 operation[52];
166160
__le64 csb_ptr;
167161
);
168162

@@ -185,7 +179,7 @@ struct sdxi_desc {
185179
#define SDXI_DSC_OP_SUBTYPE_NOP 0x01
186180
/* DmaBaseGrp: DSC_DMAB_NOP */
187181
define_sdxi_dsc(sdxi_dsc_dmab_nop, nop,
188-
__u8 rsvd_0[DSC_OPERATION_BYTES];
182+
__u8 rsvd_0[52];
189183
);
190184

191185
#define SDXI_DSC_OP_SUBTYPE_COPY 0x03

0 commit comments

Comments
 (0)