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 |
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 --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,
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(-)