Re: [PATCH v2 1/2] drivers: net: cpsw: Add helper functions for VLAN ALE implementation

2013-01-31 Thread Francois Romieu
Mugunthan V N mugunthan...@ti.com : On 1/31/2013 3:32 AM, Francois Romieu wrote: [...] It could be factored out. Are you mentioning to have static inline function for the above two statements above? Yes. The helper function does not need to be inlined if it does not save space: this path

Re: [PATCH v2 1/2] drivers: net: cpsw: Add helper functions for VLAN ALE implementation

2013-01-30 Thread Francois Romieu
Mugunthan V N mugunthan...@ti.com : [...] diff --git a/drivers/net/ethernet/ti/cpsw_ale.c b/drivers/net/ethernet/ti/cpsw_ale.c index 0e9ccc2..18b88ce 100644 --- a/drivers/net/ethernet/ti/cpsw_ale.c +++ b/drivers/net/ethernet/ti/cpsw_ale.c [...] @@ -274,19 +292,26 @@ int

Re: [PATCH v2 2/2] drivers: net:ethernet: cpsw: add support for VLAN

2013-01-30 Thread Francois Romieu
Mugunthan V N mugunthan...@ti.com : [...] diff --git a/Documentation/devicetree/bindings/net/cpsw.txt b/Documentation/devicetree/bindings/net/cpsw.txt index 6ddd028..99696bf 100644 --- a/Documentation/devicetree/bindings/net/cpsw.txt +++ b/Documentation/devicetree/bindings/net/cpsw.txt @@