[PATCH 1/5] usb/otg/ulpi: extend the generic ulpi driver.

[Eric Miao]

> +/* ULPI Function Control Register bits */
> +#define ULPI_FC_HS     0               /* Enable HS tcvr */
> +#define ULPI_FC_FS     (0x1 << 0)      /* Enable FS tcvr */
> +#define ULPI_FC_LS     (0x2 << 0)      /* Enable LS tcvr */
> +#define ULPI_FC_FS_LS  (0x3 << 0)      /* Enable FS tcvr for LS packets */
> +#define ULPI_FC_TRM_SEL        (0x1 << 2)      /* Internal pullup and HS termination */
> +#define ULPI_FC_NODRV  (0x1 << 3)      /* Non-Driving Operation */
> +#define ULPI_FC_NONRZI (0x1 << 4)      /* Disable bit-stuff and NRZI encode */
> +#define ULPI_FC_RESET  (0x1 << 5)      /* Reset the UTMI core */
> +#define ULPI_FC_DEFAULT        0x41            /* Function Control Register Default */
> +
> +
> +/* ULPI Interface Register bits */
> +#define ULPI_IC_6PIN   (1 << 0)        /* XCVR 6 serial pin mode */
> +#define ULPI_IC_3PIN   (1 << 1)        /* XCVR 3 serial pin mode */
> +#define ULPI_IC_CARKIT (1 << 2)        /* Carkit mode */
> +#define ULPI_IC_CLKSPND        (1 << 3)        /* Active low clock suspend */
> +#define ULPI_IC_AUTORES        (1 << 4)        /* PHY auto transmit resume signal */
> +#define ULPI_IC_VBUSINV        (1 << 5)        /* Invert the external VBUS indicator */
> +#define ULPI_IC_INDPT  (1 << 6)        /* Indicator Pass Through */
> +#define ULPI_IC_DISPRT (1 << 7)        /* Interface Protect Disable */
> +#define ULPI_IC_DEFAULT        0x0             /* Interface Control Register Default */
> +
>  struct otg_transceiver *otg_ulpi_create(struct otg_io_access_ops *ops,
> -                                       unsigned int flags);
> +                                       u8 fc_flags, u8 ic_flags, u8 otg_flags);
>

Just one comment about the API, I might not know all the detail of these two
FC/IC registers, but maybe it can be even better:

1. separate flags definition from the FC/IC register bits, i.e. sometimes not
all the register bits will be used, and some times a meaningful flag name is
better (though the flag definition can be encoded properly to simplify the
algorithm to map into the actual register bits)

2. extend the original 'flags' variable and bits for the above definitions, so
we can keep the API unchanged.