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

Groups > linux.kernel > #1562804

Re: [PATCH 6/7] arm, arm64: factorize common cpu capacity default code

Show all headers | View raw

On Thu, Jan 19, 2017 at 02:37:56PM +0000, Juri Lelli wrote:
> +extern unsigned long
> +arch_scale_cpu_capacity(struct sched_domain *sd, int cpu);
> +extern void set_capacity_scale(unsigned int cpu, unsigned long capacity);

These should be in a header file (please run your code through sparse).

> +extern bool cap_parsing_failed;
> +extern void normalize_cpu_capacity(void);
> +extern int __init parse_cpu_capacity(struct device_node *cpu_node, int cpu);

Same for these.

> diff --git a/drivers/base/arch_topology.c b/drivers/base/arch_topology.c
> new file mode 100644
> index 000000000000..3faf89518892
> --- /dev/null
> +++ b/drivers/base/arch_topology.c
> @@ -0,0 +1,240 @@
> +/*
> + * driver/base/arch_topology.c - Arch specific cpu topology information
> + *
> + * Written by: Juri Lelli, ARM Ltd.
> + *
> + * Copyright (C) 2016, ARM Ltd.
> + *
> + * All rights reserved.
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.

The files that you've taken this code from are GPLv2, but you've thrown
a GPLv2+ header on a file that's merely a copy of the original code.
As some of the code you've moved to this new file is from Nicolas and
Vincent, you need to seek their approval to make this change of license
terms, or keep the original license terms.

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

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

Thread

[PATCH 0/7]  Fix issues and factorize arm/arm64 capacity information code Juri Lelli <juri.lelli@arm.com> - 2017-01-19 15:40 +0100
  [PATCH 2/7] Documentation/ABI: add information about cpu_capacity Juri Lelli <juri.lelli@arm.com> - 2017-01-19 15:50 +0100
  [PATCH 7/7] arm,arm64,drivers: reduce scope of cap_parsing_failed Juri Lelli <juri.lelli@arm.com> - 2017-01-19 15:50 +0100
  [PATCH 4/7] arm: remove wrong CONFIG_PROC_SYSCTL ifdef Juri Lelli <juri.lelli@arm.com> - 2017-01-19 15:50 +0100
  [PATCH 6/7] arm, arm64: factorize common cpu capacity default code Juri Lelli <juri.lelli@arm.com> - 2017-01-19 15:50 +0100
    Re: [PATCH 6/7] arm, arm64: factorize common cpu capacity default  code Russell King - ARM Linux <linux@armlinux.org.uk> - 2017-01-19 16:00 +0100
      Re: [PATCH 6/7] arm, arm64: factorize common cpu capacity default  code Juri Lelli <juri.lelli@arm.com> - 2017-01-19 17:00 +0100
    Re: [PATCH 6/7] arm, arm64: factorize common cpu capacity default  code Dietmar Eggemann <dietmar.eggemann@arm.com> - 2017-01-19 17:20 +0100
      Re: [PATCH 6/7] arm, arm64: factorize common cpu capacity default  code Juri Lelli <juri.lelli@arm.com> - 2017-01-20 11:50 +0100
  Re: [PATCH 0/7]  Fix issues and factorize arm/arm64 capacity  information code Juri Lelli <juri.lelli@arm.com> - 2017-01-30 13:30 +0100
    Re: [PATCH 0/7]  Fix issues and factorize arm/arm64 capacity  information code Catalin Marinas <catalin.marinas@arm.com> - 2017-01-30 19:00 +0100
      Re: [PATCH 0/7]  Fix issues and factorize arm/arm64 capacity  information code Juri Lelli <juri.lelli@arm.com> - 2017-01-30 19:30 +0100

csiph-web