Commit df8d0a55 authored by Martin Kaiser's avatar Martin Kaiser Committed by Greg Kroah-Hartman
Browse files

staging: r8188eu: make power sequences static 



Simplify the handling of "power sequences". At the moment, this is rather
complicated:
The hal layer calls HalPwrSeqCmdParsing. This function takes a pointer to
a power sequence. The sequences themselves are defined as global variables
in a separate file. There's an alias for each sequence, the callers of
HalPwrSeqCmdParsing use these aliases instead of the sequences themselves.

There's no point in passing the sequences around. We can move the
sequences into the same file as the HalPwrSeqCmdParsing function where
they are used. Callers of HalPwrSeqCmdParsing can refer to a sequence by
using a numeric define rather than a pointer to the sequence.

Signed-off-by: default avatarMartin Kaiser <martin@kaiser.cx>
Link: https://lore.kernel.org/r/20220627085354.28849-1-martin@kaiser.cx


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 1b0a54b3

drivers/staging/r8188eu/Makefile

+0 −1
Original line number Diff line number Diff line
@@ -5,7 +5,6 @@ r8188eu-y = \ 
		hal/HalHWImg8188E_RF.o \

		hal/HalPhyRf_8188e.o \

		hal/HalPwrSeqCmd.o \

		hal/Hal8188EPwrSeq.o \

		hal/Hal8188ERateAdaptive.o \

		hal/hal_intf.o \

		hal/hal_com.o \

drivers/staging/r8188eu/hal/Hal8188EPwrSeq.c

deleted100644 → 0
+0 −45
Original line number Diff line number Diff line 
// SPDX-License-Identifier: GPL-2.0

/* Copyright(c) 2007 - 2011 Realtek Corporation. */



#include "../include/Hal8188EPwrSeq.h"

#include "../include/rtl8188e_hal.h"



struct wl_pwr_cfg rtl8188E_power_on_flow[] = {

	{ 0x0006, PWR_CMD_POLLING, BIT(1), BIT(1) },

	{ 0x0002, PWR_CMD_WRITE, BIT(0) | BIT(1), 0 }, /* reset BB */

	{ 0x0026, PWR_CMD_WRITE, BIT(7), BIT(7) }, /* schmitt trigger */

	{ 0x0005, PWR_CMD_WRITE, BIT(7), 0 }, /* disable HWPDN (control by DRV)*/

	{ 0x0005, PWR_CMD_WRITE, BIT(4) | BIT(3), 0 }, /* disable WL suspend*/

	{ 0x0005, PWR_CMD_WRITE, BIT(0), BIT(0) },

	{ 0x0005, PWR_CMD_POLLING, BIT(0), 0 },

	{ 0x0023, PWR_CMD_WRITE, BIT(4), 0 },

	{ 0xFFFF, PWR_CMD_END, 0, 0 },

};



struct wl_pwr_cfg rtl8188E_card_disable_flow[] = {

	{ 0x001F, PWR_CMD_WRITE, 0xFF, 0 }, /* turn off RF */

	{ 0x0023, PWR_CMD_WRITE, BIT(4), BIT(4) }, /* LDO Sleep mode */

	{ 0x0005, PWR_CMD_WRITE, BIT(1), BIT(1) }, /* turn off MAC by HW state machine */

	{ 0x0005, PWR_CMD_POLLING, BIT(1), 0 },

	{ 0x0026, PWR_CMD_WRITE, BIT(7), BIT(7) }, /* schmitt trigger */

	{ 0x0005, PWR_CMD_WRITE, BIT(3) | BIT(4), BIT(3) }, /* enable WL suspend */

	{ 0x0007, PWR_CMD_WRITE, 0xFF, 0 }, /* enable bandgap mbias in suspend */

	{ 0x0041, PWR_CMD_WRITE, BIT(4), 0 }, /* Clear SIC_EN register */

	{ 0xfe10, PWR_CMD_WRITE, BIT(4), BIT(4) }, /* Set USB suspend enable local register */

	{ 0xFFFF, PWR_CMD_END, 0, 0 },

};



/* This is used by driver for LPSRadioOff Procedure, not for FW LPS Step */

struct wl_pwr_cfg rtl8188E_enter_lps_flow[] = {

	{ 0x0522, PWR_CMD_WRITE, 0xFF, 0x7F },/* Tx Pause */

	{ 0x05F8, PWR_CMD_POLLING, 0xFF, 0 }, /* Should be zero if no packet is transmitted */

	{ 0x05F9, PWR_CMD_POLLING, 0xFF, 0 }, /* Should be zero if no packet is transmitted */

	{ 0x05FA, PWR_CMD_POLLING, 0xFF, 0 }, /* Should be zero if no packet is transmitted */

	{ 0x05FB, PWR_CMD_POLLING, 0xFF, 0 }, /* Should be zero if no packet is transmitted */

	{ 0x0002, PWR_CMD_WRITE, BIT(0), 0 }, /* CCK and OFDM are disabled, clocks are gated */

	{ 0x0002, PWR_CMD_DELAY, 0, PWRSEQ_DELAY_US },

	{ 0x0100, PWR_CMD_WRITE, 0xFF, 0x3F }, /* Reset MAC TRX */

	{ 0x0101, PWR_CMD_WRITE, BIT(1), 0 }, /* check if removed later */

	{ 0x0553, PWR_CMD_WRITE, BIT(5), BIT(5) }, /* Respond TxOK to scheduler */

	{ 0xFFFF, PWR_CMD_END, 0, 0 },

};

drivers/staging/r8188eu/hal/HalPwrSeqCmd.c

+56 −1
Original line number Diff line number Diff line
@@ -3,9 +3,50 @@ 


#include "../include/HalPwrSeqCmd.h"



u8 HalPwrSeqCmdParsing(struct adapter *padapter, struct wl_pwr_cfg pwrseqcmd[])

static struct wl_pwr_cfg rtl8188E_power_on_flow[] = {

	{ 0x0006, PWR_CMD_POLLING, BIT(1), BIT(1) },

	{ 0x0002, PWR_CMD_WRITE, BIT(0) | BIT(1), 0 }, /* reset BB */

	{ 0x0026, PWR_CMD_WRITE, BIT(7), BIT(7) }, /* schmitt trigger */

	{ 0x0005, PWR_CMD_WRITE, BIT(7), 0 }, /* disable HWPDN (control by DRV)*/

	{ 0x0005, PWR_CMD_WRITE, BIT(4) | BIT(3), 0 }, /* disable WL suspend*/

	{ 0x0005, PWR_CMD_WRITE, BIT(0), BIT(0) },

	{ 0x0005, PWR_CMD_POLLING, BIT(0), 0 },

	{ 0x0023, PWR_CMD_WRITE, BIT(4), 0 },

	{ 0xFFFF, PWR_CMD_END, 0, 0 },

};



static struct wl_pwr_cfg rtl8188E_card_disable_flow[] = {

	{ 0x001F, PWR_CMD_WRITE, 0xFF, 0 }, /* turn off RF */

	{ 0x0023, PWR_CMD_WRITE, BIT(4), BIT(4) }, /* LDO Sleep mode */

	{ 0x0005, PWR_CMD_WRITE, BIT(1), BIT(1) }, /* turn off MAC by HW state machine */

	{ 0x0005, PWR_CMD_POLLING, BIT(1), 0 },

	{ 0x0026, PWR_CMD_WRITE, BIT(7), BIT(7) }, /* schmitt trigger */

	{ 0x0005, PWR_CMD_WRITE, BIT(3) | BIT(4), BIT(3) }, /* enable WL suspend */

	{ 0x0007, PWR_CMD_WRITE, 0xFF, 0 }, /* enable bandgap mbias in suspend */

	{ 0x0041, PWR_CMD_WRITE, BIT(4), 0 }, /* Clear SIC_EN register */

	{ 0xfe10, PWR_CMD_WRITE, BIT(4), BIT(4) }, /* Set USB suspend enable local register */

	{ 0xFFFF, PWR_CMD_END, 0, 0 },

};



/* This is used by driver for LPSRadioOff Procedure, not for FW LPS Step */

static struct wl_pwr_cfg rtl8188E_enter_lps_flow[] = {

	{ 0x0522, PWR_CMD_WRITE, 0xFF, 0x7F },/* Tx Pause */

	{ 0x05F8, PWR_CMD_POLLING, 0xFF, 0 }, /* Should be zero if no packet is transmitted */

	{ 0x05F9, PWR_CMD_POLLING, 0xFF, 0 }, /* Should be zero if no packet is transmitted */

	{ 0x05FA, PWR_CMD_POLLING, 0xFF, 0 }, /* Should be zero if no packet is transmitted */

	{ 0x05FB, PWR_CMD_POLLING, 0xFF, 0 }, /* Should be zero if no packet is transmitted */

	{ 0x0002, PWR_CMD_WRITE, BIT(0), 0 }, /* CCK and OFDM are disabled, clocks are gated */

