lkml.org 
[lkml]   [2012]   [Mar]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH 1/2] gpio/davinci: fix oops on unbanked gpio irq request
Date
Hi Grant,

On Mon, Mar 12, 2012 at 23:21:35, Grant Likely wrote:
> On Sun, 11 Mar 2012 18:16:11 +0530, Sekhar Nori <nsekhar@ti.com> wrote:
> > Unbanked GPIO irq setup code was overwriting chip_data leading
> > to the following oops on request_irq()
> >
> > Unable to handle kernel paging request at virtual address febfffff
> [...]
> > Fix the issue.
>
> Applied, but for next time "Fix the issue" isn't the most helpful commit text.
> Please describe *how* you fixed the issue so that future commit readers have
> more clues as to what this patch is doing.

Thanks for the feedback and noted for future. If it's not too much trouble,
can you change the last line in commit message to:

Fix the issue by making unbanked GPIO code not overwrite chip_data because
generic irq chip code uses it for its data structures. Instead use
handler_data to pass around GPIO controller information.

Regards,
Sekhar


\
 
 \ /
  Last update: 2012-03-12 19:57    [W:0.032 / U:1.696 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site