Re: [PATCH v5 10/23] mtd: nand: denali: rework interrupt handling

2017-06-12 Thread Masahiro Yamada
Hi Boris, 2017-06-09 16:58 GMT+09:00 Boris Brezillon : > Hi Masahiro, > > On Fri, 9 Jun 2017 02:26:34 +0900 > Masahiro Yamada wrote: > >> Hi Boris >> >> 2017-06-09 0:43 GMT+09:00 Boris Brezillon >> : >> > On Thu, 8 Jun 2017 21:58:00 +0900 >> > Masahiro Yamada wrote: >> > >> >> Hi Boris, >> >>

Re: [PATCH v5 10/23] mtd: nand: denali: rework interrupt handling

2017-06-09 Thread Boris Brezillon
Hi Masahiro, On Fri, 9 Jun 2017 02:26:34 +0900 Masahiro Yamada wrote: > Hi Boris > > 2017-06-09 0:43 GMT+09:00 Boris Brezillon > : > > On Thu, 8 Jun 2017 21:58:00 +0900 > > Masahiro Yamada wrote: > > > >> Hi Boris, > >> > >> 2017-06-08 20:26 GMT+09:00 Boris Brezillon > >> : > >> > On Thu

Re: [PATCH v5 10/23] mtd: nand: denali: rework interrupt handling

2017-06-08 Thread Masahiro Yamada
2017-06-09 2:26 GMT+09:00 Masahiro Yamada : > ->dev_ready() is optional, but we may end up with waiting more than needed. > > case NAND_CMD_RESET: > if (chip->dev_ready) > break; > udelay(chip->chip_delay); > > > chip->chip_delay is

Re: [PATCH v5 10/23] mtd: nand: denali: rework interrupt handling

2017-06-08 Thread Masahiro Yamada
Hi Boris 2017-06-09 0:43 GMT+09:00 Boris Brezillon : > On Thu, 8 Jun 2017 21:58:00 +0900 > Masahiro Yamada wrote: > >> Hi Boris, >> >> 2017-06-08 20:26 GMT+09:00 Boris Brezillon >> : >> > On Thu, 8 Jun 2017 19:41:39 +0900 >> > Masahiro Yamada wrote: >> > >> >> Hi Boris, >> >> >> >> >> >> 2017-0

Re: [PATCH v5 10/23] mtd: nand: denali: rework interrupt handling

2017-06-08 Thread Boris Brezillon
On Thu, 8 Jun 2017 21:58:00 +0900 Masahiro Yamada wrote: > Hi Boris, > > 2017-06-08 20:26 GMT+09:00 Boris Brezillon > : > > On Thu, 8 Jun 2017 19:41:39 +0900 > > Masahiro Yamada wrote: > > > >> Hi Boris, > >> > >> > >> 2017-06-08 16:12 GMT+09:00 Boris Brezillon > >> : > >> > Le Thu, 8 Jun

Re: [PATCH v5 10/23] mtd: nand: denali: rework interrupt handling

2017-06-08 Thread Masahiro Yamada
Hi Boris, 2017-06-08 20:26 GMT+09:00 Boris Brezillon : > On Thu, 8 Jun 2017 19:41:39 +0900 > Masahiro Yamada wrote: > >> Hi Boris, >> >> >> 2017-06-08 16:12 GMT+09:00 Boris Brezillon >> : >> > Le Thu, 8 Jun 2017 15:10:18 +0900, >> > Masahiro Yamada a écrit : >> > >> >> Hi Boris, >> >> >> >> >>

Re: [PATCH v5 10/23] mtd: nand: denali: rework interrupt handling

2017-06-08 Thread Boris Brezillon
On Thu, 8 Jun 2017 19:41:39 +0900 Masahiro Yamada wrote: > Hi Boris, > > > 2017-06-08 16:12 GMT+09:00 Boris Brezillon > : > > Le Thu, 8 Jun 2017 15:10:18 +0900, > > Masahiro Yamada a écrit : > > > >> Hi Boris, > >> > >> > >> 2017-06-07 22:57 GMT+09:00 Boris Brezillon > >> : > >> > On Wed

Re: [PATCH v5 10/23] mtd: nand: denali: rework interrupt handling

2017-06-08 Thread Masahiro Yamada
Hi Boris, 2017-06-08 16:12 GMT+09:00 Boris Brezillon : > Le Thu, 8 Jun 2017 15:10:18 +0900, > Masahiro Yamada a écrit : > >> Hi Boris, >> >> >> 2017-06-07 22:57 GMT+09:00 Boris Brezillon >> : >> > On Wed, 7 Jun 2017 20:52:19 +0900 >> > Masahiro Yamada wrote: >> > >> > >> >> -/* >> >> - * This

Re: [PATCH v5 10/23] mtd: nand: denali: rework interrupt handling

2017-06-08 Thread Boris Brezillon
Le Thu, 8 Jun 2017 15:10:18 +0900, Masahiro Yamada a écrit : > Hi Boris, > > > 2017-06-07 22:57 GMT+09:00 Boris Brezillon > : > > On Wed, 7 Jun 2017 20:52:19 +0900 > > Masahiro Yamada wrote: > > > > > >> -/* > >> - * This is the interrupt service routine. It handles all interrupts > >> - *

Re: [PATCH v5 10/23] mtd: nand: denali: rework interrupt handling

2017-06-07 Thread Masahiro Yamada
Hi Boris, 2017-06-07 22:57 GMT+09:00 Boris Brezillon : > On Wed, 7 Jun 2017 20:52:19 +0900 > Masahiro Yamada wrote: > > >> -/* >> - * This is the interrupt service routine. It handles all interrupts >> - * sent to this device. Note that on CE4100, this is a shared interrupt. >> - */ >> -static

Re: [PATCH v5 10/23] mtd: nand: denali: rework interrupt handling

2017-06-07 Thread Boris Brezillon
On Wed, 7 Jun 2017 20:52:19 +0900 Masahiro Yamada wrote: > -/* > - * This is the interrupt service routine. It handles all interrupts > - * sent to this device. Note that on CE4100, this is a shared interrupt. > - */ > -static irqreturn_t denali_isr(int irq, void *dev_id) > +static uint32_t den