Message ID | 20210915111951.12326-1-colin.king@canonical.com (mailing list archive) |
---|---|
State | Accepted |
Commit | 4c0eee50658746b0333d35a75d3db6e0aac08ef9 |
Headers | show |
Series | dmaengine: sh: make array descs static | expand |
Incorrect $SUBJECT, V2 sent On 15/09/2021 12:19, Colin King wrote: > From: Colin Ian King <colin.king@canonical.com> > > Don't populate the read-only array ds_lut on the stack but instead it > static. Also makes the object code smaller by 163 bytes: > > Before: > text data bss dec hex filename > 23508 4796 0 28304 6e90 ./drivers/dma/sh/rz-dmac.o > > After: > text data bss dec hex filename > 23281 4860 0 28141 6ded ./drivers/dma/sh/rz-dmac.o > > (gcc version 11.2.0) > > Signed-off-by: Colin Ian King <colin.king@canonical.com> > --- > drivers/dma/sh/rz-dmac.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/dma/sh/rz-dmac.c b/drivers/dma/sh/rz-dmac.c > index f9f30cbeccbe..005f1a3ff634 100644 > --- a/drivers/dma/sh/rz-dmac.c > +++ b/drivers/dma/sh/rz-dmac.c > @@ -573,7 +573,7 @@ static void rz_dmac_issue_pending(struct dma_chan *chan) > static u8 rz_dmac_ds_to_val_mapping(enum dma_slave_buswidth ds) > { > u8 i; > - const enum dma_slave_buswidth ds_lut[] = { > + static const enum dma_slave_buswidth ds_lut[] = { > DMA_SLAVE_BUSWIDTH_1_BYTE, > DMA_SLAVE_BUSWIDTH_2_BYTES, > DMA_SLAVE_BUSWIDTH_4_BYTES, >
diff --git a/drivers/dma/sh/rz-dmac.c b/drivers/dma/sh/rz-dmac.c index f9f30cbeccbe..005f1a3ff634 100644 --- a/drivers/dma/sh/rz-dmac.c +++ b/drivers/dma/sh/rz-dmac.c @@ -573,7 +573,7 @@ static void rz_dmac_issue_pending(struct dma_chan *chan) static u8 rz_dmac_ds_to_val_mapping(enum dma_slave_buswidth ds) { u8 i; - const enum dma_slave_buswidth ds_lut[] = { + static const enum dma_slave_buswidth ds_lut[] = { DMA_SLAVE_BUSWIDTH_1_BYTE, DMA_SLAVE_BUSWIDTH_2_BYTES, DMA_SLAVE_BUSWIDTH_4_BYTES,