Skip to content
  1. Oct 07, 2012
    • Shawn Guo's avatar
      ARM: dts: remove redundant imx dtb targets from Makefile · 68528265
      Shawn Guo authored
      
      
      We already have CONFIG_ARCH_MXC cover imx5 and imx6 dtb targets.
      Remove the redundant ones with CONFIG_ARCH_IMX5 and CONFIG_SOC_IMX6Q.
      
      Signed-off-by: default avatarShawn Guo <shawn.guo@linaro.org>
      Signed-off-by: default avatarOlof Johansson <olof@lixom.net>
      68528265
    • Tony Prisk's avatar
      vt8500: Fix build warning when no framebuffer selected · a4ee7770
      Tony Prisk authored
      
      
      Check for framebuffer defines before declaring variables in vt8500.c
      Removes a compile-time warning about unused variables.
      
      Signed-off-by: default avatarTony Prisk <linux@prisktech.co.nz>
      Signed-off-by: default avatarOlof Johansson <olof@lixom.net>
      a4ee7770
    • Tony Prisk's avatar
      dtb: Add arch-vt8500 board files to arch/arm/boot/dts/Makefile · ed304be1
      Tony Prisk authored
      
      
      Add board files (vt8500-bv07, wm8505-ref and wm8650-mid)
      to allow 'make dtbs' on arch-vt8500.
      
      Signed-off-by: default avatarTony Prisk <linux@prisktech.co.nz>
      Signed-off-by: default avatarOlof Johansson <olof@lixom.net>
      ed304be1
    • Stephen Warren's avatar
      ARM: tegra: remove "Tegra board type" comment from Kconfig · 6dd41a1f
      Stephen Warren authored
      
      
      Since the complete conversion to device tree, there are no board-
      specific Kconfig options left, so remove the useless Kconfig entry.
      
      Signed-off-by: default avatarStephen Warren <swarren@nvidia.com>
      Reviewed-by: default avatarThierry Reding <thierry.reding@avionic-design.de>
      Signed-off-by: default avatarOlof Johansson <olof@lixom.net>
      6dd41a1f
    • Stephen Warren's avatar
      ARM: tegra: fix invalid unit-address in tegra*.dtsi · bbfc33bd
      Stephen Warren authored
      
      
      Unit addresses, whilst written in hex, don't contain a 0x prefix.
      
      Signed-off-by: default avatarStephen Warren <swarren@nvidia.com>
      Reviewed-by: default avatarThierry Reding <thierry.reding@avionic-design.de>
      Signed-off-by: default avatarOlof Johansson <olof@lixom.net>
      bbfc33bd
    • Olof Johansson's avatar
      ARM: iop: fix mismerge of Kconfig · c0f72d7c
      Olof Johansson authored
      
      
      I mismerged one of the branches that moves around gpio header file usage,
      and messed up for IOP. This fixes the obvious compilation failures caused
      by it.
      
      Signed-off-by: default avatarOlof Johansson <olof@lixom.net>
      c0f72d7c
    • Arnd Bergmann's avatar
      ARM: mv78xx0: correct addr_map_cfg __initdata annotation · edc9e333
      Arnd Bergmann authored
      
      
      The annotation on the addr_map_cfg variable is in the wrong place.
      
      Without this patch, building mv78xx0_defconfig results in:
      
      /home/arnd/linux-arm/arch/arm/mach-mv78xx0/addr-map.c:59:2: warning: initialization from incompatible pointer type [enabled by default]
      /home/arnd/linux-arm/arch/arm/mach-mv78xx0/addr-map.c:59:2: warning: (near initialization for 'addr_map_cfg.win_cfg_base') [enabled by default]
      
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Acked-by: default avatarAndrew Lunn <andrew@lunn.ch>
      Cc: Jason Cooper <jason@lakedaemon.net>
      edc9e333
    • Arnd Bergmann's avatar
      ARM: footbridge: remove RTC_IRQ definition · ec8e8208
      Arnd Bergmann authored
      Since commit bd8abc9a
      
       "ARM: mc146818rtc: remove unnecessary include of
      mach/irqs.h", building footbridge_defconfig results in this warning:
      
      In file included from include/linux/mc146818rtc.h:16:0,
                       from arch/arm/mach-footbridge/isa-rtc.c:21:
      arch/arm/include/asm/mc146818rtc.h:10:0: warning: "RTC_IRQ" redefined [enabled by default]
      arch/arm/mach-footbridge/include/mach/irqs.h:93:0: note: this is the location of the previous definition
      
      The above commit was intentionally made to catch errors like this,
      where code relies on the RTC_IRQ definition. The only driver using
      it is the legacy PC-style drivers/char/rtc.c driver.
      However, the ARM architecture has been using the RTC_LIB framework
      since at least 2006, and that doesn't use it.
      
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Cc: Rob Herring <rob.herring@calxeda.com>
      Cc: Alessandro Zummo <a.zummo@towertech.it>
      Cc: rtc-linux@googlegroups.com
      Cc: Russell King <linux@arm.linux.org.uk>
      ec8e8208
    • Arnd Bergmann's avatar
      ARM: soc: dependency warnings for errata · 8f90cce5
      Arnd Bergmann authored
      
      
      The PL310_ERRATA_753970 and ARM_ERRATA_764369 symbols only make sense
      when the base features for them are enabled, so select them
      conditionally in Kconfig to avoid warnings like:
      
      warning: (UX500_SOC_COMMON) selects PL310_ERRATA_753970 which has unmet direct dependencies (CACHE_PL310)
      warning: (ARCH_TEGRA_2x_SOC && ARCH_TEGRA_3x_SOC && UX500_SOC_COMMON) selects ARM_ERRATA_764369 which has unmet direct dependencies (CPU_V7 && SMP)
      
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Reviewed-by: default avatarLinus Walleij <linus.walleij@linaro.org>
      Acked-by: default avatarStephen Warren <swarren@nvidia.com>
      8f90cce5
    • Arnd Bergmann's avatar
      ARM: ks8695: __arch_virt_to_dma type handling · 113f7ae5
      Arnd Bergmann authored
      
      
      __arch_virt_to_dma expects a virtual address pointer, but
      the ks8695 implementation of this macro treats it as an
      integer. Adding a type cast avoids hundreds of identical
      warning messages.
      
      Without this patch, building acs5k_defconfig results in:
      
      arch/arm/include/asm/dma-mapping.h: In function 'virt_to_dma':
      arch/arm/include/asm/dma-mapping.h:60:2: warning: passing argument 1 of '__virt_to_phys' makes integer from pointer without a cast [enabled by default]
      arch/arm/include/asm/memory.h:172:60: note: expected 'long unsigned int' but argument is of type 'void *'
      In file included from include/linux/dma-mapping.h:73:0,
                       from include/linux/skbuff.h:33,
                       from security/commoncap.c:21:
      
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Cc: Krzysztof Halasa <khc@pm.waw.pl>
      Cc: Daniel Silverstone <dsilvers@simtec.co.uk>
      Cc: Ben Dooks <ben-linux@fluff.org>
      113f7ae5
    • Arnd Bergmann's avatar
      ARM: rpc: check device_register return code in ecard_probe · 5f07809e
      Arnd Bergmann authored
      
      
      device_register is marked __must_check, so we better propagate the error
      value by returning it from ecard_probe.
      
      Without this patch, building rpc_defconfig results in:
      
      arch/arm/mach-rpc/ecard.c: In function 'ecard_probe':
      arch/arm/mach-rpc/ecard.c:963:17: warning: ignoring return value of 'device_register', declared with attribute warn_unused_result [-Wunused-result]
      
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Cc: Russell King <rmk+kernel@arm.linux.org.uk>
      5f07809e
    • Arnd Bergmann's avatar
      ARM: davinci: don't mark da850_register_cpufreq as __init · 5063557a
      Arnd Bergmann authored
      
      
      The mityomapl138_cpufreq_init and read_factory_config function in
      board-mityomapl138.c are not __init functions and might be called
      at a later stage, so da850_register_cpufreq must not be __init either.
      
      Without this patch, building da8xx_omapl_defconfig results in:
      
      WARNING: arch/arm/mach-davinci/built-in.o(.text+0x2eb4): Section mismatch in reference from the function read_factory_config() to the function .init.text:da850_register_cpufreq()
      The function read_factory_config() references
      the function __init da850_register_cpufreq().
      This is often because read_factory_config lacks a __init
      annotation or the annotation of da850_register_cpufreq is wrong.
      
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Acked-by: default avatarSekhar Nori <nsekhar@ti.com>
      Cc: Kevin Hilman <khilman@ti.com>
      5063557a
    • Arnd Bergmann's avatar
      ARM: iop13xx: fix iq81340sc_atux_map_irq prototype · 58cbdbe0
      Arnd Bergmann authored
      
      
      The pci map_irq callbacks get a 'const' pci_dev argument, so change the
      iop13xx version to use the same prototype as everything else.
      
      Without this patch, building iop13xx_defconfig results in:
      
      arch/arm/mach-iop13xx/iq81340sc.c:63:2: warning: initialization from incompatible pointer type [enabled by default]
      arch/arm/mach-iop13xx/iq81340sc.c:63:2: warning: (near initialization for 'iq81340sc_pci.map_irq') [enabled by default]
      
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Cc: Lennert Buytenhek <kernel@wantstofly.org>
      Cc: Dan Williams <djbw@fb.com>
      58cbdbe0
    • Arnd Bergmann's avatar
      ARM: iop13xx: mark iop13xx_scan_bus as __devinit · b04e246a
      Arnd Bergmann authored
      
      
      pci_scan_root_bus is __devinit, so iop13xx_scan_bus has to be the
      same in order to safely call it. This is ok because the function
      itself is only called from the hwpci->scan callback.
      
      WARNING: vmlinux.o(.text+0x10138): Section mismatch in reference from the function iop13xx_scan_bus() to the function .devinit.text:pci_scan_root_bus()
      The function iop13xx_scan_bus() references
      the function __devinit pci_scan_root_bus().
      This is often because iop13xx_scan_bus lacks a __devinit
      annotation or the annotation of pci_scan_root_bus is wrong.
      
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Cc: Bjorn Helgaas <bhelgaas@google.com>
      Cc: Lennert Buytenhek <kernel@wantstofly.org>
      Cc: Dan Williams <djbw@fb.com>
      b04e246a
    • Arnd Bergmann's avatar
      ARM: mv78xx0: mark mv78xx0_timer_init as __init_refok · e7215fb3
      Arnd Bergmann authored
      
      
      The sys_timer init function is only called at __init time,
      so it's safe to mark mv78xx0_timer_init as __init_refok,
      which allows us to call orion_time_init without getting
      a link time warning.
      
      Without this patch, building mv78xx0_defconfig results in:
      
      WARNING: vmlinux.o(.text+0x15470): Section mismatch in reference from the function mv78xx0_timer_init() to the function .init.text:orion_time_init()
      The function mv78xx0_timer_init() references
      the function __init orion_time_init().
      This is often because mv78xx0_timer_init lacks a __init
      annotation or the annotation of orion_time_init is wrong.
      
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Cc: Jason Cooper <jason@lakedaemon.net>
      Cc: Andrew Lunn <andrew@lunn.ch>
      e7215fb3
    • Arnd Bergmann's avatar
      ARM: s3c24xx: fix multiple section mismatch warnings · 673550a1
      Arnd Bergmann authored
      
      
      The *_irq_add function should not be marked __init because the driver
      subsystem thinks they might be called at a later stage.
      
      The usb_simtec_init function accesses initdata and should be marked
      init. This is safe because the only caller is also an init function.
      
      Without this patch, building s3c2410_defconfig results in:
      
      WARNING: arch/arm/mach-s3c24xx/built-in.o(.data+0x1030): Section mismatch in reference from the variable s3c2416_irq_interface to the function .init.text:s3c2416_irq_add()
      The variable s3c2416_irq_interface references
      the function __init s3c2416_irq_add()
      If the reference is valid then annotate the
      variable with __init* or __refdata (see linux/init.h) or name the variable:
      *_template, *_timer, *_sht, *_ops, *_probe, *_probe_one, *_console
      
      WARNING: arch/arm/mach-s3c24xx/built-in.o(.data+0x1b08): Section mismatch in reference from the variable s3c2443_irq_interface to the function .init.text:s3c2443_irq_add()
      The variable s3c2443_irq_interface references
      the function __init s3c2443_irq_add()
      If the reference is valid then annotate the
      variable with __init* or __refdata (see linux/init.h) or name the variable:
      *_template, *_timer, *_sht, *_ops, *_probe, *_probe_one, *_console
      
      WARNING: arch/arm/mach-s3c24xx/built-in.o(.data+0xf44): Section mismatch in reference from the variable s3c2416_irq_interface to the function .init.text:s3c2416_irq_add()
      The variable s3c2416_irq_interface references
      the function __init s3c2416_irq_add()
      If the reference is valid then annotate the
      variable with __init* or __refdata (see linux/init.h) or name the variable:
      *_template, *_timer, *_sht, *_ops, *_probe, *_probe_one, *_console
      
      WARNING: arch/arm/mach-s3c24xx/built-in.o(.text+0x3f7c): Section mismatch in reference from the function usb_simtec_init() to the (unknown reference) .init.data:(unknown)
      The function usb_simtec_init() references
      the (unknown reference) __initdata (unknown).
      This is often because usb_simtec_init lacks a __initdata
      annotation or the annotation of (unknown) is wrong.
      
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Cc: Ben Dooks <ben-linux@fluff.org>
      Cc: Kukjin Kim <kgene.kim@samsung.com>
      673550a1
  2. Oct 06, 2012
  3. Oct 05, 2012
    • Wade Farnsworth's avatar
      ARM: 7548/1: include linux/sched.h in syscall.h · 8ef102c6
      Wade Farnsworth authored
      
      
      The syscall tracing patch introduces a compile bug in lttng-modules
      when the latter calls syscall_get_nr(), similar to the following:
      
      <path-to-linux>/arch/arm/include/asm/syscall.h:21:2: error: implicit declaration of function 'task_thread_info' [-Werror=implicit-function-declaration]
      
      The issue is that we are using task_thread_info() in the
      syscall_get_nr() function in asm/syscall.h, but not explicitly
      including sched.h from this file, so we can expect this bug might
      surface any time that syscall_get_nr() is called.
      
      Explicitly including sched.h solves the problem.
      
      Cc: <stable@vger.kernel.org> [3.5, 3.6]
      Signed-off-by: default avatarWade Farnsworth <wade_farnsworth@mentor.com>
      Acked-by: default avatarWill Deacon <will.deacon@arm.com>
      Signed-off-by: default avatarRussell King <rmk+kernel@arm.linux.org.uk>
      8ef102c6
  4. Oct 04, 2012
    • Arnd Bergmann's avatar
      ARM: at91: unused variable in at91_pm_verify_clocks · 9e0e4e11
      Arnd Bergmann authored
      
      
      The code using the variable 'i' in this function is conditional which
      results in a harmless compiler warning. Using the IS_ENABLED macro
      instead of #ifdef makes the code look nicer and gets rid of the
      warning.
      
      Without this patch, building at91sam9263_defconfig results in:
      
      /home/arnd/linux-arm/arch/arm/mach-at91/pm.c: In function 'at91_pm_verify_clocks':
      /home/arnd/linux-arm/arch/arm/mach-at91/pm.c:137:6: warning: unused variable 'i' [-Wunused-variable]
      
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Acked-by: default avatarJean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
      Cc: Nicolas Ferre <nicolas.ferre@atmel.com>
      9e0e4e11
    • Arnd Bergmann's avatar
      ARM: at91: skip at91_io_desc definition for NOMMU · ac09281a
      Arnd Bergmann authored
      
      
      On NOMMU systems, we do cannot remap the MMIO space, so the
      definition of at91_io_desc is unused.
      
      Without this patch, building at91x40_defconfig results in:
      
      arch/arm/mach-at91/setup.c:90:24: warning: 'at91_io_desc' defined but not used [-Wunused-variable]
      
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Cc: Nicolas Ferre <nicolas.ferre@atmel.com>
      Cc: Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>
      ac09281a
    • Arnd Bergmann's avatar
      ARM: pxa: work around duplicate definition of GPIO24_SSP1_SFRM · e0347c52
      Arnd Bergmann authored
      
      
      The symbol "GPIO24_SSP1_SFRM" is defined in both mfp-pxa27x.h and
      mfp-pxa25x.h. Since the macro is not actually used in the cm-x2xx.c
      file, but it includes both headers, a safe workaround should be
      to just undefine it from the .c file. This is a bit hacky and
      the headers should be fixed to not both define it, but for now
      it gets us around an annoying warning.
      
      Without this patch, building cm_x2xx_defconfig results in:
      
      In file included from arch/arm/mach-pxa/include/mach-pxa/pxa27x.h:7:0,
                       from arch/arm/mach-pxa/cm-x2xx.c:25:
      arch/arm/mach-pxa/include/mach-pxa/mfp-pxa27x.h:215:0: warning: "GPIO24_SSP1_SFRM" redefined [enabled by default]
      arch/arm/mach-pxa/include/mach-pxa/mfp-pxa25x.h:111:0: note: this is the location of the previous definition
      
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Acked-by: default avatarIgor Grinberg <grinberg@compulab.co.il>
      Cc: Haojian Zhuang <haojian.zhuang@gmail.com>
      Cc: Mike Rapoport <mike@compulab.co.il>
      e0347c52
    • Arnd Bergmann's avatar
      ARM: pxa: remove sharpsl_fatal_check function · 066258f4
      Arnd Bergmann authored
      The sharpsl_fatal_check has not been used since Pavel Machek removed
      the caller in 99f329a2
      
       "pxa/sharpsl_pm: zaurus c3000 aka spitz: fix
      resume". Nobody has complained since 2009, so it's safe to assume we
      can just remove the function.
      
      Without this patch, building corgi_defconfig results in:
      
      /home/arnd/linux-arm/arch/arm/mach-pxa/sharpsl_pm.c:693:12: warning: 'sharpsl_fatal_check' defined but not used [-Wunused-function]
      
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Cc: Pavel Machek <pavel@ucw.cz>
      Cc: Stanislav Brabec <utx@penguin.cz>
      Cc: Eric Miao <eric.y.miao@gmail.com>
      Cc: Haojian Zhuang <haojian.zhuang@gmail.com>
      066258f4
    • Arnd Bergmann's avatar
      ARM: pxa: define palmte2_pxa_keys conditionally · 30bda0eb
      Arnd Bergmann authored
      
      
      Gcc prints a harmless warning about palmte2_pxa_keys not being used
      when the gpio keyboard driver is disabled. The solution is to use
      the same #ifdef that is already present in the place where the
      symbol is used.
      
      Without this patch, building palmz72_defconfig results in:
      
      /home/arnd/linux-arm/arch/arm/mach-pxa/palmte2.c:128:31: warning: 'palmte2_pxa_keys' defined but not used [-Wunused-variable]
      
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Acked-by: default avatarMarek Vasut <marek.vasut@gmail.com>
      Cc: Carlos Eduardo Medaglia Dyonisio <cadu@nerdfeliz.com>
      Cc: Haojian Zhuang <haojian.zhuang@gmail.com>
      Cc: Eric Miao <eric.y.miao@gmail.com>
      30bda0eb
    • Arnd Bergmann's avatar
      ARM: pxa: Wunused-result warning in viper board file · 5f0cc6d1
      Arnd Bergmann authored
      
      
      Calling kstrtoul requires checking the result. In case of
      the viper_tpm_setup function, let's fail the __setup function
      if the number was invalid.
      
      Without this patch, building viper_defconfig results in:
      
      arch/arm/mach-pxa/viper.c: In function 'viper_tpm_setup':
      arch/arm/mach-pxa/viper.c:771:10: warning: ignoring return value of 'kstrtoul', declared with attribute warn_unused_result [-Wunused-result]
      
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Acked-by: default avatarMarc Zyngier <maz@misterjones.org>
      Cc: Haojian Zhuang <haojian.zhuang@marvell.com>
      Cc: Eric Miao <eric.y.miao@gmail.com>
      5f0cc6d1
    • Arnd Bergmann's avatar
      ARM: shark: fix shark_pci_init return code · 9c7f4f5c
      Arnd Bergmann authored
      
      
      When run on the wrong platform, the shark_pci_init function
      returns an undefined value, as reported by a gcc warning,
      so let's just return -ENODEV.
      
      Without this patch, building shark_defconfig results in:
      
      arch/arm/mach-shark/pci.c: In function 'shark_pci_init':
      arch/arm/mach-shark/pci.c:42:3: warning: 'return' with no value, in function returning non-void [-Wreturn-type]
      
      Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
      Cc: "Krzysztof Halasa" <khc@pm.waw.pl>
      9c7f4f5c
  5. Oct 03, 2012
  6. Oct 02, 2012
  7. Sep 30, 2012
  8. Sep 29, 2012