Re: [RFC 07/12] omap: mcbsp: Get rid of remaining is_omap tests

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Jarkko,

On Fri, Jul 1, 2011 at 14:22, Jarkko Nikula <jhnikula@xxxxxxxxx> wrote:
> We can get rid of remaining is_omap tests in the driver by using
> mcbsp_config_type that is set in omap hwmod data for 2430, 34xx and 44xx.
>
> Signed-off-by: Jarkko Nikula <jhnikula@xxxxxxxxx>
> ---
>  arch/arm/plat-omap/mcbsp.c |   30 ++++++++++++++----------------
>  1 files changed, 14 insertions(+), 16 deletions(-)
>
> diff --git a/arch/arm/plat-omap/mcbsp.c b/arch/arm/plat-omap/mcbsp.c
> index 92d6d4f..47720b1 100644
> --- a/arch/arm/plat-omap/mcbsp.c
> +++ b/arch/arm/plat-omap/mcbsp.c
> @@ -193,7 +193,7 @@ void omap_mcbsp_config(unsigned int id, const struct omap_mcbsp_reg_cfg *config)
>        MCBSP_WRITE(mcbsp, MCR2, config->mcr2);
>        MCBSP_WRITE(mcbsp, MCR1, config->mcr1);
>        MCBSP_WRITE(mcbsp, PCR0, config->pcr0);
> -       if (cpu_is_omap2430() || cpu_is_omap34xx() || cpu_is_omap44xx()) {
> +       if (mcbsp->mcbsp_config_type >= MCBSP_CONFIG_TYPE2) {
>                MCBSP_WRITE(mcbsp, XCCR, config->xccr);
>                MCBSP_WRITE(mcbsp, RCCR, config->rccr);
>        }
> @@ -531,14 +531,13 @@ void omap_mcbsp_set_tx_threshold(unsigned int id, u16 threshold)
>  {
>        struct omap_mcbsp *mcbsp;
>
> -       if (!cpu_is_omap34xx() && !cpu_is_omap44xx())
> -               return;
> -
>        if (!omap_mcbsp_check_valid_id(id)) {
>                printk(KERN_ERR "%s: Invalid id (%d)\n", __func__, id + 1);
>                return;
>        }
>        mcbsp = id_to_mcbsp_ptr(id);
> +       if (mcbsp->mcbsp_config_type < MCBSP_CONFIG_TYPE3)
> +               return;
>
>        if (threshold && threshold <= mcbsp->max_tx_thres)
>                MCBSP_WRITE(mcbsp, THRSH2, threshold - 1);
> @@ -554,14 +553,13 @@ void omap_mcbsp_set_rx_threshold(unsigned int id, u16 threshold)
>  {
>        struct omap_mcbsp *mcbsp;
>
> -       if (!cpu_is_omap34xx() && !cpu_is_omap44xx())
> -               return;
> -
>        if (!omap_mcbsp_check_valid_id(id)) {
>                printk(KERN_ERR "%s: Invalid id (%d)\n", __func__, id + 1);
>                return;
>        }
>        mcbsp = id_to_mcbsp_ptr(id);
> +       if (mcbsp->mcbsp_config_type < MCBSP_CONFIG_TYPE3)
> +               return;
>
>        if (threshold && threshold <= mcbsp->max_rx_thres)
>                MCBSP_WRITE(mcbsp, THRSH1, threshold - 1);
> @@ -698,7 +696,7 @@ static inline void omap34xx_mcbsp_request(struct omap_mcbsp *mcbsp)
>         * Enable wakup behavior, smart idle and all wakeups
>         * REVISIT: some wakeups may be unnecessary
>         */
> -       if (cpu_is_omap34xx() || cpu_is_omap44xx()) {
> +       if (mcbsp->mcbsp_config_type >= MCBSP_CONFIG_TYPE3) {
>                MCBSP_WRITE(mcbsp, WAKEUPEN, XRDYEN | RRDYEN);
>        }
>  }
> @@ -712,7 +710,7 @@ static inline void omap34xx_mcbsp_free(struct omap_mcbsp *mcbsp)
>        /*
>         * Disable wakup behavior, smart idle and all wakeups
>         */
> -       if (cpu_is_omap34xx() || cpu_is_omap44xx()) {
> +       if (mcbsp->mcbsp_config_type >= MCBSP_CONFIG_TYPE3) {


I think that it would be a good idea to not use
MCBSP_CONFIG_TYPE* in plat-omap/* as it is the .rev info.
At later point of time, this might make the code look uglier
as it happened with GPIO_METHOD_* flags in OMAP GPIO driver.
Instead use this .rev info in mach-omap*/ to fill the pdata and pass it
to driver.

-V Charulatha

>                /*
>                 * HW bug workaround - If no_idle mode is taken, we need to
>                 * go to smart_idle before going to always_idle, or the
> @@ -872,7 +870,7 @@ void omap_mcbsp_start(unsigned int id, int tx, int rx)
>        }
>        mcbsp = id_to_mcbsp_ptr(id);
>
> -       if (cpu_is_omap34xx())
> +       if (mcbsp->mcbsp_config_type == MCBSP_CONFIG_TYPE3)
>                omap_st_start(mcbsp);
>
>        /* Only enable SRG, if McBSP is master */
> @@ -910,7 +908,7 @@ void omap_mcbsp_start(unsigned int id, int tx, int rx)
>                MCBSP_WRITE(mcbsp, SPCR2, w | (1 << 7));
>        }
>
> -       if (cpu_is_omap2430() || cpu_is_omap34xx() || cpu_is_omap44xx()) {
> +       if (mcbsp->mcbsp_config_type >= MCBSP_CONFIG_TYPE2) {
>                /* Release the transmitter and receiver */
>                w = MCBSP_READ_CACHE(mcbsp, XCCR);
>                w &= ~(tx ? XDISABLE : 0);
> @@ -940,7 +938,7 @@ void omap_mcbsp_stop(unsigned int id, int tx, int rx)
>
>        /* Reset transmitter */
>        tx &= 1;
> -       if (cpu_is_omap2430() || cpu_is_omap34xx() || cpu_is_omap44xx()) {
> +       if (mcbsp->mcbsp_config_type >= MCBSP_CONFIG_TYPE2) {
>                w = MCBSP_READ_CACHE(mcbsp, XCCR);
>                w |= (tx ? XDISABLE : 0);
>                MCBSP_WRITE(mcbsp, XCCR, w);
> @@ -950,7 +948,7 @@ void omap_mcbsp_stop(unsigned int id, int tx, int rx)
>
>        /* Reset receiver */
>        rx &= 1;
> -       if (cpu_is_omap2430() || cpu_is_omap34xx() || cpu_is_omap44xx()) {
> +       if (mcbsp->mcbsp_config_type >= MCBSP_CONFIG_TYPE2) {
>                w = MCBSP_READ_CACHE(mcbsp, RCCR);
>                w |= (rx ? RDISABLE : 0);
>                MCBSP_WRITE(mcbsp, RCCR, w);
> @@ -967,7 +965,7 @@ void omap_mcbsp_stop(unsigned int id, int tx, int rx)
>                MCBSP_WRITE(mcbsp, SPCR2, w & ~(1 << 6));
>        }
>
> -       if (cpu_is_omap34xx())
> +       if (mcbsp->mcbsp_config_type == MCBSP_CONFIG_TYPE3)
>                omap_st_stop(mcbsp);
>  }
>  EXPORT_SYMBOL(omap_mcbsp_stop);
> @@ -1230,7 +1228,7 @@ static void __devexit omap_st_remove(struct omap_mcbsp *mcbsp)
>  static inline void __devinit omap34xx_device_init(struct omap_mcbsp *mcbsp)
>  {
>        mcbsp->dma_op_mode = MCBSP_DMA_MODE_ELEMENT;
> -       if (cpu_is_omap34xx()) {
> +       if (mcbsp->mcbsp_config_type == MCBSP_CONFIG_TYPE3) {
>                /*
>                 * Initially configure the maximum thresholds to a safe value.
>                 * The McBSP FIFO usage with these values should not go under
> @@ -1262,7 +1260,7 @@ static inline void __devinit omap34xx_device_init(struct omap_mcbsp *mcbsp)
>
>  static inline void __devexit omap34xx_device_exit(struct omap_mcbsp *mcbsp)
>  {
> -       if (cpu_is_omap34xx()) {
> +       if (mcbsp->mcbsp_config_type == MCBSP_CONFIG_TYPE3) {
>                omap_additional_remove(mcbsp->dev);
>
>                if (mcbsp->id == 2 || mcbsp->id == 3)
> --
> 1.7.0.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux