From owner-svn-src-head@FreeBSD.ORG Fri Dec 23 08:53:23 2011 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 4AAD9106566B; Fri, 23 Dec 2011 08:53:23 +0000 (UTC) (envelope-from adrian@FreeBSD.org) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:4f8:fff6::2c]) by mx1.freebsd.org (Postfix) with ESMTP id 294EE8FC0C; Fri, 23 Dec 2011 08:53:23 +0000 (UTC) Received: from svn.freebsd.org (localhost [127.0.0.1]) by svn.freebsd.org (8.14.4/8.14.4) with ESMTP id pBN8rNB8056419; Fri, 23 Dec 2011 08:53:23 GMT (envelope-from adrian@svn.freebsd.org) Received: (from adrian@localhost) by svn.freebsd.org (8.14.4/8.14.4/Submit) id pBN8rMb3056417; Fri, 23 Dec 2011 08:53:22 GMT (envelope-from adrian@svn.freebsd.org) Message-Id: <201112230853.pBN8rMb3056417@svn.freebsd.org> From: Adrian Chadd Date: Fri, 23 Dec 2011 08:53:22 +0000 (UTC) To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org X-SVN-Group: head MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Cc: Subject: svn commit: r228836 - head/sys/dev/ath/ath_hal/ar5416 X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 23 Dec 2011 08:53:23 -0000 Author: adrian Date: Fri Dec 23 08:53:22 2011 New Revision: 228836 URL: http://svn.freebsd.org/changeset/base/228836 Log: Port over some more GPIO fixes from the atheros reference HAL. * Bring the AR5416 GPIO mux mask code in line with the code from the HAL. * Add HAL_DEBUG_GPIO debugging statements, to track what's going on. * Add Kiwi GPIO specific changes for reading values back. Obtained from: Atheros Modified: head/sys/dev/ath/ath_hal/ar5416/ar5416_gpio.c Modified: head/sys/dev/ath/ath_hal/ar5416/ar5416_gpio.c ============================================================================== --- head/sys/dev/ath/ath_hal/ar5416/ar5416_gpio.c Fri Dec 23 08:50:26 2011 (r228835) +++ head/sys/dev/ath/ath_hal/ar5416/ar5416_gpio.c Fri Dec 23 08:53:22 2011 (r228836) @@ -35,7 +35,10 @@ static void cfgOutputMux(struct ath_hal *ah, uint32_t gpio, uint32_t type) { int addr; - uint32_t gpio_shift, reg; + uint32_t gpio_shift, tmp; + + HALDEBUG(ah, HAL_DEBUG_GPIO, "%s: gpio=%d, type=%d\n", + __func__, gpio, type); /* each MUX controls 6 GPIO pins */ if (gpio > 11) @@ -61,12 +64,17 @@ cfgOutputMux(struct ath_hal *ah, uint32_ * will never be used. So it should be fine that bit 4 won't be * able to recover. */ - reg = OS_REG_READ(ah, addr); - if (addr == AR_GPIO_OUTPUT_MUX1 && !AR_SREV_MERLIN_20_OR_LATER(ah)) - reg = ((reg & 0x1F0) << 1) | (reg & ~0x1F0); - reg &= ~(0x1f << gpio_shift); - reg |= type << gpio_shift; - OS_REG_WRITE(ah, addr, reg); + if (AR_SREV_MERLIN_20_OR_LATER(ah) || + (addr != AR_GPIO_OUTPUT_MUX1)) { + OS_REG_RMW(ah, addr, (type << gpio_shift), + (0x1f << gpio_shift)); + } else { + tmp = OS_REG_READ(ah, addr); + tmp = ((tmp & 0x1F0) << 1) | (tmp & ~0x1F0); + tmp &= ~(0x1f << gpio_shift); + tmp |= type << gpio_shift; + OS_REG_WRITE(ah, addr, tmp); + } } /* @@ -79,12 +87,17 @@ ar5416GpioCfgOutput(struct ath_hal *ah, HALASSERT(gpio < AH_PRIVATE(ah)->ah_caps.halNumGpioPins); + HALDEBUG(ah, HAL_DEBUG_GPIO, + "%s: gpio=%d, type=%d\n", __func__, gpio, type); + /* NB: type maps directly to hardware */ + /* XXX this may not actually be the case, for anything but output */ cfgOutputMux(ah, gpio, type); gpio_shift = gpio << 1; /* 2 bits per output mode */ reg = OS_REG_READ(ah, AR_GPIO_OE_OUT); reg &= ~(AR_GPIO_OE_OUT_DRV << gpio_shift); + /* Always drive, rather than tristate/drive low/drive high */ reg |= AR_GPIO_OE_OUT_DRV_ALL << gpio_shift; OS_REG_WRITE(ah, AR_GPIO_OE_OUT, reg); @@ -101,6 +114,8 @@ ar5416GpioCfgInput(struct ath_hal *ah, u HALASSERT(gpio < AH_PRIVATE(ah)->ah_caps.halNumGpioPins); + HALDEBUG(ah, HAL_DEBUG_GPIO, "%s: gpio=%d\n", __func__, gpio); + /* TODO: configure input mux for AR5416 */ /* If configured as input, set output to tristate */ gpio_shift = gpio << 1; @@ -122,6 +137,8 @@ ar5416GpioSet(struct ath_hal *ah, uint32 uint32_t reg; HALASSERT(gpio < AH_PRIVATE(ah)->ah_caps.halNumGpioPins); + HALDEBUG(ah, HAL_DEBUG_GPIO, + "%s: gpio=%d, val=%d\n", __func__, gpio, val); reg = OS_REG_READ(ah, AR_GPIO_IN_OUT); if (val & 1) @@ -146,6 +163,8 @@ ar5416GpioGet(struct ath_hal *ah, uint32 * Read output value for all gpio's, shift it, * and verify whether the specific bit is set. */ + if (AR_SREV_KIWI_10_OR_LATER(ah)) + bits = MS(OS_REG_READ(ah, AR_GPIO_IN_OUT), AR9287_GPIO_IN_VAL); if (AR_SREV_KITE_10_OR_LATER(ah)) bits = MS(OS_REG_READ(ah, AR_GPIO_IN_OUT), AR9285_GPIO_IN_VAL); else if (AR_SREV_MERLIN_10_OR_LATER(ah)) @@ -165,6 +184,8 @@ ar5416GpioSetIntr(struct ath_hal *ah, u_ uint32_t val, mask; HALASSERT(gpio < AH_PRIVATE(ah)->ah_caps.halNumGpioPins); + HALDEBUG(ah, HAL_DEBUG_GPIO, + "%s: gpio=%d, ilevel=%d\n", __func__, gpio, ilevel); if (ilevel == HAL_GPIO_INTR_DISABLE) { val = MS(OS_REG_READ(ah, AR_INTR_ASYNC_ENABLE),