Re: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif capture driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Em Wed, 19 Aug 2009 10:32:07 -0500
"Karicheri, Muralidharan" <m-karicheri2@xxxxxx> escreveu:

> Mauro,
> 
> Kevin has approved the architecture part of this patch. When can I expect these to be merged to linux-next?
> 
> Thanks.
> 
> Murali Karicheri
> Software Design Engineer
> Texas Instruments Inc.
> Germantown, MD 20874
> email: m-karicheri2@xxxxxx
> 
> >-----Original Message-----
> >From: Karicheri, Muralidharan
> >Sent: Tuesday, August 18, 2009 5:51 PM
> >To: 'Mauro Carvalho Chehab'
> >Cc: Mauro Carvalho Chehab; linux-media@xxxxxxxxxxxxxxx; davinci-linux-open-
> >source@xxxxxxxxxxxxxxxxxxxx; khilman@xxxxxxxxxxxxxxxxxxx; Hans Verkuil
> >Subject: RE: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif
> >capture driver
> >
> >Mauro,
> >
> >Here are the patches from Chaithrika that I am referring to.
> >http://www.mail-archive.com/linux-media@xxxxxxxxxxxxxxx/msg08254.html

There's something wrong with this patch:

$ patch -p1 -i 12453a.patch
patching file arch/arm/mach-davinci/board-dm646x-evm.c
Reversed (or previously applied) patch detected!  Assume -R? [n] y
Hunk #1 succeeded at 52 (offset -11 lines).
Hunk #2 succeeded at 218 with fuzz 1 (offset -70 lines).
Hunk #3 succeeded at 286 with fuzz 2 (offset -14 lines).
Hunk #4 FAILED at 293.
Hunk #5 succeeded at 254 (offset -79 lines).
1 out of 5 hunks FAILED -- saving rejects to file arch/arm/mach-davinci/board-dm646x-evm.c.rej
patching file arch/arm/mach-davinci/dm646x.c
Hunk #1 succeeded at 40 with fuzz 2 (offset 8 lines).
Hunk #2 succeeded at 550 with fuzz 1 (offset -145 lines).
Hunk #3 succeeded at 866 with fuzz 1 (offset 12 lines).
patching file arch/arm/mach-davinci/include/mach/dm646x.h
Hunk #1 succeeded at 47 with fuzz 2 (offset 18 lines).

It seems that this patch is not based on my linux-next -git tree. Probably,
this patch is dependent on some patch at Kevin tree.

Kevin,

As this patch touches only arch/arm/ stuff, I suspect that we'll have less
conflicts if you could merge this one. From my side:

Acked-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>

> >http://www.mail-archive.com/linux-media@xxxxxxxxxxxxxxx/msg07676.html

Hmm... the second patch shows that bisect will be broken with the platform
changes. This patch should be fold with the one that renamed the field, or
before Kconfig/Makefile changes.

I've applied this one on my tree, just before the Kbuild patch.

Due to the DaVinci dependency order, I'll need to hold the DaVinci patches at
the next upstream window to happen after Russell/Kevin trees, to avoid bisect
troubles



Cheers,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux