[U-Boot] [PATCH v2] ulpi: add generic ULPI support header file

Igor Grinberg
2011-11-08T10:52:59Z

On 11/02/11 20:42, Jana Rapava wrote:
> Add ULPI header file needed by Efika USB support patchset and generic
> ULPI support patch, which is to be posted soon.
>
> Signed-off-by: Jana Rapava <fermata7@gmail.com>
> Cc: Marek Vasut <marek.vasut@gmail.com>
> Cc: Remy Bohmer <linux@bohmer.net>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Igor Grinberg <grinberg@compulab.co.il>
> ---
> Changes for v2:
> - whitespace and comment changes
> include/usb/ulpi.h | 204 ++++++++++++++++++++++++++++++++++++++++++++++++++++
> 1 files changed, 204 insertions(+), 0 deletions(-)
> create mode 100644 include/usb/ulpi.h
>
> diff --git a/include/usb/ulpi.h b/include/usb/ulpi.h
> new file mode 100644
> index 0000000..1519cc5
> --- /dev/null
> +++ b/include/usb/ulpi.h
> @@ -0,0 +1,204 @@
> +/*
> + * Copyright (C) 2011 Jana Rapava <fermata7@gmail.com>
> + * Based on:
> + * linux/include/linux/usb/ulpi.h
> + * ULPI defines and function prototypes
> + *
> + * Original Copyrights follow:
> + * Copyright (C) 2010 Nokia Corporation
> + *
> + * This software is distributed under the terms of the GNU General
> + * Public License ("GPL") as published by the Free Software Foundation,
> + * version 2 of that License.
> + */
> +
> +#ifndef __USB_ULPI_H
> +#define __USB_ULPI_H
> +
> +#define ULPI_ID_REGS_COUNT 4
> +#define ULPI_TEST_VALUE 0x55
> +#define ULPI_TIMEOUT 1000 /* some reasonable value */
> +
> +/* ULPI viewport control bits */
> +#define ULPI_WU (1 << 31)
> +#define ULPI_SS (1 << 27)
> +#define ULPI_RWRUN (1 << 30)
> +#define ULPI_RWCTRL (1 << 29)
> +
> +struct ulpi_regs {
> + /* Vendor ID and Product ID: 0x00 - 0x03 Read-only */
> + u8 vendor_id_low;
> + u8 vendor_id_high;
> + u8 product_id_low;
> + u8 product_id_high;
> + /* Function Control: 0x04 - 0x06 Read */
> + u8 function_ctrl_write; /* 0x04 Write */
> + u8 function_ctrl_set; /* 0x05 Set */
> + u8 function_ctrl_clear; /* 0x06 Clear */
> + /* Interface Control: 0x07 - 0x09 Read */
> + u8 iface_ctrl_write; /* 0x07 Write */
> + u8 iface_ctrl_set; /* 0x08 Set */
> + u8 iface_ctrl_clear; /* 0x09 Clear */
> + /* OTG Control: 0x0A - 0x0C Read */
> + u8 otg_ctrl_write; /* 0x0A Write */
> + u8 otg_ctrl_set; /* 0x0B Set */
> + u8 otg_ctrl_clear; /* 0x0C Clear */
> + /* USB Interrupt Enable Rising: 0x0D - 0x0F Read */
> + u8 usb_ie_rising_write; /* 0x0D Write */
> + u8 usb_ie_rising_set; /* 0x0E Set */
> + u8 usb_ie_rising_clear; /* 0x0F Clear */
> + /* USB Interrupt Enable Falling: 0x10 - 0x12 Read */
> + u8 usb_ie_falling_write; /* 0x10 Write */
> + u8 usb_ie_falling_set; /* 0x11 Set */
> + u8 usb_ie_falling_clear; /* 0x12 Clear */
> + /* USB Interrupt Status: 0x13 Read-only */
> + u8 usb_int_status;
> + /* USB Interrupt Latch: 0x14 Read-only with auto-clear */
> + u8 usb_int_latch;
> + /* Debug: 0x15 Read-only */
> + u8 debug;
> + /* Scratch Register: 0x16 - 0x18 Read */
> + u8 scratch_write; /* 0x16 Write */
> + u8 scratch_set; /* 0x17 Set */
> + u8 scratch_clear; /* 0x18 Clear */
> + /*
> + * Optional Carkit registers
> + */
> + /* Carkit Control: 0x19 - 0x1B Read */
> + u8 carkit_ctrl_write; /* 0x19 Write */
> + u8 carkit_ctrl_set; /* 0x1A Set */
> + u8 carkit_ctrl_clear; /* 0x1B Clear */
> + /* Carkit Interrupt Delay: 0x1C Read, Write */
> + u8 carkit_int_delay;
> + /* Carkit Interrupt Enable: 0x1D - 0x1F Read */
> + u8 carkit_ie_write; /* 0x1D Write */
> + u8 carkit_ie_set; /* 0x1E Set */
> + u8 carkit_ie_clear; /* 0x1F Clear */
> + /* Carkit Interrupt Status: 0x20 Read-only */
> + u8 carkit_int_status;
> + /* Carkit Interrupt Latch: 0x21 Read-only with auto-clear */
> + u8 carkit_int_latch;
> + /* Carkit Pulse Control: 0x22 - 0x24 Read */
> + u8 carkit_pulse_ctrl_write; /* 0x22 Write */
> + u8 carkit_pulse_ctrl_set; /* 0x23 Set */
> + u8 carkit_pulse_ctrl_clear; /* 0x24 Clear */
> + /*
> + * Other optional registers
> + */
> + /* Transmit Positive Width: 0x25 Read, Write */
> + u8 transmit_pos_width;
> + /* Transmit Negative Width: 0x26 Read, Write */
> + u8 transmit_neg_width;
> + /* Receive Polarity Recovery: 0x27 Read, Write */
> + u8 recv_pol_recovery;
> + /*
> + * Addresses 0x28 - 0x2E are reserved, so we use offsets
> + * for immediate registers with higher addresses
> + */
> +};

