Re: [PATCH/RFC] rapidio: temporarily exclude FSL_RIO from 64 bit builds

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

 



Hi Ben,

Given a week has passed with the absence of any feedback
with respect to SRIO on 64 bit, are you OK with applying the
below patch[1] pretty much as-is?

Thanks,
Paul.

[1] http://patchwork.ozlabs.org/patch/142383/

---

On Tue, Feb 21, 2012 at 6:10 PM, Paul Gortmaker
<paul.gortmaker@xxxxxxxxxxxxx> wrote:
> The following are seen while using the corenet64_smp_defconfig:
>
> arch/powerpc/sysdev/fsl_rmu.c:315: error: cast from pointer to integer of different size
> arch/powerpc/sysdev/fsl_rmu.c:320: error: cast to pointer from integer of different size
> arch/powerpc/sysdev/fsl_rmu.c:320: error: cast to pointer from integer of different size
> arch/powerpc/sysdev/fsl_rmu.c:320: error: cast to pointer from integer of different size
> arch/powerpc/sysdev/fsl_rmu.c:330: error: cast to pointer from integer of different size
> arch/powerpc/sysdev/fsl_rmu.c:332: error: cast to pointer from integer of different size
> arch/powerpc/sysdev/fsl_rmu.c:339: error: cast to pointer from integer of different size
> arch/powerpc/sysdev/fsl_rmu.c:340: error: cast to pointer from integer of different size
> arch/powerpc/sysdev/fsl_rmu.c:341: error: cast to pointer from integer of different size
> arch/powerpc/sysdev/fsl_rmu.c:348: error: cast to pointer from integer of different size
> arch/powerpc/sysdev/fsl_rmu.c:348: error: cast to pointer from integer of different size
> arch/powerpc/sysdev/fsl_rmu.c:348: error: cast to pointer from integer of different size
> arch/powerpc/sysdev/fsl_rmu.c:659: error: cast from pointer to integer of different size
> arch/powerpc/sysdev/fsl_rmu.c:659: error: format '%8.8x' expects type 'unsigned int', but argument 5 has type 'size_t'
> arch/powerpc/sysdev/fsl_rmu.c:985: error: cast from pointer to integer of different size
> arch/powerpc/sysdev/fsl_rmu.c:997: error: cast to pointer from integer of different size
>
> A quick inspection of the code leaves one with the impression
> that it was not explicitly written with 64 bit support in mind,
> so just block that as a possible config selection for now.
>
> Signed-off-by: Paul Gortmaker <paul.gortmaker@xxxxxxxxxxxxx>
>
> ---
>
> [This is seen in linux-next -- not sure if this is the right way to go
>  but at least it will give the issue some visibility to the right ppl.]
>
> diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
> index 75f061e..5ad8013 100644
> --- a/arch/powerpc/Kconfig
> +++ b/arch/powerpc/Kconfig
> @@ -773,7 +773,7 @@ config RAPIDIO
>
>  config FSL_RIO
>        bool "Freescale Embedded SRIO Controller support"
> -       depends on RAPIDIO && HAS_RAPIDIO
> +       depends on RAPIDIO && HAS_RAPIDIO && (BROKEN || !PPC64)
>        default "n"
>        ---help---
>          Include support for RapidIO controller on Freescale embedded
> --
> 1.7.9.1
>
> _______________________________________________
> Linuxppc-dev mailing list
> Linuxppc-dev@xxxxxxxxxxxxxxxx
> https://lists.ozlabs.org/listinfo/linuxppc-dev
--
To unsubscribe from this list: send the line "unsubscribe linux-next" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel]     [Linux USB Development]     [Yosemite News]     [Linux SCSI]

  Powered by Linux