From mboxrd@z Thu Jan 1 00:00:00 1970 From: dinguyen@opensource.altera.com (Dinh Nguyen) Date: Wed, 10 Sep 2014 09:19:18 -0500 Subject: [GIT PULL 1/2] SOCFPGA updates for 3.18 In-Reply-To: <20140910110057.GA9354@amd> References: <1409847105-27241-1-git-send-email-dinguyen@opensource.altera.com> <20140909141756.GA30100@amd> <4134101.OOENTRabu1@wuerfel> <20140909210232.GA539@amd> <20140910083337.GA5178@amd> <20140910110057.GA9354@amd> Message-ID: <54105DE6.9070807@opensource.altera.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 9/10/14, 6:00 AM, Pavel Machek wrote: > Hi! > >>>> 2) On the host level, the support for handle multiple slots in DT for >>>> dw-mmc has been broken. While dw-mmc parsed the DT nodes for slots, it >>>> screwed up configurations. Thus the support for slots have never >>>> worked as expected from DT point of view. >>> >>> Well, DT is supposed to describe the hardware. From your description, >>> it seems that linux does not support two slots on one controller and >>> DT parsing code basically ignores the slots. (Logical, if it can't >>> support two slots). >>> >>> So now we are breaking DT description due to Linux limitations. Which >>> >>> a) makes it hard for any other os not having same limitation >>> >>> b) makes it hard for people to fix the limitation >>> >>> c) does not really solve anything >> >> Yes it does, the problem in 2) gets fixed. >> >>> >>> d) breaks backward compatibility with old dts >> >> According to 2), it has never worked - so we don't break anything. > > Umm? u-SD worked for me in 3.14, 3.15 and 3.16 with the device tree, > and with single-slot described in the device tree. Now I have to > change dtb-s to keep working configuration, which is something device > tree should never ever do. I tested the DTB with and without the patch, and it all works fine. Dinh > > Is it so hard to just fix the multiple slot parsing? > > Pavel >