Sorry for not spotting this earlier, but it is confusing
to call the *_write registers that way.
Can those be called without the _write suffix?

> +
> +/* Access Extended Register Set (indicator) */
> +#define ACCESS_EXT_REGS_OFFSET 0x2f /* read-write */
> +/* Vendor-specific */
> +#define VENDOR_SPEC_OFFSET 0x30
> +
> +/*
> + * Extended Register Set
> + *
> + * Addresses 0x00-0x3F map directly to Immediate Register Set.
> + * Addresses 0x40-0x7F are reserved.
> + * Addresses 0x80-0xff are vendor-specific.
> + */
> +#define EXT_VENDOR_SPEC_OFFSET 0x80
> +
> +/*
> + * Register Bits
> + */
> +
> +/* Function Control */
> +#define ULPI_FC_XCVRSEL (1 << 0)
> +#define ULPI_FC_XCVRSEL_MASK (3 << 0)
> +#define ULPI_FC_HIGH_SPEED (0 << 0)
> +#define ULPI_FC_FULL_SPEED (1 << 0)
> +#define ULPI_FC_LOW_SPEED (2 << 0)
> +#define ULPI_FC_FS4LS (3 << 0)
> +#define ULPI_FC_TERMSELECT (1 << 2)
> +#define ULPI_FC_OPMODE (1 << 3)
> +#define ULPI_FC_OPMODE_MASK (3 << 3)
> +#define ULPI_FC_OPMODE_NORMAL (0 << 3)
> +#define ULPI_FC_OPMODE_NONDRIVING (1 << 3)
> +#define ULPI_FC_OPMODE_DISABLE_NRZI (2 << 3)
> +#define ULPI_FC_OPMODE_NOSYNC_NOEOP (3 << 3)
> +#define ULPI_FC_RESET (1 << 5)
> +#define ULPI_FC_SUSPENDM (1 << 6)
> +
> +/* Interface Control */
> +#define ULPI_IFACE_6_PIN_SERIAL_MODE (1 << 0)
> +#define ULPI_IFACE_3_PIN_SERIAL_MODE (1 << 1)
> +#define ULPI_IFACE_CARKITMODE (1 << 2)
> +#define ULPI_IFACE_CLOCKSUSPENDM (1 << 3)
> +#define ULPI_IFACE_AUTORESUME (1 << 4)
> +#define ULPI_IFACE_EXTVBUS_COMPLEMENT (1 << 5)
> +#define ULPI_IFACE_PASSTHRU (1 << 6)
> +#define ULPI_IFACE_PROTECT_IFC_DISABLE (1 << 7)
> +
> +/* OTG Control */
> +#define ULPI_OTG_ID_PULLUP (1 << 0)
> +#define ULPI_OTG_DP_PULLDOWN (1 << 1)
> +#define ULPI_OTG_DM_PULLDOWN (1 << 2)
> +#define ULPI_OTG_DISCHRGVBUS (1 << 3)
> +#define ULPI_OTG_CHRGVBUS (1 << 4)
> +#define ULPI_OTG_DRVVBUS (1 << 5)
> +#define ULPI_OTG_DRVVBUS_EXT (1 << 6)
> +#define ULPI_OTG_EXTVBUSIND (1 << 7)
> +
> +/*
> + * USB Interrupt Enable Rising,
> + * USB Interrupt Enable Falling,
> + * USB Interrupt Status and
> + * USB Interrupt Latch
> + */
> +#define ULPI_INT_HOST_DISCONNECT (1 << 0)
> +#define ULPI_INT_VBUS_VALID (1 << 1)
> +#define ULPI_INT_SESS_VALID (1 << 2)
> +#define ULPI_INT_SESS_END (1 << 3)
> +#define ULPI_INT_IDGRD (1 << 4)
> +
> +/* Debug */
> +#define ULPI_DEBUG_LINESTATE0 (1 << 0)
> +#define ULPI_DEBUG_LINESTATE1 (1 << 1)
> +
> +/* Carkit Control */
> +#define ULPI_CARKIT_CTRL_CARKITPWR (1 << 0)
> +#define ULPI_CARKIT_CTRL_IDGNDDRV (1 << 1)
> +#define ULPI_CARKIT_CTRL_TXDEN (1 << 2)
> +#define ULPI_CARKIT_CTRL_RXDEN (1 << 3)
> +#define ULPI_CARKIT_CTRL_SPKLEFTEN (1 << 4)
> +#define ULPI_CARKIT_CTRL_SPKRIGHTEN (1 << 5)
> +#define ULPI_CARKIT_CTRL_MICEN (1 << 6)
> +
> +/* Carkit Interrupt Enable */
> +#define ULPI_CARKIT_INT_EN_IDFLOAT_RISE (1 << 0)
> +#define ULPI_CARKIT_INT_EN_IDFLOAT_FALL (1 << 1)
> +#define ULPI_CARKIT_INT_EN_CARINTDET (1 << 2)
> +#define ULPI_CARKIT_INT_EN_DP_RISE (1 << 3)
> +#define ULPI_CARKIT_INT_EN_DP_FALL (1 << 4)
> +
> +/*
> + * Carkit Interrupt Status and
> + * Carkit Interrupt Latch
> + */
> +#define ULPI_CARKIT_INT_IDFLOAT (1 << 0)
> +#define ULPI_CARKIT_INT_CARINTDET (1 << 1)
> +#define ULPI_CARKIT_INT_DP (1 << 2)
> +
> +/* Carkit Pulse Control*/
> +#define ULPI_CARKIT_PLS_CTRL_TXPLSEN (1 << 0)
> +#define ULPI_CARKIT_PLS_CTRL_RXPLSEN (1 << 1)
> +#define ULPI_CARKIT_PLS_CTRL_SPKRLEFT_BIASEN (1 << 2)
> +#define ULPI_CARKIT_PLS_CTRL_SPKRRIGHT_BIASEN (1 << 3)
> +
> +void ulpi_write(struct usb_ehci *ehci, u32 reg, u32 value);
> +u32 ulpi_read(struct usb_ehci *ehci, u32 reg);

Also here, this should not be EHCI specific.
I think for now it is fine to have u32 instead of struct usb_ehci here.

> +
> +#endif /* __USB_ULPI_H */

--
Regards,
Igor.