libertas_tf: move the banner to a more appropriate place
diff mbox series

Message ID 20190210194811.18345-1-lkundrak@v3.sk
State Accepted
Commit c7a5682ded7e68146c00f2bc30020ef0de152ece
Delegated to: Kalle Valo
Headers show
Series
  • libertas_tf: move the banner to a more appropriate place
Related show

Commit Message

Lubomir Rintel Feb. 10, 2019, 7:48 p.m. UTC
Also, turn it to a dev_info() to make checkpatch.pl happy.

Signed-off-by: Lubomir Rintel <lkundrak@v3.sk>
---
 drivers/net/wireless/marvell/libertas_tf/main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Steve deRosier Feb. 11, 2019, 1:38 p.m. UTC | #1
On Sun, Feb 10, 2019 at 11:48 AM Lubomir Rintel <lkundrak@v3.sk> wrote:
>
> Also, turn it to a dev_info() to make checkpatch.pl happy.
>
> Signed-off-by: Lubomir Rintel <lkundrak@v3.sk>
> ---
>  drivers/net/wireless/marvell/libertas_tf/main.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/wireless/marvell/libertas_tf/main.c b/drivers/net/wireless/marvell/libertas_tf/main.c
> index 1d45da187b9b..c71ad87cdc44 100644
> --- a/drivers/net/wireless/marvell/libertas_tf/main.c
> +++ b/drivers/net/wireless/marvell/libertas_tf/main.c
> @@ -318,7 +318,6 @@ static int lbtf_op_start(struct ieee80211_hw *hw)
>                 goto err_prog_firmware;
>         }
>
> -       printk(KERN_INFO "libertastf: Marvell WLAN 802.11 thinfirm adapter\n");
>         lbtf_deb_leave(LBTF_DEB_MACOPS);
>         return 0;
>
> @@ -649,6 +648,7 @@ struct lbtf_private *lbtf_add_card(void *card, struct device *dmdev)
>         if (ieee80211_register_hw(hw))
>                 goto err_init_adapter;
>
> +       dev_info(dmdev, "libertastf: Marvell WLAN 802.11 thinfirm adapter\n");
>         goto done;
>
>  err_init_adapter:
> --
> 2.20.1
>

Reviewed-by: Steve deRosier <derosier@cal-sierra.com>
Kalle Valo Feb. 19, 2019, 3:11 p.m. UTC | #2
Lubomir Rintel <lkundrak@v3.sk> wrote:

> Also, turn it to a dev_info() to make checkpatch.pl happy.
> 
> Signed-off-by: Lubomir Rintel <lkundrak@v3.sk>
> Reviewed-by: Steve deRosier <derosier@cal-sierra.com>

Patch applied to wireless-drivers-next.git, thanks.

c7a5682ded7e libertas_tf: move the banner to a more appropriate place

Patch
diff mbox series

diff --git a/drivers/net/wireless/marvell/libertas_tf/main.c b/drivers/net/wireless/marvell/libertas_tf/main.c
index 1d45da187b9b..c71ad87cdc44 100644
--- a/drivers/net/wireless/marvell/libertas_tf/main.c
+++ b/drivers/net/wireless/marvell/libertas_tf/main.c
@@ -318,7 +318,6 @@  static int lbtf_op_start(struct ieee80211_hw *hw)
 		goto err_prog_firmware;
 	}
 
-	printk(KERN_INFO "libertastf: Marvell WLAN 802.11 thinfirm adapter\n");
 	lbtf_deb_leave(LBTF_DEB_MACOPS);
 	return 0;
 
@@ -649,6 +648,7 @@  struct lbtf_private *lbtf_add_card(void *card, struct device *dmdev)
 	if (ieee80211_register_hw(hw))
 		goto err_init_adapter;
 
+	dev_info(dmdev, "libertastf: Marvell WLAN 802.11 thinfirm adapter\n");
 	goto done;
 
 err_init_adapter: