Re: [PATCH] net: phy: Remove inline definitions from convinience functions

2022-08-08 Thread Tom Rini
On Sun, Jun 05, 2022 at 03:44:15AM +0300, Ramon Fried wrote: > The convinience functions are not that small and they caused > bloated text segments because of their usage. > There was no need to inline them in the first place, as > they're not part of a fastpath. > > Signed-off-by: Ramon Fried >

Re: [PATCH] net: phy: Remove inline definitions from convinience functions

2022-06-08 Thread Vladimir Oltean
On Sun, Jun 05, 2022 at 03:44:15AM +0300, Ramon Fried wrote: > The convinience functions are not that small and they caused convenience > bloated text segments because of their usage. > There was no need to inline them in the first place, as > they're not part of a fastpath. > > Signed-off-by: R

Re: [PATCH] net: phy: Remove inline definitions from convinience functions

2022-06-06 Thread Ramon Fried
On Sun, Jun 5, 2022 at 9:59 AM Michael Nazzareno Trimarchi wrote: > > Hi Ramon > > Il dom 5 giu 2022, 02:44 Ramon Fried ha scritto: >> >> The convinience functions are not that small and they caused >> bloated text segments because of their usage. >> There was no need to inline them in the first

Re: [PATCH] net: phy: Remove inline definitions from convinience functions

2022-06-05 Thread Marek BehĂșn
On Sun, 5 Jun 2022 03:44:15 +0300 Ramon Fried wrote: > The convinience functions are not that small and they caused > bloated text segments because of their usage. > There was no need to inline them in the first place, as > they're not part of a fastpath. Reviewed-by: Marek BehĂșn

Re: [PATCH] net: phy: Remove inline definitions from convinience functions

2022-06-05 Thread Michael Nazzareno Trimarchi
Hi Ramon Il dom 5 giu 2022, 02:44 Ramon Fried ha scritto: > The convinience functions are not that small and they caused > bloated text segments because of their usage. > There was no need to inline them in the first place, as > they're not part of a fastpath. > Please give numbers here in com

[PATCH] net: phy: Remove inline definitions from convinience functions

2022-06-04 Thread Ramon Fried
The convinience functions are not that small and they caused bloated text segments because of their usage. There was no need to inline them in the first place, as they're not part of a fastpath. Signed-off-by: Ramon Fried --- drivers/net/phy/phy.c | 174 ++ in