	{ 0x0002, PWR_CMD_DELAY, 0, PWRSEQ_DELAY_US },

	{ 0x0100, PWR_CMD_WRITE, 0xFF, 0x3F }, /* Reset MAC TRX */

	{ 0x0101, PWR_CMD_WRITE, BIT(1), 0 }, /* check if removed later */

	{ 0x0553, PWR_CMD_WRITE, BIT(5), BIT(5) }, /* Respond TxOK to scheduler */

	{ 0xFFFF, PWR_CMD_END, 0, 0 },

};



u8 HalPwrSeqCmdParsing(struct adapter *padapter, enum r8188eu_pwr_seq seq)

{

	struct wl_pwr_cfg pwrcfgcmd = {0};

	struct wl_pwr_cfg *pwrseqcmd;

	u8 poll_bit = false;

	u32 aryidx = 0;

	u8 value = 0;
@@ -14,6 +55,20 @@ u8 HalPwrSeqCmdParsing(struct adapter *padapter, struct wl_pwr_cfg pwrseqcmd[]) 
	u32 max_poll_count = 5000;

	int res;



	switch (seq) {

	case PWR_ON_FLOW:

		pwrseqcmd = rtl8188E_power_on_flow;

		break;

	case DISABLE_FLOW:

		pwrseqcmd = rtl8188E_card_disable_flow;

		break;

	case LPS_ENTER_FLOW:

		pwrseqcmd = rtl8188E_enter_lps_flow;

		break;

	default:

		return false;

	};



	do {

		pwrcfgcmd = pwrseqcmd[aryidx];

drivers/staging/r8188eu/hal/usb_halinit.c

+4 −4
Original line number Diff line number Diff line
@@ -11,7 +11,7 @@ 
#include "../include/rtw_iol.h"

#include "../include/usb_ops.h"

#include "../include/usb_osintf.h"

#include "../include/Hal8188EPwrSeq.h"

#include "../include/HalPwrSeqCmd.h"



static void _ConfigNormalChipOutEP_8188E(struct adapter *adapt, u8 NumOutPipe)

{
@@ -59,7 +59,7 @@ u32 rtl8188eu_InitPowerOn(struct adapter *adapt) 
	if (haldata->bMacPwrCtrlOn)

		return _SUCCESS;



	if (!HalPwrSeqCmdParsing(adapt, Rtl8188E_NIC_PWR_ON_FLOW))

	if (!HalPwrSeqCmdParsing(adapt, PWR_ON_FLOW))

		return _FAIL;



	/*  Enable MAC DMA/WMAC/SCHEDULE/SEC block */
@@ -798,7 +798,7 @@ static void CardDisableRTL8188EU(struct adapter *Adapter) 
	rtw_write8(Adapter, REG_CR, 0x0);



	/*  Run LPS WL RFOFF flow */

	HalPwrSeqCmdParsing(Adapter, Rtl8188E_NIC_LPS_ENTER_FLOW);

	HalPwrSeqCmdParsing(Adapter, LPS_ENTER_FLOW);



	/*  2. 0x1F[7:0] = 0		turn off RF */
@@ -828,7 +828,7 @@ static void CardDisableRTL8188EU(struct adapter *Adapter) 
	rtw_write8(Adapter, REG_32K_CTRL, val8 & (~BIT(0)));



	/*  Card disable power action flow */

	HalPwrSeqCmdParsing(Adapter, Rtl8188E_NIC_DISABLE_FLOW);

	HalPwrSeqCmdParsing(Adapter, DISABLE_FLOW);



	/*  Reset MCU IO Wrapper */

	res = rtw_read8(Adapter, REG_RSV_CTRL + 1, &val8);

drivers/staging/r8188eu/include/Hal8188EPwrSeq.h

deleted100644 → 0
+0 −13
Original line number Diff line number Diff line 
/* SPDX-License-Identifier: GPL-2.0 OR BSD-3-Clause */

/* Copyright(c) 2007 - 2011 Realtek Corporation. */



#ifndef __HAL8188EPWRSEQ_H__

#define __HAL8188EPWRSEQ_H__



#include "HalPwrSeqCmd.h"



extern struct wl_pwr_cfg rtl8188E_power_on_flow[];

extern struct wl_pwr_cfg rtl8188E_card_disable_flow[];

extern struct wl_pwr_cfg rtl8188E_enter_lps_flow[];



#endif /* __HAL8188EPWRSEQ_H__ */