[PATCH 40/54] ARM: Samsung SoCs: irq_data conversion.

'Lennert Buytenhek' buytenh at wantstofly.org
Tue Dec 21 05:24:10 EST 2010


On Tue, Dec 21, 2010 at 07:03:45PM +0900, Kukjin Kim wrote:

> > > (Added Mark Brown in Cc)
> > >
> > > Looks ok to me but there are some comments.
> > >
> > > Firstly, how about to separate this to regarding S3C24XX(mach-s3c24xx
> and
> > > plat-s3c24xx), S5P(mach-s5pxxxx and plat-s5p) and plat-samsung?
> > > (Actually, already applied S3C64XX changes so that we can drop that in
> your
> > > patch.)
> > >
> > > As you know, I already applied some changes which is from Mark Brown in
> my
> > > tree.
> > > You can find that in
> > > git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
> > > #for-next
> > >
> > > So could you please re-work this based on that?
> > 
> > I've reworked the patch I had in my tree on top of your git tree
> > (latest commit "Merge branch 'next-s5p64x0' into for-next"), and came
> > up with the three attached patches.  I've incorporated the whitespace
> > changes you requested, and changed the struct irq_data * variable
> > names from 'd' to 'data' to match Mark's earlier patches.
> > 
> > Caveat emptor -- uncompiled, and totally untested.
> > 
> > How's this look?
> > 
> > Feel free to merge this via your tree -- I'll drop the Samsung patch
> > from my tree then.
> 
> Hi Lennert :-)

Hello,


> I found your different e-mail in your patch like following.
> 
> "From: Lennert Buytenhek <buytenh at wantstofly.org>"
> ...
> "Signed-off-by: Lennert Buytenhek <buytenh at secretlab.ca>"
> 
> I think, should be used same e-mail address.
> Which one do you want to use?...

I don't think there is a requirement to use the same one for both,
but if it bothers you, please use the latter one.


thanks,
Lennert



More information about the linux-arm-kernel mailing list