From mboxrd@z Thu Jan 1 00:00:00 1970 From: geomatsi@gmail.com (Sergey Matyukevich) Date: Mon, 13 Nov 2017 22:24:11 +0300 Subject: [PATCH 1/2] arm64: dts: orange-pi-zero-plus2: fix sdcard detect In-Reply-To: References: <20171103195855.15283-1-geomatsi@gmail.com> <20171103195855.15283-2-geomatsi@gmail.com> <20171104154221.ao7kkcwqhnja3cxe@speedy.hunter> <20171106122633.5mnqfh6xhmmxr45s@flea> <20171106161306.tq6yeq6bq36ridj3@speedy.hunter> <20171107201245.gptgfli54o6eqmrj@speedy.hunter> Message-ID: <20171113192411.kmohcthzickk4n5v@speedy.hunter> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hello Maxime, On Wed, Nov 08, 2017 at 04:55:32PM +0530, Jagan Teki wrote: > On Wed, Nov 8, 2017 at 1:42 AM, Sergey Matyukevich wrote: > > Hi, > > > >> >>> > Did you try to boot from sdcard ? I am not able to boot my board from > >> >>> > sd-card without this change. As I mentioned earlier in my email > >> >>> > to buildroot mailing list, with mmc debug enabled I see that mmc > >> >>> > tries to init sd-card when I remove it from the slot. > >> >>> > > >> >>> > Maybe there is a minor differences between h/w versions or batches ? > >> >>> > What is you board version ? I tested on board v1.0. > >> >>> > >> >>> Looking at the schematics, it definitely looks like it's active low. > >> >> > >> >> Is it ok to merge it then ? Or using 'cd-inverted' property > >> >> is the preferable option ? > >> > > >> > W/o any change mainline works unstable for me, it wasn't booting > >> > at-all[1] did you find the same? > >> > > >> > Even tried with active LOW and cd-inverted. > >> > > >> > [1] https://paste.ubuntu.com/25909064/ > >> > >> Look like something broken for H5 and A64 between v4.14-rc8 to latest > > > > Both 4.13.7 and 4.14-rc8 (synched today) kernels worked fine for me. > > DTS behavior is all the same: > > I've tried fresh sync [2] but still see the issue, can you check the > Image size of log(suspecting on that area) Do you plan to accept this patch as well ? Or you would prefer to wait for the confirmation from Jagan as well ? Regards, Sergey