lkml.org 
[lkml]   [2016]   [Nov]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH 12/16] ARM: imx: use generic API for enabling SCU
    From
    Date
    Hi Shawn,

    On Monday 14 November 2016 07:56 PM, Shawn Guo wrote:
    > On Mon, Nov 14, 2016 at 10:32:07AM +0530, Pankaj Dubey wrote:
    >> Now as we have of_scu_enable which takes care of mapping
    >> scu base from DT, lets use it.
    >>
    >> At the same time this patch cleans up mach-imx platform files by
    >> removing static mapping of SCU and dropping imx_scu_map_io function.
    >
    > I remember that the static mapping of SCU is necessary because SCU is
    > being accessed at very early boot stage where dynamic mapping hasn't
    > been set up.
    >
    >> CC: Shawn Guo <shawnguo@kernel.org>
    >> CC: Sascha Hauer <kernel@pengutronix.de>
    >> Signed-off-by: Pankaj Dubey <pankaj.dubey@samsung.com>
    >> ---
    >> arch/arm/mach-imx/common.h | 5 -----
    >> arch/arm/mach-imx/mach-imx6q.c | 8 +-------
    >> arch/arm/mach-imx/platsmp.c | 32 +++++---------------------------
    >> arch/arm/mach-imx/pm-imx6.c | 3 ++-
    >> 4 files changed, 8 insertions(+), 40 deletions(-)
    >
    > I tested it and saw that the booting of imx6q is broken like below.

    Thanks for testing and letting me know about this.

    Currently only two platforms (IMX and ZYNQ) are using SCU address at
    very early stage of boot, for configuring possible cpus via
    set_cpu_possible().. rest platforms are either using DT method or they
    handle this in smp_prepare_cpus.

    Since I am not sure if all boards based on IMX has been moved to DT
    based boot, if it has moved to completely DT based then we do not need
    this set_cpu_possible in smp_init_cpus, it will be taken care via
    "arm_dt_init_cpu_maps" and this will avoid need of early mapping of SCU
    base as well.

    If not then, currently I can't see any other way to handle this and in
    that case I will drop-out IMX platform's patch of using generic SCU APIs
    in v2 version of series.

    Thanks,
    Pankaj Dubey

    \
     
     \ /
      Last update: 2016-11-17 05:26    [W:3.097 / U:0.100 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site