On 09-06-10 10.57, Andrew Morton wrote:
> On Wed, 10 Jun 2009 10:48:58 +0200 Richard R__jfors 
> <[email protected]> wrote:
> 
>> On 09-06-10 01.09, Andrew Morton wrote:
>>> On Fri, 05 Jun 2009 15:40:19 +0200
>>> Richard R__jfors <[email protected]> wrote:
>>>
>>>> This patch splits xilinx_spi into three parts, an OF and a platform
>>>> driver and generic part.
>>>>
>>>> The generic part now also works on X86 and also supports the Xilinx
>>>> SPI IP DS570
>>> Unfortunately we already have two fairly significant
>>> drivers/spi/xilinx_spi.c patches queued in -mm:
>>> spi-move-more-spi_setup-functionality-into-core.patch and
>>> spi-move-common-spi_setup-functionality-into-core.patch.  Applying this
>>> patch on top of those makes rather a mess.
>>>
>>> Hopefully things will still be OK if I skip this patch.  You didn't
>>> identify the dependencies between those nine patchs.  Can you please
>>> explain that?
>> Actually all patches except for the 7th(of 9), would work without the
>> xilinx_spi-patch.
>> The 7th (Timberdale MFD) would not build. Some of the othe drivers are not
>> possible to select because they depend on the MFD in their Kconfigs,
>> but not the code.
> 
> Confused.

Sorry, will try to explain below.

> See, my problem is that normally I will hold off on sending a patch to
> its subsystem maintainer until its prerequisite patch(es) are in mainline.

That was what I started doing, the MFD-driver requires plaform-data structs
from some of the drivers.

On the other hand some of the drivers depends on the MFD conf-option, which
you found in the GPIO-driver :-)

> But for each of these patches, I don't know what the exact prerequisites
> are.  Help?

Patch dependencies
1/9 -> (mc33880) no dependencies on my patches
2/9 -> (xilinx_spi) no dependencies on my patches
3/9 -> (i2c-ocores) no dependencies on my patches
4/9 -> (timb-dma) Kconfig depends on config set by timberdale (patch 7/9)
5/9 -> (logiwin) code depends on timb-dma (patch 4/9)
6/9 -> (v4l2 Kconfig) Kconfig depends on config set by timb-dma (patch(4/9)
7/9 -> (timberdale) code depends on mc3380 (patch 1/9), xilinx_spi (patch 2/9)
        i2c-ocores (patch 3/9), logiwin (patch 5/9)
8/9 -> (timbserial) Kconfig depends on config set by timberdale (patch 7/9)
        Btw Alan Cox said i has queued it for the ttydev stack, and it is
        further patched.
9/9 -> (timbgpio) Kconfig depends on config set by timberdale (patch 7/9)

So my suggestion is;
Apply all but patch 2 and 7. They will apply, some have Kconfig dependencies
to patch 7.

>> I would be very happy if you applied all but the 7th (MFD) and 2nd (SPI) 
>> anyway.
>>
>> I will update my SPI-patch to work against the two patches you point out 
>> above.
>> And them I will resend the Timberdale MFD and the Xilinx SPI patch.
>>
>> Would that be OK?
> 
> That would be great, thanks.

I will come up with a patch as soon as possible.

Thanks
--Richard

------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
_______________________________________________
spi-devel-general mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/spi-devel-general

Reply via email to