Re: [PATCH 0/3] ARM: dts: Enable EDMA, MMC and SPI on AM33XX for v3.13

From: Koen Kooi
Date: Wed Sep 11 2013 - 01:18:43 EST



Op 10 sep. 2013, om 22:14 heeft Joel Fernandes <joelf@xxxxxx> het volgende geschreven:

> On 09/10/2013 02:39 PM, Koen Kooi wrote:
>>
>> Op 10 sep. 2013, om 21:24 heeft Joel Fernandes <joelf@xxxxxx> het volgende geschreven:
>>
>>> Here are last few patches required to add EDMA and MMC/SPI support for AM33xx.
>>>
>>> Now that all dependent DMA patches and fixes are in linux next or mainline, except
>>> for [1] which should go in for 3.12 -rc cycle, it is safe to enable MMC and SPI support
>>> and this patch series enables it. These are originally Matt Porter's patches with
>>> changes to make it work with recent kernels, addition of irq, memory resources and
>>> enable other extra properties.
>>>
>>> These patches should cleanly apply on master branch after Koen's patch [2] for basic
>>> BBB DT support is applied.
>>>
>>> MMC support is enabled for: Beaglebone, AM335x EVM and EVM-SK boards. MMC support
>>> for BBB is intentionally not added due to custom fixes and other patches that are
>>> in Koen's tree and which will be separately submitted by him.
>>
>> Correct, but your patches for MMC support on BBW are missing the card detect entries to make it hotplug work.
>
> I thought it was determined that this would be submitted by you separately after
> rebasing as we discussed [1] and [2].

I have no problem submitting that, I just think it's weird that the patch you submitted contains a known broken version for BBW.

>
> [1] https://lkml.org/lkml/2013/9/6/183
> [2] http://marc.info/?l=linux-omap&m=137879246709612&w=2
>
>
> Regards,
>
> -Joel

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/