[lkml]   [2011]   [Oct]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 17/26] ARM: pxa: pxa95x is incompatible with earlier pxa
    On Saturday 08 October 2011 11:32:14 Haojian Zhuang wrote:
    > Eric,
    > At first, a new macro (ARCH_PXA_V7) is defined in
    > arch/arm/mach-pxa/Kconfig in this patch.
    > I prefer to move this macro to arch/arm/Kconfig.

    If we move it to arch/arm/Kconfig, I would prefer making it a global option,
    not a pxa specific one. If we introduce a top-level CONFIG_CPU_V6PLUS
    option, we can make a number of decisions inside of Kconfig depend on that,
    especially as we move to allow building multiple v6/v7 platforms together,
    or multiple v5 platforms for that matter. I believe we don't need to
    worry about v5+v7 at this point and can instead assume that won't ever

    > Secondly, pxa95x is both used in saarb and tavorevb3.

    The patch makes that very explicit, doesn't it?

    > Thirdly, PXA_V7_MACH_AUTO is unnecessary. We just need to select those
    > machines in defconfig or define a new DT machine type to select all
    > machines.

    Enabling them in defconfig will not help here, it still allows creating
    an invalid configuration by disabling both saarb and tavorevb3.
    I agree that it would be best to have a single DT machine type that can
    handle both saarb and tavorevb3 as well as any future pxa95x based machines,
    but nobody has implemented that yet. In the meantime, I think we should
    have the PXA_V7_MACH_AUTO or an equivalent mechanism to enforce that at
    least one of the two board files gets built into any kernel. This is mostly
    important to help the 'make randconfig' builds succeed, not for actual
    users getting it wrong accidentally.


     \ /
      Last update: 2011-10-08 15:27    [W:0.027 / U:77.144 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site