| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-Id: <9A0C149F-8BF4-4970-859F-A8E59EAC31E4@dominion.thruhere.net> Date: Wed, 11 Sep 2013 08:06:15 +0200 From: Koen Kooi <koen@...inion.thruhere.net> To: Joel Fernandes <joelf@...com> Cc: Benoit Cousson <bcousson@...libre.com>, Nishanth Menon <nm@...com>, Devicetree Discuss <devicetree@...r.kernel.org>, Linux OMAP List <linux-omap@...r.kernel.org>, Linux ARM Kernel List <linux-arm-kernel@...ts.infradead.org>, Linux Kernel Mailing List <linux-kernel@...r.kernel.org>, Linux Documentation List <linux-doc@...r.kernel.org>, Linux MMC List <linux-mmc@...r.kernel.org>, Linux SPI Devel List <spi-devel-general@...ts.sourceforge.net> Subject: Re: [PATCH 0/3] ARM: dts: Enable EDMA, MMC and SPI on AM33XX for v3.13 Op 11 sep. 2013, om 08:00 heeft Joel Fernandes <joelf@...com> het volgende geschreven: > On 09/11/2013 12:18 AM, Koen Kooi wrote: >> >> Op 10 sep. 2013, om 22:14 heeft Joel Fernandes <joelf@...com> het volgende geschreven: >> >>> On 09/10/2013 02:39 PM, Koen Kooi wrote: >>>> >>>> Op 10 sep. 2013, om 21:24 heeft Joel Fernandes <joelf@...com> 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. > > There's nothing "broken" about $subject series. Please don't confuse maintainers > by using wrong words like that. This series is perfectly OK as such to be merged. But plugging in an SD card doesn't work in this series, I'd call that broken. > Further, I am puzzled by all this noise because card-detect additions were > initially agreed to be posted separately by you along with other custom DTS for > BBW MMC. I agreed to post the mmc fixes needed for BBW and BBB, so I assumed you'd drop all beaglebone entries from your series. > Its obvious I wouldn't squash patches that we _agreed_ you would send > out- and that are especially additions than any real fixes. Hopefully this makes > it clear, if you need any help please let me know. Right, so now your patch series will add a half-assed DT entry which only works if the uSD card is present at boot. Anyway, fixup series underway already > > Thanks! > > -Joel > > >>> >>> [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@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists