diff options
author | Colin Ian King <colin.king@canonical.com> | 2019-02-19 14:21:20 +0000 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2019-02-19 14:15:40 -0800 |
commit | 58066ac9d7f5dcde4ef08c03b7e127f0522d9ea0 (patch) | |
tree | 6751fcd5a6c6ef8d81726d12bc0c62a6ee63949a /include | |
parent | d2cf821ff677d3108c783f1b41f558069c955c82 (diff) | |
download | linux-58066ac9d7f5dcde4ef08c03b7e127f0522d9ea0.tar.bz2 |
ptp_qoriq: don't pass a large struct by value but instead pass it by reference
Passing the struct ptp_clock_info caps by parameter is passing over 130 bytes
of data by value on the stack. Optimize this by passing it by reference instead.
Also shinks the object code size:
Before:
text data bss dec hex filename
12596 2160 64 14820 39e4 drivers/ptp/ptp_qoriq.o
After:
text data bss dec hex filename
12567 2160 64 14791 39c7 drivers/ptp/ptp_qoriq.o
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/fsl/ptp_qoriq.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/fsl/ptp_qoriq.h b/include/linux/fsl/ptp_qoriq.h index f127adb71041..992bf9fa1729 100644 --- a/include/linux/fsl/ptp_qoriq.h +++ b/include/linux/fsl/ptp_qoriq.h @@ -183,7 +183,7 @@ static inline void qoriq_write_le(unsigned __iomem *addr, u32 val) irqreturn_t ptp_qoriq_isr(int irq, void *priv); int ptp_qoriq_init(struct ptp_qoriq *ptp_qoriq, void __iomem *base, - const struct ptp_clock_info caps); + const struct ptp_clock_info *caps); void ptp_qoriq_free(struct ptp_qoriq *ptp_qoriq); int ptp_qoriq_adjfine(struct ptp_clock_info *ptp, long scaled_ppm); int ptp_qoriq_adjtime(struct ptp_clock_info *ptp, s64 delta); |