2019-02-10 19:48:24

by Lubomir Rintel

Subject: [PATCH] libertas_tf: move the banner to a more appropriate place

Also, turn it to a dev_info() to make checkpatch.pl happy.

Signed-off-by: Lubomir Rintel <[email protected]>
---
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

2019-02-11 13:39:39

by Steve deRosier

Subject: Re: [PATCH] libertas_tf: move the banner to a more appropriate place

On Sun, Feb 10, 2019 at 11:48 AM Lubomir Rintel <[email protected]> wrote:
>
> Also, turn it to a dev_info() to make checkpatch.pl happy.
>
> Signed-off-by: Lubomir Rintel <[email protected]>
> ---
> 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 <[email protected]>

2019-02-19 15:11:49

by Kalle Valo

Subject: Re: [PATCH] libertas_tf: move the banner to a more appropriate place

Lubomir Rintel <[email protected]> wrote:

> Also, turn it to a dev_info() to make checkpatch.pl happy.
>
> Signed-off-by: Lubomir Rintel <[email protected]>
> Reviewed-by: Steve deRosier <[email protected]>

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

c7a5682ded7e libertas_tf: move the banner to a more appropriate place

--
https://patchwork.kernel.org/patch/10804877/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches