diff mbox series

[3/3] arm: fsl: csu: Reduce size of ns_dev

Message ID 20221011215106.4133222-4-sean.anderson@seco.com
State Awaiting Upstream
Delegated to: Peng Fan
Headers show
Series arm64: layerscape: Various small size reductions for SPL | expand

Commit Message

Sean Anderson Oct. 11, 2022, 9:51 p.m. UTC 
None of the values in this struct are larger than 256, so we can reduce
the members to u8s. This saves around 1K.

Signed-off-by: Sean Anderson <sean.anderson@seco.com>
---

 include/fsl_csu.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Peng Fan (OSS) Oct. 14, 2022, 12:41 a.m. UTC | #1 
On 10/12/2022 5:51 AM, Sean Anderson wrote:
> None of the values in this struct are larger than 256, so we can reduce
> the members to u8s. This saves around 1K.
> 
> Signed-off-by: Sean Anderson <sean.anderson@seco.com>

Reviewed-by: Peng Fan <peng.fan@nxp.com>


> ---
> 
>   include/fsl_csu.h | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/fsl_csu.h b/include/fsl_csu.h
> index 0e59ac3c324..40c29687979 100644
> --- a/include/fsl_csu.h
> +++ b/include/fsl_csu.h
> @@ -24,8 +24,8 @@ enum csu_cslx_access {
>   };
>   
>   struct csu_ns_dev {
> -	unsigned long ind;
> -	uint32_t val;
> +	u8 ind;
> +	u8 val;
>   };
>   
>   void enable_layerscape_ns_access(void);
diff mbox series

Patch

diff --git a/include/fsl_csu.h b/include/fsl_csu.h
index 0e59ac3c324..40c29687979 100644
--- a/include/fsl_csu.h
+++ b/include/fsl_csu.h
@@ -24,8 +24,8 @@  enum csu_cslx_access {
 };
 
 struct csu_ns_dev {
-	unsigned long ind;
-	uint32_t val;
+	u8 ind;
+	u8 val;
 };
 
 void enable_layerscape_ns_access(void);