[U-Boot] [linux-sunxi] [PATCH 1/9] sun6i: Add new p2wi controller driver
Hans de Goede
hdegoede at redhat.com
Wed Nov 5 13:59:32 CET 2014
Hi,
On 11/03/2014 04:59 PM, Chen-Yu Tsai wrote:
> Hi,
>
> On Mon, Nov 3, 2014 at 11:34 PM, Hans de Goede <hdegoede at redhat.com> wrote:
>> From: Oliver Schinagl <oliver at schinagl.nl>
>>
>> The A31 uses a new push-pull two wire interface, which features higher
>> transfer speeds (upto 6 MHz) in theory. While the hardware can burst 8
>> bytes each time, this driver will only see very little use and thus is
>> limited to single byte transmission only.
>>
>> Signed-off-by: Oliver Schinagl <oliver at schinagl.nl>
>> Signed-off-by: Hans de Goede <hdegoede at redhat.com>
>> ---
>> arch/arm/cpu/armv7/sunxi/Makefile | 1 +
>> arch/arm/cpu/armv7/sunxi/p2wi.c | 117 +++++++++++++++++++++++++++
>> arch/arm/include/asm/arch-sunxi/gpio.h | 3 +
>> arch/arm/include/asm/arch-sunxi/p2wi.h | 140 +++++++++++++++++++++++++++++++++
>> 4 files changed, 261 insertions(+)
>> create mode 100644 arch/arm/cpu/armv7/sunxi/p2wi.c
>> create mode 100644 arch/arm/include/asm/arch-sunxi/p2wi.h
>>
>> diff --git a/arch/arm/cpu/armv7/sunxi/Makefile b/arch/arm/cpu/armv7/sunxi/Makefile
>> index 82dbf76..b3a3601 100644
>> --- a/arch/arm/cpu/armv7/sunxi/Makefile
>> +++ b/arch/arm/cpu/armv7/sunxi/Makefile
>> @@ -13,6 +13,7 @@ obj-y += clock.o
>> obj-y += pinmux.o
>> obj-$(CONFIG_MACH_SUN6I) += prcm.o
>> obj-$(CONFIG_MACH_SUN8I) += prcm.o
>> +obj-$(CONFIG_MACH_SUN6I) += p2wi.o
>> obj-$(CONFIG_MACH_SUN4I) += clock_sun4i.o
>> obj-$(CONFIG_MACH_SUN5I) += clock_sun4i.o
>> obj-$(CONFIG_MACH_SUN6I) += clock_sun6i.o
>> diff --git a/arch/arm/cpu/armv7/sunxi/p2wi.c b/arch/arm/cpu/armv7/sunxi/p2wi.c
>> new file mode 100644
>> index 0000000..10df348
>> --- /dev/null
>> +++ b/arch/arm/cpu/armv7/sunxi/p2wi.c
>> @@ -0,0 +1,117 @@
>> +/*
>> + * Sunxi A31 Power Management Unit
>> + *
>> + * (C) Copyright 2013 Oliver Schinagl <oliver at schinagl.nl>
>> + * http://linux-sunxi.org
>> + *
>> + * Based on sun6i sources and earlier U-Boot Allwiner A10 SPL work
>> + *
>> + * (C) Copyright 2006-2013
>> + * Allwinner Technology Co., Ltd. <www.allwinnertech.com>
>> + * Berg Xing <bergxing at allwinnertech.com>
>> + * Tom Cubie <tangliang at allwinnertech.com>
>> + *
>> + * SPDX-License-Identifier: GPL-2.0+
>> + */
>> +
>> +#include <common.h>
>> +#include <errno.h>
>> +#include <asm/io.h>
>> +#include <asm/arch/cpu.h>
>> +#include <asm/arch/gpio.h>
>> +#include <asm/arch/p2wi.h>
>> +#include <asm/arch/prcm.h>
>> +#include <asm/arch/clock.h>
>> +#include <asm/arch/sys_proto.h>
>> +
>> +void p2wi_init(void)
>> +{
>> + struct sunxi_p2wi_reg *p2wi = (struct sunxi_p2wi_reg *)SUNXI_P2WI_BASE;
>> +
>> + /* Enable p2wi and PIO clk, and de-assert their resets */
>> + prcm_apb0_enable(PRCM_APB0_GATE_PIO | PRCM_APB0_GATE_P2WI);
>> +
>> + sunxi_gpio_set_cfgpin(SUNXI_GPL(0), SUNXI_GPL0_R_P2WI_SCK);
>> + sunxi_gpio_set_cfgpin(SUNXI_GPL(1), SUNXI_GPL1_R_P2WI_SDA);
>> +
>> + /* Reset p2wi controller and set clock to CLKIN(12)/8 = 1.5 MHz */
>> + writel(P2WI_CTRL_RESET, &p2wi->ctrl);
>> + sdelay(0x100);
>> + writel(P2WI_CC_SDA_OUT_DELAY(1) | P2WI_CC_CLK_DIV(8),
>> + &p2wi->cc);
>> +}
>> +
>> +int p2wi_set_pmu_address(u8 slave_addr, u8 ctrl_reg, u8 init_data)
>
> set_pmu_address is actually a misnomer. What it actually does is
> write <init_data> to <ctrl_reg> on device found at <slave_addr>,
> probably using I2C, with the sole purpose of putting the device
> into P2WI mode (or RSB mode for later AXPs).
>
> According to the docs/code we have, it is always writing 0x3e to
> register 0x3e.
I've renamed this to p2wi_change_to_p2wi_mode for the next version
of this patch-set.
>> +{
>> + struct sunxi_p2wi_reg *p2wi = (struct sunxi_p2wi_reg *)SUNXI_P2WI_BASE;
>> + unsigned long tmo = timer_get_us() + 1000000;
>> +
>> + writel(P2WI_PM_DEV_ADDR(slave_addr) |
>> + P2WI_PM_CTRL_ADDR(ctrl_reg) |
>> + P2WI_PM_INIT_DATA(init_data) |
>> + P2WI_PM_INIT_SEND,
>> + &p2wi->pm);
>> +
>> + while ((readl(&p2wi->pm) & P2WI_PM_INIT_SEND)) {
>> + if (timer_get_us() > tmo)
>> + return -EFAULT;
>> + }
>> +
>> + return 0;
>> +}
>> +
>> +static int p2wi_await_trans(void)
>> +{
>> + struct sunxi_p2wi_reg *p2wi = (struct sunxi_p2wi_reg *)SUNXI_P2WI_BASE;
>> + unsigned long tmo = timer_get_us() + 1000000;
>> + int ret;
>> + u8 reg;
>> +
>> + while (1) {
>> + reg = readl(&p2wi->status);
>> + if (reg & P2WI_STAT_TRANS_ERR) {
>> + ret = -EIO;
>> + break;
>> + }
>> + if (reg & P2WI_STAT_TRANS_DONE) {
>> + ret = 0;
>> + break;
>> + }
>> + if (timer_get_us() > tmo) {
>> + ret = -ETIME;
>> + break;
>> + }
>> + }
>> + writel(reg, &p2wi->status); /* Clear status bits */
>> + return ret;
>> +}
>> +
>> +int p2wi_read(const u8 addr, u8 *data)
>> +{
>> + struct sunxi_p2wi_reg *p2wi = (struct sunxi_p2wi_reg *)SUNXI_P2WI_BASE;
>> + int ret;
>> +
>> + writel(P2WI_DATADDR_BYTE_1(addr), &p2wi->dataddr0);
>> + writel(P2WI_DATA_NUM_BYTES(1) |
>> + P2WI_DATA_NUM_BYTES_READ, &p2wi->numbytes);
>> + writel(P2WI_STAT_TRANS_DONE, &p2wi->status);
>> + writel(P2WI_CTRL_TRANS_START, &p2wi->ctrl);
>> +
>> + ret = p2wi_await_trans();
>> +
>> + *data = readl(&p2wi->data0) & P2WI_DATA_BYTE_1_MASK;
>> + return ret;
>> +}
>> +
>> +int p2wi_write(const u8 addr, u8 data)
>> +{
>> + struct sunxi_p2wi_reg *p2wi = (struct sunxi_p2wi_reg *)SUNXI_P2WI_BASE;
>> +
>> + writel(P2WI_DATADDR_BYTE_1(addr), &p2wi->dataddr0);
>> + writel(P2WI_DATA_BYTE_1(data), &p2wi->data0);
>> + writel(P2WI_DATA_NUM_BYTES(1), &p2wi->numbytes);
>> + writel(P2WI_STAT_TRANS_DONE, &p2wi->status);
>> + writel(P2WI_CTRL_TRANS_START, &p2wi->ctrl);
>> +
>> + return p2wi_await_trans();
>> +}
>> diff --git a/arch/arm/include/asm/arch-sunxi/gpio.h b/arch/arm/include/asm/arch-sunxi/gpio.h
>> index 437dd35..c734cf0 100644
>> --- a/arch/arm/include/asm/arch-sunxi/gpio.h
>> +++ b/arch/arm/include/asm/arch-sunxi/gpio.h
>> @@ -172,6 +172,9 @@ enum sunxi_gpio_number {
>>
>> #define SUN4I_GPI4_SDC3 2
>>
>> +#define SUNXI_GPL0_R_P2WI_SCK 3
>> +#define SUNXI_GPL1_R_P2WI_SDA 3
>> +
>> #define SUN8I_GPL2_R_UART_TX 2
>> #define SUN8I_GPL3_R_UART_RX 2
>>
>> diff --git a/arch/arm/include/asm/arch-sunxi/p2wi.h b/arch/arm/include/asm/arch-sunxi/p2wi.h
>> new file mode 100644
>> index 0000000..1788fa4
>> --- /dev/null
>> +++ b/arch/arm/include/asm/arch-sunxi/p2wi.h
>> @@ -0,0 +1,140 @@
>> +/*
>> + * Sunxi platform Push-Push i2c register definition.
>> + *
>> + * (c) Copyright 2013 Oliver Schinagl <oliver at schinagl.nl>
>> + * http://linux-sunxi.org
>> + *
>> + * (c)Copyright 2006-2013
>> + * Allwinner Technology Co., Ltd. <www.allwinnertech.com>
>> + * Berg Xing <bergxing at allwinnertech.com>
>> + * Tom Cubie <tangliang at allwinnertech.com>
>> + *
>> + * SPDX-License-Identifier: GPL-2.0+
>> + */
>> +
>> +#ifndef _SUNXI_P2WI_H
>> +#define _SUNXI_P2WI_H
>> +
>> +#include <linux/types.h>
>> +
>> +#define P2WI_CTRL_RESET (0x1 << 0)
>> +#define P2WI_CTRL_IRQ_EN (0x1 << 1)
>> +#define P2WI_CTRL_TRANS_ABORT (0x1 << 6)
>> +#define P2WI_CTRL_TRANS_START (0x1 << 7)
>> +
>> +#define __P2WI_CC_CLK(n) (((n) & 0xff) << 0)
>> +#define P2WI_CC_CLK_MASK __P2WI_CC_CLK_DIV(0xff)
>> +#define __P2WI_CC_CLK_DIV(n) (((n) >> 1) - 1)
>> +#define P2WI_CC_CLK_DIV(n) \
>> + __P2WI_CC_CLK(__P2WI_CC_CLK_DIV(n))
>> +#define P2WI_CC_SDA_OUT_DELAY(n) (((n) & 0x7) << 8)
>> +#define P2WI_CC_SDA_OUT_DELAY_MASK P2WI_CC_SDA_OUT_DELAY(0x7)
>> +
>> +#define P2WI_IRQ_TRANS_DONE (0x1 << 0)
>> +#define P2WI_IRQ_TRANS_ERR (0x1 << 1)
>> +#define P2WI_IRQ_LOAD_BUSY (0x1 << 2)
>> +
>> +#define P2WI_STAT_TRANS_DONE (0x1 << 0)
>> +#define P2WI_STAT_TRANS_ERR (0x1 << 1)
>> +#define P2WI_STAT_LOAD_BUSY (0x1 << 2)
>> +#define __P2WI_STAT_TRANS_ERR(n) (((n) & 0xff) << 8)
>> +#define P2WI_STAT_TRANS_ERR_MASK __P2WI_STAT_TRANS_ERR_ID(0xff)
>> +#define __P2WI_STAT_TRANS_ERR_BYTE_1 0x01
>> +#define __P2WI_STAT_TRANS_ERR_BYTE_2 0x02
>> +#define __P2WI_STAT_TRANS_ERR_BYTE_3 0x04
>> +#define __P2WI_STAT_TRANS_ERR_BYTE_4 0x08
>> +#define __P2WI_STAT_TRANS_ERR_BYTE_5 0x10
>> +#define __P2WI_STAT_TRANS_ERR_BYTE_6 0x20
>> +#define __P2WI_STAT_TRANS_ERR_BYTE_7 0x40
>> +#define __P2WI_STAT_TRANS_ERR_BYTE_8 0x80
>> +#define P2WI_STAT_TRANS_ERR_BYTE_1 \
>> + __P2WI_STAT_TRANS_ERR(__P2WI_STAT_TRANS_ERR_BYTE_1)
>> +#define P2WI_STAT_TRANS_ERR_BYTE_2 \
>> + __P2WI_STAT_TRANS_ERR(__P2WI_STAT_TRANS_ERR_BYTE_2)
>> +#define P2WI_STAT_TRANS_ERR_BYTE_3 \
>> + __P2WI_STAT_TRANS_ERR(__P2WI_STAT_TRANS_ERR_BYTE_3)
>> +#define P2WI_STAT_TRANS_ERR_BYTE_4 \
>> + __P2WI_STAT_TRANS_ERR(__P2WI_STAT_TRANS_ERR_BYTE_4)
>> +#define P2WI_STAT_TRANS_ERR_BYTE_5 \
>> + __P2WI_STAT_TRANS_ERR(__P2WI_STAT_TRANS_ERR_BYTE_5)
>> +#define P2WI_STAT_TRANS_ERR_BYTE_6 \
>> + __P2WI_STAT_TRANS_ERR(__P2WI_STAT_TRANS_ERR_BYTE_6)
>> +#define P2WI_STAT_TRANS_ERR_BYTE_7 \
>> + __P2WI_STAT_TRANS_ERR(__P2WI_STAT_TRANS_ERR_BYTE_7)
>> +#define P2WI_STAT_TRANS_ERR_BYTE_8 \
>> + __P2WI_STAT_TRANS_ERR(__P2WI_STAT_TRANS_ERR_BYTE_8)
>> +
>> +#define P2WI_DATADDR_BYTE_1(n) (((n) & 0xff) << 0)
>> +#define P2WI_DATADDR_BYTE_1_MASK P2WI_DATADDR_BYTE_1(0xff)
>> +#define P2WI_DATADDR_BYTE_2(n) (((n) & 0xff) << 8)
>> +#define P2WI_DATADDR_BYTE_2_MASK P2WI_DATADDR_BYTE_2(0xff)
>> +#define P2WI_DATADDR_BYTE_3(n) (((n) & 0xff) << 16)
>> +#define P2WI_DATADDR_BYTE_3_MASK P2WI_DATADDR_BYTE_3(0xff)
>> +#define P2WI_DATADDR_BYTE_4(n) (((n) & 0xff) << 24)
>> +#define P2WI_DATADDR_BYTE_4_MASK P2WI_DATADDR_BYTE_4(0xff)
>> +#define P2WI_DATADDR_BYTE_5(n) (((n) & 0xff) << 0)
>> +#define P2WI_DATADDR_BYTE_5_MASK P2WI_DATADDR_BYTE_5(0xff)
>> +#define P2WI_DATADDR_BYTE_6(n) (((n) & 0xff) << 8)
>> +#define P2WI_DATADDR_BYTE_6_MASK P2WI_DATADDR_BYTE_6(0xff)
>> +#define P2WI_DATADDR_BYTE_7(n) (((n) & 0xff) << 16)
>> +#define P2WI_DATADDR_BYTE_7_MASK P2WI_DATADDR_BYTE_7(0xff)
>> +#define P2WI_DATADDR_BYTE_8(n) (((n) & 0xff) << 24)
>> +#define P2WI_DATADDR_BYTE_8_MASK P2WI_DATADDR_BYTE_8(0xff)
>> +
>> +#define __P2WI_DATA_NUM_BYTES(n) (((n) & 0x7) << 0)
>> +#define P2WI_DATA_NUM_BYTES_MASK __P2WI_DATA_NUM_BYTES(0x7)
>> +#define P2WI_DATA_NUM_BYTES(n) __P2WI_DATA_NUM_BYTES((n) - 1)
>> +#define P2WI_DATA_NUM_BYTES_READ (0x1 << 4)
>> +
>> +#define P2WI_DATA_BYTE_1(n) (((n) & 0xff) << 0)
>> +#define P2WI_DATA_BYTE_1_MASK P2WI_DATA_BYTE_1(0xff)
>> +#define P2WI_DATA_BYTE_2(n) (((n) & 0xff) << 8)
>> +#define P2WI_DATA_BYTE_2_MASK P2WI_DATA_BYTE_2(0xff)
>> +#define P2WI_DATA_BYTE_3(n) (((n) & 0xff) << 16)
>> +#define P2WI_DATA_BYTE_3_MASK P2WI_DATA_BYTE_3(0xff)
>> +#define P2WI_DATA_BYTE_4(n) (((n) & 0xff) << 24)
>> +#define P2WI_DATA_BYTE_4_MASK P2WI_DATA_BYTE_4(0xff)
>> +#define P2WI_DATA_BYTE_5(n) (((n) & 0xff) << 0)
>> +#define P2WI_DATA_BYTE_5_MASK P2WI_DATA_BYTE_5(0xff)
>> +#define P2WI_DATA_BYTE_6(n) (((n) & 0xff) << 8)
>> +#define P2WI_DATA_BYTE_6_MASK P2WI_DATA_BYTE_6(0xff)
>> +#define P2WI_DATA_BYTE_7(n) (((n) & 0xff) << 16)
>> +#define P2WI_DATA_BYTE_7_MASK P2WI_DATA_BYTE_7(0xff)
>> +#define P2WI_DATA_BYTE_8(n) (((n) & 0xff) << 24)
>> +#define P2WI_DATA_BYTE_8_MASK P2WI_DATA_BYTE_8(0xff)
>> +
>> +#define P2WI_LINECTRL_SDA_CTRL_EN (0x1 << 0)
>> +#define P2WI_LINECTRL_SDA_OUT_HIGH (0x1 << 1)
>> +#define P2WI_LINECTRL_SCL_CTRL_EN (0x1 << 2)
>> +#define P2WI_LINECTRL_SCL_OUT_HIGH (0x1 << 3)
>> +#define P2WI_LINECTRL_SDA_STATE_HIGH (0x1 << 4)
>> +#define P2WI_LINECTRL_SCL_STATE_HIGH (0x1 << 5)
>> +
>> +#define P2WI_PM_DEV_ADDR(n) (((n) & 0xff) << 0)
>> +#define P2WI_PM_DEV_ADDR_MASK P2WI_PM_DEV_ADDR(0xff)
>> +#define P2WI_PM_CTRL_ADDR(n) (((n) & 0xff) << 8)
>> +#define P2WI_PM_CTRL_ADDR_MASK P2WI_PM_CTRL_ADDR(0xff)
>> +#define P2WI_PM_INIT_DATA(n) (((n) & 0xff) << 16)
>> +#define P2WI_PM_INIT_DATA_MASK P2WI_PM_INIT_DATA(0xff)
>> +#define P2WI_PM_INIT_SEND (0x1 << 31)
>> +
>> +struct sunxi_p2wi_reg {
>> + u32 ctrl; /* 0x00 control */
>> + u32 cc; /* 0x04 clock control */
>> + u32 irq; /* 0x08 interrupt */
>> + u32 status; /* 0x0c status */
>> + u32 dataddr0; /* 0x10 data address 0 */
>> + u32 dataddr1; /* 0x14 data address 1 */
>> + u32 numbytes; /* 0x18 num bytes */
>> + u32 data0; /* 0x1c data buffer 0 */
>> + u32 data1; /* 0x20 data buffer 1 */
>> + u32 linectrl; /* 0x24 line control */
>> + u32 pm; /* 0x28 power management */
>> +};
>
> Is there any reason not to put all of the above in the driver (.c) itself?
Not really, this is what all the axpXXX drivers also do, and I prefer it
this way as it keeps the .c file a lot more readable.
>> +
>> +void p2wi_init(void);
>> +int p2wi_set_pmu_address(u8 slave_addr, u8 ctrl_reg, u8 init_data);
>> +int p2wi_read(const u8 addr, u8 *data);
>> +int p2wi_write(const u8 addr, u8 data);
>> +
>> +#endif /* _SUNXI_P2WI_H */
>> --
>> 2.1.0
>
> Thanks for working on this! I haven't had time to mainline the p2wi driver.
You're welcome.
Regards,
Hans
More information about the U-Boot
mailing list