Re: [linux-yocto] [PATCH] gpio-pca953x: add PCAL9535 interrupt support

2016-03-30 Thread Bruce Ashfield
On 2016-03-30 9:46 PM, Yong Li wrote: Thanks Ismo for your test. I can perform the two patches re-designing: In linux-yocto-4.4, first revert the "drive" patch, then submit the pcal9535 interrupt patch, then re-submit the "drive" patch. In the mean time, I will send the pcal9535 interrupt patch t

Re: [linux-yocto] [PATCH] gpio-pca953x: add PCAL9535 interrupt support

2016-03-30 Thread Yong Li
Thanks Ismo for your test. I can perform the two patches re-designing: In linux-yocto-4.4, first revert the "drive" patch, then submit the pcal9535 interrupt patch, then re-submit the "drive" patch. In the mean time, I will send the pcal9535 interrupt patch to linux-gpio. How do you think about it

Re: [linux-yocto] [PATCH] gpio-pca953x: add PCAL9535 interrupt support

2016-03-30 Thread Bruce Ashfield
On 16-03-30 07:34 AM, Puustinen, Ismo wrote: Hi! On Wed, 2016-03-30 at 16:37 +0800, Yong Li wrote: Galileo gen 2 board uses the PCAL9535 as the GPIO expansion, it is different from PCA9535 and includes interrupt mask/status registers, The current driver does not support the interrupt registers

Re: [linux-yocto] [PATCH] gpio-pca953x: add PCAL9535 interrupt support

2016-03-30 Thread Puustinen, Ismo
Hi! On Wed, 2016-03-30 at 16:37 +0800, Yong Li wrote: > Galileo gen 2 board uses the PCAL9535 as the GPIO expansion, > it is different from PCA9535 and includes interrupt mask/status > registers, > The current driver does not support the interrupt registers > configuration, > it causes some gpio p

[linux-yocto] [PATCH] gpio-pca953x: add PCAL9535 interrupt support

2016-03-30 Thread Yong Li
Galileo gen 2 board uses the PCAL9535 as the GPIO expansion, it is different from PCA9535 and includes interrupt mask/status registers, The current driver does not support the interrupt registers configuration, it causes some gpio pins cannot trigger interrupt events, this patch fix this issue. The

Re: [linux-yocto] [PATCH] gpio-pca953x: add PCAL9535 Interrupt support

2016-03-29 Thread Yong Li
Thanks Bruce! Let me re-send the patch v2 later Yong 2016-03-30 12:43 GMT+08:00 Bruce Ashfield : > On 2016-03-29 9:47 PM, Yong Li wrote: >> >> >> >> Hi Bruce, >> >> >> >> This patch is based on the below patch in Kernel 3.8: >> >> commit f22e9ce7703448e1a2063fa171fb22dfef1da6dc >> >> Author: Jose

Re: [linux-yocto] [PATCH] gpio-pca953x: add PCAL9535 Interrupt support

2016-03-29 Thread Bruce Ashfield
On 2016-03-29 9:47 PM, Yong Li wrote: Hi Bruce, This patch is based on the below patch in Kernel 3.8: commit f22e9ce7703448e1a2063fa171fb22dfef1da6dc Author: Josef Ahmad mailto:josef.ah...@linux.intel.com>> Date: Wed Apr 9 16:57:30 2014 +0100 Quark GPIO 1/2 Since there are s

Re: [linux-yocto] [PATCH] gpio-pca953x: add PCAL9535 Interrupt support

2016-03-29 Thread Yong Li
Hi Bruce, This patch is based on the below patch in Kernel 3.8: commit f22e9ce7703448e1a2063fa171fb22dfef1da6dc Author: Josef Ahmad Date: Wed Apr 9 16:57:30 2014 +0100 Quark GPIO 1/2 Since there are several changes in the drivers/gpio/gpio-pca953x.c file are not merged into upst

Re: [linux-yocto] [PATCH] gpio-pca953x: add PCAL9535 Interrupt support

2016-03-29 Thread Bruce Ashfield
On 2016-03-29 3:49 AM, Yong Li wrote: Dear Maintainers, This patch fixes the "some Galileo Gen 2 gpio cannot trigger interrupt" bug: https://jira01.devtools.intel.com/browse/IOTOS-1365. Please merge it into standard/base branch for linux-yocto-4.4 if this looks okay. What's the upstream sta

[linux-yocto] [PATCH] gpio-pca953x: add PCAL9535 Interrupt support

2016-03-29 Thread Yong Li
Galileo gen 2 uses the PCAL9535 as the GPIO expansion, it is different from PCA9535 and includes interrupt mask/status registers, The current driver does not support the interrupt registers configuration, it causes some gpio pins cannot trigger interrupt events, this patch fix this issue. Signed-o