From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andre Heider Date: Fri, 11 Sep 2020 17:54:05 +0200 Subject: [PATCH] mmc: xenon_sdhci: Add missing common host capabilities In-Reply-To: <352f97ed-0f7c-df3b-2938-18f3c1b19678@denx.de> References: <20200910175340.515227-1-a.heider@gmail.com> <20200910190420.rcllylp3ct323rsr@pali> <144a2c4b-e06b-5d89-1a80-f1ec7e846e07@denx.de> <64adca11-c77e-feb0-5b54-4fc97d3529b0@gmail.com> <20200911140930.0d2e8029@dellmb.labs.office.nic.cz> <352f97ed-0f7c-df3b-2938-18f3c1b19678@denx.de> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 11/09/2020 14:21, Stefan Roese wrote: > On 11.09.20 14:09, Marek Beh?n wrote: >> On Fri, 11 Sep 2020 10:37:42 +0200 >> Andre Heider wrote: >> >>> On 11/09/2020 08:43, Stefan Roese wrote: >>>> Hi Pali, Andre and others, >>>> >>>> On 10.09.20 21:04, Pali Roh?r wrote: >>>>> On Thursday 10 September 2020 19:53:40 Andre Heider wrote: >>>>>> Use mmc_of_parse() to set the common host properties. That >>>>>> includes "bus-width", so parsing it can be removed from the >>>>>> driver. >>>>>> >>>>>> But more importantly, "non-removable" is now respected, which >>>>>> fixes the usage of eMMC. >>>>>> >>>>>> Signed-off-by: Andre Heider >>>>>> --- >>>>> >>>>> Adding Marek to loop, can you test this patch on MOX A to verify >>>>> that it does not break MOX uSD and SDIO support? >>>>> >>>>> (PS: Andre, for one week I would not be available, therefore in >>>>> this time I would not be able to test or review Armada 3720 >>>>> patches...) >>>> >>>> I'll also be on vacation next week. Sorry, I was not able to follow >>>> all discussions on these Armada related patches lately. Could >>>> someone please summarize, which patches (fixes) are "okay" for >>>> merging in this rc stage (rc4)? That would really be helpful. >>> >>> Going by patchwork we have: >>> >>> All superseded: >>> https://patchwork.ozlabs.org/project/uboot/list/?series=198544 >>> https://patchwork.ozlabs.org/project/uboot/list/?series=198545 >>> https://patchwork.ozlabs.org/project/uboot/list/?series=198756 >>> https://patchwork.ozlabs.org/project/uboot/list/?series=199690 >>> >>> ? From my point of view these can go in: >>> >>> This one, fix emmc: >>> https://patchwork.ozlabs.org/project/uboot/list/?series=200919 >> >> I shall check this if it does not break our boards. >> >>> dts: fix vendor, add -emmc variant: >>> https://patchwork.ozlabs.org/project/uboot/list/?series=199579 >>> >>> These two for uDPU from its maintainer, taken from the OpenWRT >>> uboot-mvebu package: >>> CONFIG_SYS_BOOTM_LEN to 64MB: >>> https://patchwork.ozlabs.org/project/uboot/list/?series=199858 >>> add support for Macronix mx25u12835f flash: >>> https://patchwork.ozlabs.org/project/uboot/list/?series=199857 >>> >>> And I'd take these two too, Pali has different opinion though: >>> enable NET_RANDOM_ETHADDR: >>> https://patchwork.ozlabs.org/project/uboot/list/?series=200115 >>> set fdtfile: >>> https://patchwork.ozlabs.org/project/uboot/list/?series=201051 >> >> I have replied to the NET_RANDOM_ETHADDR patch, please comment on my >> reply. >> >> As for the fdtfile I think it is good to go. > > Thanks. Unfortunately this is taking a bit too long for me, as I > have to leave in about an hour. I will take care of these patches > in a bit over a week. Sounds good, thanks! Have a nice vacation, Andre