lkml.org 
[lkml]   [2015]   [Feb]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [PATCH v2 02/18] ARM: ARMv7M: Enlarge vector table to 256 entries
    From
    2015-02-23 11:33 GMT+01:00 Maxime Coquelin <mcoquelin.stm32@gmail.com>:
    > 2015-02-20 20:47 GMT+01:00 Uwe Kleine-König <u.kleine-koenig@pengutronix.de>:
    >> On Fri, Feb 20, 2015 at 07:01:01PM +0100, Maxime Coquelin wrote:
    >>> From Cortex-M reference manuals, the nvic supports up to 240 interrupts.
    >>> So the number of entries in vectors table is up to 256.
    >>>
    >>> This patch adds a new config flag to specify the number of external interrupts.
    >>> Some ifdeferies are added in order to respect the natural alignment without
    >>> wasting too much space on smaller systems.
    >>>
    >>> Signed-off-by: Maxime Coquelin <mcoquelin.stm32@gmail.com>
    >>> ---
    >>> arch/arm/kernel/entry-v7m.S | 13 +++++++++----
    >>> arch/arm/mm/Kconfig | 15 +++++++++++++++
    >>> 2 files changed, 24 insertions(+), 4 deletions(-)
    >>>
    >>> diff --git a/arch/arm/kernel/entry-v7m.S b/arch/arm/kernel/entry-v7m.S
    >>> index 8944f49..68cde36 100644
    >>> --- a/arch/arm/kernel/entry-v7m.S
    >>> +++ b/arch/arm/kernel/entry-v7m.S
    >>> @@ -117,9 +117,14 @@ ENTRY(__switch_to)
    >>> ENDPROC(__switch_to)
    >>>
    >>> .data
    >>> - .align 8
    >>> +#if CONFIG_CPUV7M_NUM_IRQ <= 112
    >> I would have called this CONFIG_CPU_V7M_NUM_IRQ to match the already
    >> existing CPU_V7M symbol.
    >
    > That's better indeed.
    > It will be changed in v3.
    >
    >>
    >>> + .align 9
    >>> +#else
    >>> + .align 10
    >>> +#endif
    >>> +
    >>
    >> Other than that:
    >> Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
    >>
    >> Who do you target to apply this series?
    >
    > I guess it should go through Russell's Patch Tracking System?

    Sorry, I answered your question too quickly.
    I meant this patch should go through Russell's Patch Tracking System.

    For the other patches, I think it should be picked by their respective
    maintainers.
    Or I can create an immutable tag (on github or kernel.org?) that will
    be merged by the different sub-systems?
    What would you advise?

    Thanks,
    Maxime

    >
    > Thanks,
    > Maxime
    >>
    >> Best regards
    >> Uwe
    >>
    >> --
    >> Pengutronix e.K. | Uwe Kleine-König |
    >> Industrial Linux Solutions | http://www.pengutronix.de/ |


    \
     
     \ /
      Last update: 2015-02-26 12:01    [W:26.002 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site