Re: [PATCH][V2] net: phy: marvell: make some functions static

2017-06-04 Thread David Miller
From: Colin King Date: Fri, 2 Jun 2017 15:13:34 +0100 > From: Colin Ian King > > functions m88e1510_get_temp_critical, m88e1510_set_temp_critical and > m88e1510_get_temp_alarm can be made static as they not need to be > in global scope. > >

Re: [PATCH][V2] net: phy: marvell: make some functions static

2017-06-04 Thread David Miller
From: Colin King Date: Fri, 2 Jun 2017 15:13:34 +0100 > From: Colin Ian King > > functions m88e1510_get_temp_critical, m88e1510_set_temp_critical and > m88e1510_get_temp_alarm can be made static as they not need to be > in global scope. > > Cleans up sparse warnings: > "symbol

Re: [PATCH][V2] net: phy: marvell: make some functions static

2017-06-02 Thread Florian Fainelli
On 06/02/2017 07:13 AM, Colin King wrote: > From: Colin Ian King > > functions m88e1510_get_temp_critical, m88e1510_set_temp_critical and > m88e1510_get_temp_alarm can be made static as they not need to be > in global scope. > > Cleans up sparse warnings: > "symbol

Re: [PATCH][V2] net: phy: marvell: make some functions static

2017-06-02 Thread Florian Fainelli
On 06/02/2017 07:13 AM, Colin King wrote: > From: Colin Ian King > > functions m88e1510_get_temp_critical, m88e1510_set_temp_critical and > m88e1510_get_temp_alarm can be made static as they not need to be > in global scope. > > Cleans up sparse warnings: > "symbol 'm88e1510_get_temp_alarm'

[PATCH][V2] net: phy: marvell: make some functions static

2017-06-02 Thread Colin King
From: Colin Ian King functions m88e1510_get_temp_critical, m88e1510_set_temp_critical and m88e1510_get_temp_alarm can be made static as they not need to be in global scope. Cleans up sparse warnings: "symbol 'm88e1510_get_temp_alarm' was not declared. Should it be

[PATCH][V2] net: phy: marvell: make some functions static

2017-06-02 Thread Colin King
From: Colin Ian King functions m88e1510_get_temp_critical, m88e1510_set_temp_critical and m88e1510_get_temp_alarm can be made static as they not need to be in global scope. Cleans up sparse warnings: "symbol 'm88e1510_get_temp_alarm' was not declared. Should it be static?" "symbol