Re: [PATCH] Remove explicit return type cast

[hari prasath]

On 3 August 2017 at 11:52, kbuild test robot <lkp@xxxxxxxxx> wrote:
> Hi Hari,
>
> [auto build test WARNING on staging/staging-testing]
> [also build test WARNING on next-20170802]
> [cannot apply to v4.13-rc3]
> [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/Hari-Prasath/Remove-explicit-return-type-cast/20170803-080312
> config: blackfin-allyesconfig (attached as .config)
> compiler: bfin-uclinux-gcc (GCC) 6.2.0
> reproduce:
>         wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         # save the attached .config to linux build tree
>         make.cross ARCH=blackfin
>

>> I tried these steps, it's giving me build error as below.

  CC      security/selinux/exports.o
  CC      security/apparmor/file.o
fs/ubifs/lpt_commit.c: In function 'next_pnode_to_dirty':
fs/ubifs/lpt_commit.c:617:1: internal compiler error: in
bfin_optimize_loop, at config/bfin/bfin.c:3978
Please submit a full bug report,
with preprocessed source if appropriate.
See <http://gcc.gnu.org/bugs.html> for instructions.
scripts/Makefile.build:302: recipe for target 'fs/ubifs/lpt_commit.o' failed
make[2]: *** [fs/ubifs/lpt_commit.o] Error 1
make[2]: *** Waiting for unfinished jobs....



> All warnings (new ones prefixed by >>):
>
>    drivers/staging/pi433/pi433_if.c: In function 'DIO0_irq_handler':
>>> drivers/staging/pi433/pi433_if.c:154:9: warning: return makes pointer from integer without a cast [-Wint-conversion]
>      return IRQ_HANDLED;
>             ^~~~~~~~~~~
>    drivers/staging/pi433/pi433_if.c: In function 'DIO1_irq_handler':
>    drivers/staging/pi433/pi433_if.c:174:9: warning: return makes pointer from integer without a cast [-Wint-conversion]
>      return IRQ_HANDLED;
>             ^~~~~~~~~~~
>
> vim +154 drivers/staging/pi433/pi433_if.c
>
>    129
>    130  /* GPIO interrupt handlers */
>    131  static irq_handler_t
>    132  DIO0_irq_handler(unsigned int irq, void *dev_id, struct pt_regs *regs)
>    133  {
>    134          struct pi433_device *device = dev_id;
>    135
>    136          if      (device->irq_state[DIO0] == DIO_PacketSent)
>    137          {
>    138                  device->free_in_fifo = FIFO_SIZE;
>    139                  printk("DIO0 irq: Packet sent\n"); // TODO: printk() should include KERN_ facility level
>    140                  wake_up_interruptible(&device->fifo_wait_queue);
>    141          }
>    142          else if (device->irq_state[DIO0] == DIO_Rssi_DIO0)
>    143          {
>    144                  printk("DIO0 irq: RSSI level over threshold\n");
>    145                  wake_up_interruptible(&device->rx_wait_queue);
>    146          }
>    147          else if (device->irq_state[DIO0] == DIO_PayloadReady)
>    148          {
>    149                  printk("DIO0 irq: PayloadReady\n");
>    150                  device->free_in_fifo = 0;
>    151                  wake_up_interruptible(&device->fifo_wait_queue);
>    152          }
>    153
>  > 154          return IRQ_HANDLED;
>    155  }
>    156
>
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation



-- 
Regards,
G.E.Hari Prasath