diff mbox series

board: ti: am335x_evm: Remove duplicate setting of bd_ram_ofs member

Message ID 20200604200129.4817-1-trini@konsulko.com
State Accepted
Commit f18f823c130cbfeb3124c1d62baad857fc0e8d77
Headers show
Series board: ti: am335x_evm: Remove duplicate setting of bd_ram_ofs member | expand

Commit Message

Tom Rini June 4, 2020, 8:01 p.m. UTC
With clang we get a report that we are setting this member twice.
Fortunately it is to the same value, so drop the hard-coded value line.

Signed-off-by: Tom Rini <trini at konsulko.com>
---
 board/ti/am335x/board.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Grygorii Strashko June 6, 2020, 3:12 a.m. UTC | #1
On 04/06/2020 23:01, Tom Rini wrote:
> With clang we get a report that we are setting this member twice.
> Fortunately it is to the same value, so drop the hard-coded value line.
> 
> Signed-off-by: Tom Rini <trini at konsulko.com>
> ---
>   board/ti/am335x/board.c | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/board/ti/am335x/board.c b/board/ti/am335x/board.c
> index 4199bee2e64a..123ccaac44e5 100644
> --- a/board/ti/am335x/board.c
> +++ b/board/ti/am335x/board.c
> @@ -912,7 +912,6 @@ struct cpsw_platform_data am335_eth_data = {
>   	.slaves			= 2,
>   	.slave_data		= slave_data,
>   	.ale_entries		= 1024,
> -	.bd_ram_ofs		= 0x2000,
>   	.mac_control		= 0x20,
>   	.active_slave		= 0,
>   	.mdio_base		= 0x4a101000,
> 

Reviewed-by: Grygorii Strashko <grygorii.strashko at ti.com>
diff mbox series

Patch

diff --git a/board/ti/am335x/board.c b/board/ti/am335x/board.c
index 4199bee2e64a..123ccaac44e5 100644
--- a/board/ti/am335x/board.c
+++ b/board/ti/am335x/board.c
@@ -912,7 +912,6 @@  struct cpsw_platform_data am335_eth_data = {
 	.slaves			= 2,
 	.slave_data		= slave_data,
 	.ale_entries		= 1024,
-	.bd_ram_ofs		= 0x2000,
 	.mac_control		= 0x20,
 	.active_slave		= 0,
 	.mdio_base		= 0x4a101000,