[U-Boot] [PATCH 6/6] linux/kernel.h: sync min, max, min3, max3 macros with Linux

Lukasz Majewski l.majewski at samsung.com
Thu Nov 6 10:22:23 CET 2014


Hi Masahiro,

> U-Boot has never cared about the type when we get max/min of two
> values, but Linux Kernel does.  This commit gets min, max, min3, max3
> macros synced with the kernel introduing type checks.
> 
> Many of references of those macros must be fixed to suppress warnings.
> We have two options:
>  - Use min, max, min3, max3 only when the arguments have the same type
>    (or add casts to the arguments)
>  - Use min_t/max_t instead with the appropriate type for the first
>    argument
> 
> Signed-off-by: Masahiro Yamada <yamada.m at jp.panasonic.com>
> ---
> 
>  arch/arm/cpu/arm1176/tnetv107x/clock.c    |  5 ++-
>  arch/arm/cpu/armv7/exynos/clock.c         |  4 +-
>  arch/arm/cpu/armv7/tegra20/display.c      |  4 +-
>  arch/avr32/cpu/at32ap700x/clk.c           |  2 +-
>  arch/blackfin/cpu/jtag-console.c          |  2 +-
>  arch/blackfin/lib/string.c                |  2 +-
>  arch/powerpc/cpu/mpc85xx/tlb.c            |  2 +-
>  arch/powerpc/cpu/ppc4xx/44x_spd_ddr2.c    | 32 ++++++++++------
>  arch/powerpc/lib/bootm.c                  |  2 +-
>  arch/sandbox/cpu/start.c                  |  2 +-
>  arch/x86/cpu/coreboot/pci.c               |  2 +-
>  arch/x86/cpu/coreboot/sdram.c             |  2 +-
>  board/freescale/common/sys_eeprom.c       |  6 +--
>  board/gdsys/p1022/controlcenterd-id.c     |  5 ++-
>  board/imgtec/malta/malta.c                |  2 +-
>  common/cmd_elf.c                          | 12 +++---
>  common/cmd_sf.c                           | 16 ++++----
>  common/env_nand.c                         |  4 +-
>  common/fdt_support.c                      |  3 +-
>  common/lcd.c                              |  6 +--
>  common/usb_hub.c                          |  3 +-
>  drivers/block/ahci.c                      |  2 +-
>  drivers/ddr/fsl/ctrl_regs.c               | 22 +++++------
>  drivers/ddr/fsl/lc_common_dimm_params.c   | 62
> ++++++++++++++++++-------------
> drivers/ddr/fsl/main.c                    |  3 +-
> drivers/dfu/dfu.c                         |  2 +-
> drivers/i2c/fsl_i2c.c                     |  2 +-
> drivers/misc/cros_ec_spi.c                |  2 +-
> drivers/mmc/fsl_esdhc.c                   |  2 +-
> drivers/mmc/pxa_mmc_gen.c                 |  6 +--
> drivers/mtd/nand/denali_spl.c             |  2 +-
> drivers/mtd/spi/sandbox.c                 |  2 +-
> drivers/mtd/spi/sf_ops.c                  |  5 ++-
> drivers/net/netconsole.c                  |  2 +-
> drivers/pci/pci.c                         |  6 ++-
> drivers/pci/pci_auto.c                    |  2 +-
> drivers/serial/usbtty.c                   |  2 +-
> drivers/spi/fsl_espi.c                    |  4 +-
> drivers/spi/mxc_spi.c                     |  2 +-
> drivers/spi/spi-uclass.c                  |  2 +-
> drivers/tpm/tpm_tis_lpc.c                 |  2 +-
> drivers/usb/gadget/composite.c            |  4 +-
> drivers/usb/gadget/designware_udc.c       |  4 +-
> drivers/usb/gadget/pxa27x_udc.c           |  3 +-
> drivers/usb/gadget/s3c_udc_otg_xfer_dma.c |  4 +-

From my side:
Acked-by: Lukasz Majewski <l.majewski at samsung.com>

Tested-by: Lukasz Majewski <l.majewski at samsung.com>

Test HW: Trats2 (Exynos4412)

-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group




More information about the U-Boot mailing list