Quoting Linus Walleij (2019-01-10 04:50:56) > On Thu, Dec 13, 2018 at 7:00 PM Russell King <rmk+kernel@xxxxxxxxxxxxxxx> wrote: > > > The boot_lock is something that was required for ARM development > > platforms to ensure that the delay calibration worked properly. This > > is not necessary for modern platforms that have better bus bandwidth > > and do not need to calibrate the delay loop for secondary cores. > > Remove the boot_lock entirely. > > > > Signed-off-by: Russell King <rmk+kernel@xxxxxxxxxxxxxxx> > > Andy have you had a chance to test/apply this patch? > > I think the majority of Qcom systems use CLKSRC_QCOM > which registers an arch delay timer and swiftly bypasses the > calibration anyways so this code is pretty much pointless. > > There is a question about MSM8974 though. This appears > to not use CLKSRC_QCOM but rather HAVE_ARM_ARCH_TIMER. > drivers/clocksource/arm_arch_timer.c seems to > not register a delay timer, which is probably something that > should be fixed? > > Mark/Marc: is there any specific reason to why the ARM arch > timer does not register an arch delay timer? > Doesn't the ARM arch timer get registered in arch/arm/kernel/arch_timer.c via arch_timer_delay_timer_register()?
