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

From: Steve deRosier
Date: Mon Feb 11 2019 - 08:39:37 EST


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