Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1571932

Re: [PATCH 1/3] arc: vdk: Disable halt on reset

From Vineet Gupta <Vineet.Gupta1@synopsys.com>
Newsgroups linux.kernel
Subject Re: [PATCH 1/3] arc: vdk: Disable halt on reset
Date 2017-02-01 21:00 +0100
Message-ID <t69eq-1O3-17@gated-at.bofh.it> (permalink)
References (1 earlier) <t66gx-8kU-1@gated-at.bofh.it> <t66qe-8ov-15@gated-at.bofh.it> <t66JA-iP-21@gated-at.bofh.it> <t672X-pD-31@gated-at.bofh.it> <t67mi-x1-17@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On 02/01/2017 09:52 AM, Alexey Brodkin wrote:
>> The whole point of adding this to defconfig is to override the default from Kconfig ?
> Not anymore :)
> 
> Since commit c4c9a040ecb7 ("clocksource: import ARC timer driver"),
> see http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=c4c9a040ecb7297e011e579f5a9cc280e42d725f
> we have this:
> ---------------------->8----------------------
> config ISA_ARCV2
> 	bool "ARC ISA v2"
> 	select ARC_TIMERS_64BIT
> ---------------------->8----------------------
> 
> which really means if one selects ISA_ARCV2 then ARC_TIMERS_64BIT gets selected automatically
> and there's no way to override it from either menuconfig or defconfig.

Bummer - this means VDK based off 4.9+ kernel might be affected with nsim GFRC issue !

> Probably behavior that you meant was to keep a separate "config ARC_TIMERS_64BIT"
> and have it "default y if ISA_ARCV2".

No, the whole point of moving it out of arch/arc was to increase test coverage etc
so it was not tied to ISA_ARCV2 on purpose so that it would atleast build.

Lets see what Rudd has to say abt this. But GFRC can't be used this would need
fixing after all by introducing an additional ARC_PLAT_CANT_USE_TIMERS_64BIT which
is def_bool set to n, but selected y in VDK Kconfig.

-Vineet

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 1/3] arc: vdk: Disable halt on reset Alexey Brodkin <Alexey.Brodkin@synopsys.com> - 2017-02-01 17:50 +0100
  Re: [PATCH 1/3] arc: vdk: Disable halt on reset Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2017-02-01 18:00 +0100
    Re: [PATCH 1/3] arc: vdk: Disable halt on reset Alexey Brodkin <Alexey.Brodkin@synopsys.com> - 2017-02-01 18:20 +0100
      Re: [PATCH 1/3] arc: vdk: Disable halt on reset Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2017-02-01 18:40 +0100
        Re: [PATCH 1/3] arc: vdk: Disable halt on reset Alexey Brodkin <Alexey.Brodkin@synopsys.com> - 2017-02-01 19:00 +0100
          Re: [PATCH 1/3] arc: vdk: Disable halt on reset Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2017-02-01 21:00 +0100
            RE: [PATCH 1/3] arc: vdk: Disable halt on reset Alexey Brodkin <Alexey.Brodkin@synopsys.com> - 2017-02-01 21:10 +0100
            RE: [PATCH 1/3] arc: vdk: Disable halt on reset Ruud Derwig <Ruud.Derwig@synopsys.com> - 2017-02-02 15:10 +0100

csiph-web