lkml.org 
[lkml]   [2016]   [Dec]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v5 2/5] i2c: Add STM32F4 I2C driver
    Hi M'boumba,

    [auto build test WARNING on wsa/i2c/for-next]
    [also build test WARNING on v4.9-rc8 next-20161208]
    [if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

    url: https://github.com/0day-ci/linux/commits/M-boumba-Cedric-Madianga/Add-support-for-the-STM32F4-I2C/20161208-173240
    base: https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git i2c/for-next
    config: i386-allmodconfig (attached as .config)
    compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
    reproduce:
    # save the attached .config to linux build tree
    make ARCH=i386

    All warnings (new ones prefixed by >>):

    In file included from include/linux/clk.h:16:0,
    from drivers/i2c/busses/i2c-stm32f4.c:12:
    drivers/i2c/busses/i2c-stm32f4.c: In function 'stm32f4_i2c_set_periph_clk_freq':
    include/linux/kernel.h:749:16: warning: comparison of distinct pointer types lacks a cast
    (void) (&max1 == &max2); \
    ^
    include/linux/kernel.h:737:2: note: in definition of macro '__min'
    t1 min1 = (x); \
    ^~
    include/linux/kernel.h:778:28: note: in expansion of macro 'min'
    #define clamp(val, lo, hi) min((typeof(val))max(val, lo), hi)
    ^~~
    >> include/linux/kernel.h:752:2: note: in expansion of macro '__max'
    __max(typeof(x), typeof(y), \
    ^~~~~
    >> include/linux/kernel.h:778:45: note: in expansion of macro 'max'
    #define clamp(val, lo, hi) min((typeof(val))max(val, lo), hi)
    ^~~
    >> drivers/i2c/busses/i2c-stm32f4.c:201:9: note: in expansion of macro 'clamp'
    freq = clamp(freq, STM32F4_I2C_MIN_FREQ, STM32F4_I2C_MAX_FREQ);
    ^~~~~
    include/linux/kernel.h:749:16: warning: comparison of distinct pointer types lacks a cast
    (void) (&max1 == &max2); \
    ^
    include/linux/kernel.h:737:13: note: in definition of macro '__min'
    t1 min1 = (x); \
    ^
    include/linux/kernel.h:778:28: note: in expansion of macro 'min'
    #define clamp(val, lo, hi) min((typeof(val))max(val, lo), hi)
    ^~~
    >> include/linux/kernel.h:752:2: note: in expansion of macro '__max'
    __max(typeof(x), typeof(y), \
    ^~~~~
    >> include/linux/kernel.h:778:45: note: in expansion of macro 'max'
    #define clamp(val, lo, hi) min((typeof(val))max(val, lo), hi)
    ^~~
    >> drivers/i2c/busses/i2c-stm32f4.c:201:9: note: in expansion of macro 'clamp'
    freq = clamp(freq, STM32F4_I2C_MIN_FREQ, STM32F4_I2C_MAX_FREQ);
    ^~~~~
    include/linux/kernel.h:739:16: warning: comparison of distinct pointer types lacks a cast
    (void) (&min1 == &min2); \
    ^
    include/linux/kernel.h:742:2: note: in expansion of macro '__min'
    __min(typeof(x), typeof(y), \
    ^~~~~
    include/linux/kernel.h:778:28: note: in expansion of macro 'min'
    #define clamp(val, lo, hi) min((typeof(val))max(val, lo), hi)
    ^~~
    >> drivers/i2c/busses/i2c-stm32f4.c:201:9: note: in expansion of macro 'clamp'
    freq = clamp(freq, STM32F4_I2C_MIN_FREQ, STM32F4_I2C_MAX_FREQ);
    ^~~~~

    vim +/clamp +201 drivers/i2c/busses/i2c-stm32f4.c

    185
    186 static void stm32f4_i2c_disable_it(struct stm32f4_i2c_dev *i2c_dev)
    187 {
    188 void __iomem *reg = i2c_dev->base + STM32F4_I2C_CR2;
    189
    190 stm32f4_i2c_clr_bits(reg, STM32F4_I2C_CR2_IRQ_MASK);
    191 }
    192
    193 static void stm32f4_i2c_set_periph_clk_freq(struct stm32f4_i2c_dev *i2c_dev)
    194 {
    195 u32 clk_rate, cr2, freq;
    196
    197 cr2 = readl_relaxed(i2c_dev->base + STM32F4_I2C_CR2);
    198 cr2 &= ~STM32F4_I2C_CR2_FREQ_MASK;
    199 clk_rate = clk_get_rate(i2c_dev->clk);
    200 freq = clk_rate / MHZ_TO_HZ;
    > 201 freq = clamp(freq, STM32F4_I2C_MIN_FREQ, STM32F4_I2C_MAX_FREQ);
    202 cr2 |= STM32F4_I2C_CR2_FREQ(freq);
    203 writel_relaxed(cr2, i2c_dev->base + STM32F4_I2C_CR2);
    204 }
    205
    206 static void stm32f4_i2c_set_rise_time(struct stm32f4_i2c_dev *i2c_dev)
    207 {
    208 u32 trise, freq, cr2, val;
    209

    ---
    0-DAY kernel test infrastructure Open Source Technology Center
    https://lists.01.org/pipermail/kbuild-all Intel Corporation
    [unhandled content-type:application/gzip]
    \
     
     \ /
      Last update: 2016-12-08 11:48    [W:3.229 / U:0.588 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site