* [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino @ 2019-07-27 21:03 Sunil Mohan Adapa 2019-08-01 2:42 ` Chen-Yu Tsai 0 siblings, 1 reply; 19+ messages in thread From: Sunil Mohan Adapa @ 2019-07-27 21:03 UTC (permalink / raw) To: linux-arm-kernel; +Cc: maxime.ripard, Sunil Mohan Adapa, wens A64-OLinuXino board has three variants that have eMMC support. Add support for eMMC on boards that have it. This patch has been tested on A64-OLinuXino-1Ge16GW with Linux 5.0 from Debain. Basic benchmarks using Flexible IO Tester show reasonable performance from the eMMC. eMMC - Random Write: 23.1MiB/s eMMC - Sequential Write: 74.5MiB/s SD Card - Random Write: 1690KiB/s SD Card - Sequential Write: 11.0MiB/s v2: Fix descriptions for VCC and VCCQ. [sunil@medhas.org Fix descriptions for VCC and VCCQ] Signed-off-by: Sunil Mohan Adapa <sunil@medhas.org> Tested-by: Sunil Mohan Adapa <sunil@medhas.org> From: Martin Ayotte <martinayotte@gmail.com> Link: https://github.com/armbian/build/commit/174953de1eb09e6aa1ef7075066b573dba625398 --- .../arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts b/arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts index 01a9a52edae4..28bb67ebe173 100644 --- a/arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts @@ -155,6 +155,17 @@ }; }; +&mmc2 { + pinctrl-names = "default"; + pinctrl-0 = <&mmc2_pins>; + vmmc-supply = <®_dcdc1>; + vqmmc-supply = <®_dcdc1>; + bus-width = <8>; + non-removable; + cap-mmc-hw-reset; + status = "okay"; +}; + &ohci0 { status = "okay"; }; -- 2.20.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino 2019-07-27 21:03 [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino Sunil Mohan Adapa @ 2019-08-01 2:42 ` Chen-Yu Tsai 2019-08-01 14:05 ` Martin Ayotte [not found] ` <CAKQ8BtjLmhDgA8woY5NqaifODLUh_w_K4QYOUuqc4Six5Amerg@mail.gmail.com> 0 siblings, 2 replies; 19+ messages in thread From: Chen-Yu Tsai @ 2019-08-01 2:42 UTC (permalink / raw) To: Sunil Mohan Adapa; +Cc: Maxime Ripard, Martin Ayotte, linux-arm-kernel On Sun, Jul 28, 2019 at 5:03 AM Sunil Mohan Adapa <sunil@medhas.org> wrote: > > A64-OLinuXino board has three variants that have eMMC support. Add support for > eMMC on boards that have it. > > This patch has been tested on A64-OLinuXino-1Ge16GW with Linux 5.0 from Debain. > Basic benchmarks using Flexible IO Tester show reasonable performance from the > eMMC. > > eMMC - Random Write: 23.1MiB/s > eMMC - Sequential Write: 74.5MiB/s > SD Card - Random Write: 1690KiB/s > SD Card - Sequential Write: 11.0MiB/s > > v2: Fix descriptions for VCC and VCCQ. > > [sunil@medhas.org Fix descriptions for VCC and VCCQ] > Signed-off-by: Sunil Mohan Adapa <sunil@medhas.org> > Tested-by: Sunil Mohan Adapa <sunil@medhas.org> > > From: Martin Ayotte <martinayotte@gmail.com> > Link: https://github.com/armbian/build/commit/174953de1eb09e6aa1ef7075066b573dba625398 Thanks. The patch looks good overall. The authorship is a little confusing though. If it was initially done by Martin (CC-ed), then he should be the author, and we should get his Signed-off-by if possible. ChenYu _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 19+ messages in thread
* RE: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino 2019-08-01 2:42 ` Chen-Yu Tsai @ 2019-08-01 14:05 ` Martin Ayotte [not found] ` <CAKQ8BtjLmhDgA8woY5NqaifODLUh_w_K4QYOUuqc4Six5Amerg@mail.gmail.com> 1 sibling, 0 replies; 19+ messages in thread From: Martin Ayotte @ 2019-08-01 14:05 UTC (permalink / raw) To: 'Chen-Yu Tsai', 'Sunil Mohan Adapa' Cc: 'Maxime Ripard', 'linux-arm-kernel' If my SOB could help here, I don't mind since I've done the commit more than a year ago for Armbian ... Signed-off-by: Martin Ayotte <martinayotte@gmai.com> Tested-by: Martin Ayotte <martinayotte@gmai.com> -----Message d'origine----- De : Chen-Yu Tsai [mailto:wens@csie.org] Envoyé : Wednesday, July 31, 2019 10:42 PM À : Sunil Mohan Adapa Cc : linux-arm-kernel; Maxime Ripard; Martin Ayotte Objet : Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino On Sun, Jul 28, 2019 at 5:03 AM Sunil Mohan Adapa <sunil@medhas.org> wrote: > > A64-OLinuXino board has three variants that have eMMC support. Add support for > eMMC on boards that have it. > > This patch has been tested on A64-OLinuXino-1Ge16GW with Linux 5.0 from Debain. > Basic benchmarks using Flexible IO Tester show reasonable performance from the > eMMC. > > eMMC - Random Write: 23.1MiB/s > eMMC - Sequential Write: 74.5MiB/s > SD Card - Random Write: 1690KiB/s > SD Card - Sequential Write: 11.0MiB/s > > v2: Fix descriptions for VCC and VCCQ. > > [sunil@medhas.org Fix descriptions for VCC and VCCQ] > Signed-off-by: Sunil Mohan Adapa <sunil@medhas.org> > Tested-by: Sunil Mohan Adapa <sunil@medhas.org> > > From: Martin Ayotte <martinayotte@gmail.com> > Link: https://github.com/armbian/build/commit/174953de1eb09e6aa1ef7075066b573dba62 5398 Thanks. The patch looks good overall. The authorship is a little confusing though. If it was initially done by Martin (CC-ed), then he should be the author, and we should get his Signed-off-by if possible. ChenYu _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 19+ messages in thread
[parent not found: <CAKQ8BtjLmhDgA8woY5NqaifODLUh_w_K4QYOUuqc4Six5Amerg@mail.gmail.com>]
* Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino [not found] ` <CAKQ8BtjLmhDgA8woY5NqaifODLUh_w_K4QYOUuqc4Six5Amerg@mail.gmail.com> @ 2019-08-01 18:47 ` Sunil Mohan Adapa 2019-08-02 12:45 ` Martin Ayotte 2019-08-05 3:33 ` Chen-Yu Tsai 0 siblings, 2 replies; 19+ messages in thread From: Sunil Mohan Adapa @ 2019-08-01 18:47 UTC (permalink / raw) To: Martin Ayotte, Chen-Yu Tsai; +Cc: Maxime Ripard, linux-arm-kernel [-- Attachment #1.1.1: Type: text/plain, Size: 683 bytes --] On 01/08/19 6:49 am, Martin Ayotte wrote: > If my SOB could help here, I don't mind since I've done the commit > more than a year ago for Armbian ... > > Signed-off-by: Martin Ayotte <martinayotte@gmai.com> > Tested-by: Martin Ayotte <martinayotte@gmai.com> gmai.com is likely a typo. > On Wed, Jul 31, 2019 at 10:42 PM Chen-Yu Tsai <wens@csie.org > >> Thanks. The patch looks good overall. The authorship is a little >> confusing though. If it was initially done by Martin (CC-ed), then >> he should be the author, and we should get his Signed-off-by if >> possible. Martin is indeed the original author of the patch. Thank you for reviewing. -- Sunil [-- Attachment #1.2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 833 bytes --] [-- Attachment #2: Type: text/plain, Size: 176 bytes --] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 19+ messages in thread
* RE: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino 2019-08-01 18:47 ` Sunil Mohan Adapa @ 2019-08-02 12:45 ` Martin Ayotte 2019-08-05 3:33 ` Chen-Yu Tsai 1 sibling, 0 replies; 19+ messages in thread From: Martin Ayotte @ 2019-08-02 12:45 UTC (permalink / raw) To: 'Sunil Mohan Adapa', 'Chen-Yu Tsai' Cc: 'Maxime Ripard', 'linux-arm-kernel' > gmai.com is likely a typo. Right ! Signed-off-by: Martin Ayotte <martinayotte@gmail.com> Tested-by: Martin Ayotte <martinayotte@gmail.com> -----Message d'origine----- De : Sunil Mohan Adapa [mailto:sunil@medhas.org] Envoyé : Thursday, August 01, 2019 2:48 PM À : Martin Ayotte; Chen-Yu Tsai Cc : linux-arm-kernel; Maxime Ripard Objet : Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino On 01/08/19 6:49 am, Martin Ayotte wrote: > If my SOB could help here, I don't mind since I've done the commit > more than a year ago for Armbian ... > > Signed-off-by: Martin Ayotte <martinayotte@gmai.com> > Tested-by: Martin Ayotte <martinayotte@gmai.com> gmai.com is likely a typo. > On Wed, Jul 31, 2019 at 10:42 PM Chen-Yu Tsai <wens@csie.org > >> Thanks. The patch looks good overall. The authorship is a little >> confusing though. If it was initially done by Martin (CC-ed), then >> he should be the author, and we should get his Signed-off-by if >> possible. Martin is indeed the original author of the patch. Thank you for reviewing. -- Sunil _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino 2019-08-01 18:47 ` Sunil Mohan Adapa 2019-08-02 12:45 ` Martin Ayotte @ 2019-08-05 3:33 ` Chen-Yu Tsai 2019-08-05 5:24 ` Sunil Mohan Adapa 1 sibling, 1 reply; 19+ messages in thread From: Chen-Yu Tsai @ 2019-08-05 3:33 UTC (permalink / raw) To: Sunil Mohan Adapa; +Cc: Maxime Ripard, Martin Ayotte, linux-arm-kernel On Fri, Aug 2, 2019 at 2:47 AM Sunil Mohan Adapa <sunil@medhas.org> wrote: > > On 01/08/19 6:49 am, Martin Ayotte wrote: > > If my SOB could help here, I don't mind since I've done the commit > > more than a year ago for Armbian ... > > > > Signed-off-by: Martin Ayotte <martinayotte@gmai.com> > > Tested-by: Martin Ayotte <martinayotte@gmai.com> > gmai.com is likely a typo. > > > On Wed, Jul 31, 2019 at 10:42 PM Chen-Yu Tsai <wens@csie.org > > > >> Thanks. The patch looks good overall. The authorship is a little > >> confusing though. If it was initially done by Martin (CC-ed), then > >> he should be the author, and we should get his Signed-off-by if > >> possible. > > Martin is indeed the original author of the patch. Thank you for reviewing. I'd like to apply this patch with Martin as the author, if that's OK with you both? Thanks ChenYu > -- > Sunil > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino 2019-08-05 3:33 ` Chen-Yu Tsai @ 2019-08-05 5:24 ` Sunil Mohan Adapa 2019-08-05 12:58 ` Martin Ayotte 0 siblings, 1 reply; 19+ messages in thread From: Sunil Mohan Adapa @ 2019-08-05 5:24 UTC (permalink / raw) To: Chen-Yu Tsai; +Cc: Maxime Ripard, Martin Ayotte, linux-arm-kernel [-- Attachment #1.1.1: Type: text/plain, Size: 978 bytes --] On 04/08/19 8:33 pm, Chen-Yu Tsai wrote: > On Fri, Aug 2, 2019 at 2:47 AM Sunil Mohan Adapa <sunil@medhas.org> wrote: >> >> On 01/08/19 6:49 am, Martin Ayotte wrote: >>> If my SOB could help here, I don't mind since I've done the commit >>> more than a year ago for Armbian ... >>> >>> Signed-off-by: Martin Ayotte <martinayotte@gmai.com> >>> Tested-by: Martin Ayotte <martinayotte@gmai.com> >> gmai.com is likely a typo. >> >>> On Wed, Jul 31, 2019 at 10:42 PM Chen-Yu Tsai <wens@csie.org >>> >>>> Thanks. The patch looks good overall. The authorship is a little >>>> confusing though. If it was initially done by Martin (CC-ed), then >>>> he should be the author, and we should get his Signed-off-by if >>>> possible. >> >> Martin is indeed the original author of the patch. Thank you for reviewing. > > I'd like to apply this patch with Martin as the author, if that's OK with you > both? That is completely okay with me. Thanks, -- Sunil [-- Attachment #1.2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 833 bytes --] [-- Attachment #2: Type: text/plain, Size: 176 bytes --] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 19+ messages in thread
* RE: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino 2019-08-05 5:24 ` Sunil Mohan Adapa @ 2019-08-05 12:58 ` Martin Ayotte 2019-08-06 6:25 ` Chen-Yu Tsai 0 siblings, 1 reply; 19+ messages in thread From: Martin Ayotte @ 2019-08-05 12:58 UTC (permalink / raw) To: 'Sunil Mohan Adapa', 'Chen-Yu Tsai' Cc: 'Maxime Ripard', 'linux-arm-kernel' Fine for me too. Thanks . -----Message d'origine----- De : Sunil Mohan Adapa [mailto:sunil@medhas.org] Envoyé : Monday, August 05, 2019 1:25 AM À : Chen-Yu Tsai Cc : Maxime Ripard; Martin Ayotte; linux-arm-kernel Objet : Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino On 04/08/19 8:33 pm, Chen-Yu Tsai wrote: > On Fri, Aug 2, 2019 at 2:47 AM Sunil Mohan Adapa <sunil@medhas.org> wrote: >> >> On 01/08/19 6:49 am, Martin Ayotte wrote: >>> If my SOB could help here, I don't mind since I've done the commit >>> more than a year ago for Armbian ... >>> >>> Signed-off-by: Martin Ayotte <martinayotte@gmai.com> >>> Tested-by: Martin Ayotte <martinayotte@gmai.com> >> gmai.com is likely a typo. >> >>> On Wed, Jul 31, 2019 at 10:42 PM Chen-Yu Tsai <wens@csie.org >>> >>>> Thanks. The patch looks good overall. The authorship is a little >>>> confusing though. If it was initially done by Martin (CC-ed), then >>>> he should be the author, and we should get his Signed-off-by if >>>> possible. >> >> Martin is indeed the original author of the patch. Thank you for reviewing. > > I'd like to apply this patch with Martin as the author, if that's OK with you > both? That is completely okay with me. Thanks, -- Sunil _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino 2019-08-05 12:58 ` Martin Ayotte @ 2019-08-06 6:25 ` Chen-Yu Tsai 2019-08-06 12:49 ` Martin Ayotte 2019-08-07 12:01 ` Maxime Ripard 0 siblings, 2 replies; 19+ messages in thread From: Chen-Yu Tsai @ 2019-08-06 6:25 UTC (permalink / raw) To: Martin Ayotte; +Cc: Maxime Ripard, Sunil Mohan Adapa, linux-arm-kernel On Mon, Aug 5, 2019 at 8:58 PM Martin Ayotte <martinayotte@gmail.com> wrote: > > Fine for me too. > > Thanks . > > -----Message d'origine----- > De : Sunil Mohan Adapa [mailto:sunil@medhas.org] > Envoyé : Monday, August 05, 2019 1:25 AM > À : Chen-Yu Tsai > Cc : Maxime Ripard; Martin Ayotte; linux-arm-kernel > Objet : Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on > A64-OLinuXino > > On 04/08/19 8:33 pm, Chen-Yu Tsai wrote: > > On Fri, Aug 2, 2019 at 2:47 AM Sunil Mohan Adapa <sunil@medhas.org> wrote: > >> > >> On 01/08/19 6:49 am, Martin Ayotte wrote: > >>> If my SOB could help here, I don't mind since I've done the commit > >>> more than a year ago for Armbian ... > >>> > >>> Signed-off-by: Martin Ayotte <martinayotte@gmai.com> > >>> Tested-by: Martin Ayotte <martinayotte@gmai.com> > >> gmai.com is likely a typo. > >> > >>> On Wed, Jul 31, 2019 at 10:42 PM Chen-Yu Tsai <wens@csie.org > >>> > >>>> Thanks. The patch looks good overall. The authorship is a little > >>>> confusing though. If it was initially done by Martin (CC-ed), then > >>>> he should be the author, and we should get his Signed-off-by if > >>>> possible. > >> > >> Martin is indeed the original author of the patch. Thank you for > reviewing. > > > > I'd like to apply this patch with Martin as the author, if that's OK with > you > > both? > > That is completely okay with me. Applied for 5.4. I reordered the tags so they make more sense: https://git.kernel.org/pub/scm/linux/kernel/git/sunxi/linux.git/commit/?h=sunxi/dt-for-5.4&id=0834887732df5af41b59b2e4d530fc1f5478965f _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 19+ messages in thread
* RE: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino 2019-08-06 6:25 ` Chen-Yu Tsai @ 2019-08-06 12:49 ` Martin Ayotte 2019-08-07 6:18 ` Chen-Yu Tsai 2019-08-07 12:01 ` Maxime Ripard 1 sibling, 1 reply; 19+ messages in thread From: Martin Ayotte @ 2019-08-06 12:49 UTC (permalink / raw) To: 'Chen-Yu Tsai' Cc: 'Maxime Ripard', 'Sunil Mohan Adapa', 'linux-arm-kernel' Just to let you know : typo in my email still there... -----Message d'origine----- De : Chen-Yu Tsai [mailto:wens@csie.org] Envoyé : Tuesday, August 06, 2019 2:25 AM À : Martin Ayotte Cc : Sunil Mohan Adapa; Maxime Ripard; linux-arm-kernel Objet : Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino On Mon, Aug 5, 2019 at 8:58 PM Martin Ayotte <martinayotte@gmail.com> wrote: > > Fine for me too. > > Thanks . > > -----Message d'origine----- > De : Sunil Mohan Adapa [mailto:sunil@medhas.org] > Envoyé : Monday, August 05, 2019 1:25 AM > À : Chen-Yu Tsai > Cc : Maxime Ripard; Martin Ayotte; linux-arm-kernel > Objet : Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on > A64-OLinuXino > > On 04/08/19 8:33 pm, Chen-Yu Tsai wrote: > > On Fri, Aug 2, 2019 at 2:47 AM Sunil Mohan Adapa <sunil@medhas.org> wrote: > >> > >> On 01/08/19 6:49 am, Martin Ayotte wrote: > >>> If my SOB could help here, I don't mind since I've done the commit > >>> more than a year ago for Armbian ... > >>> > >>> Signed-off-by: Martin Ayotte <martinayotte@gmai.com> > >>> Tested-by: Martin Ayotte <martinayotte@gmai.com> > >> gmai.com is likely a typo. > >> > >>> On Wed, Jul 31, 2019 at 10:42 PM Chen-Yu Tsai <wens@csie.org > >>> > >>>> Thanks. The patch looks good overall. The authorship is a little > >>>> confusing though. If it was initially done by Martin (CC-ed), then > >>>> he should be the author, and we should get his Signed-off-by if > >>>> possible. > >> > >> Martin is indeed the original author of the patch. Thank you for > reviewing. > > > > I'd like to apply this patch with Martin as the author, if that's OK with > you > > both? > > That is completely okay with me. Applied for 5.4. I reordered the tags so they make more sense: https://git.kernel.org/pub/scm/linux/kernel/git/sunxi/linux.git/commit/?h=su nxi/dt-for-5.4&id=0834887732df5af41b59b2e4d530fc1f5478965f _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino 2019-08-06 12:49 ` Martin Ayotte @ 2019-08-07 6:18 ` Chen-Yu Tsai 0 siblings, 0 replies; 19+ messages in thread From: Chen-Yu Tsai @ 2019-08-07 6:18 UTC (permalink / raw) To: Martin Ayotte; +Cc: Maxime Ripard, Sunil Mohan Adapa, linux-arm-kernel On Tue, Aug 6, 2019 at 8:49 PM Martin Ayotte <martinayotte@gmail.com> wrote: > > Just to let you know : typo in my email still there... Duh. Fixed it now. https://git.kernel.org/pub/scm/linux/kernel/git/sunxi/linux.git/commit/?h=sunxi/dt-for-5.4&id=8d3071f3e85894be35a1b35bcf6fdef970c81018 > -----Message d'origine----- > De : Chen-Yu Tsai [mailto:wens@csie.org] > Envoyé : Tuesday, August 06, 2019 2:25 AM > À : Martin Ayotte > Cc : Sunil Mohan Adapa; Maxime Ripard; linux-arm-kernel > Objet : Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on > A64-OLinuXino > > On Mon, Aug 5, 2019 at 8:58 PM Martin Ayotte <martinayotte@gmail.com> wrote: > > > > Fine for me too. > > > > Thanks . > > > > -----Message d'origine----- > > De : Sunil Mohan Adapa [mailto:sunil@medhas.org] > > Envoyé : Monday, August 05, 2019 1:25 AM > > À : Chen-Yu Tsai > > Cc : Maxime Ripard; Martin Ayotte; linux-arm-kernel > > Objet : Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on > > A64-OLinuXino > > > > On 04/08/19 8:33 pm, Chen-Yu Tsai wrote: > > > On Fri, Aug 2, 2019 at 2:47 AM Sunil Mohan Adapa <sunil@medhas.org> > wrote: > > >> > > >> On 01/08/19 6:49 am, Martin Ayotte wrote: > > >>> If my SOB could help here, I don't mind since I've done the commit > > >>> more than a year ago for Armbian ... > > >>> > > >>> Signed-off-by: Martin Ayotte <martinayotte@gmai.com> > > >>> Tested-by: Martin Ayotte <martinayotte@gmai.com> > > >> gmai.com is likely a typo. > > >> > > >>> On Wed, Jul 31, 2019 at 10:42 PM Chen-Yu Tsai <wens@csie.org > > >>> > > >>>> Thanks. The patch looks good overall. The authorship is a little > > >>>> confusing though. If it was initially done by Martin (CC-ed), then > > >>>> he should be the author, and we should get his Signed-off-by if > > >>>> possible. > > >> > > >> Martin is indeed the original author of the patch. Thank you for > > reviewing. > > > > > > I'd like to apply this patch with Martin as the author, if that's OK > with > > you > > > both? > > > > That is completely okay with me. > > Applied for 5.4. > > I reordered the tags so they make more sense: > > https://git.kernel.org/pub/scm/linux/kernel/git/sunxi/linux.git/commit/?h=su > nxi/dt-for-5.4&id=0834887732df5af41b59b2e4d530fc1f5478965f > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino 2019-08-06 6:25 ` Chen-Yu Tsai 2019-08-06 12:49 ` Martin Ayotte @ 2019-08-07 12:01 ` Maxime Ripard 2019-08-07 12:09 ` Chen-Yu Tsai 1 sibling, 1 reply; 19+ messages in thread From: Maxime Ripard @ 2019-08-07 12:01 UTC (permalink / raw) To: Chen-Yu Tsai; +Cc: Sunil Mohan Adapa, Martin Ayotte, linux-arm-kernel On Tue, Aug 06, 2019 at 02:25:17PM +0800, Chen-Yu Tsai wrote: > On Mon, Aug 5, 2019 at 8:58 PM Martin Ayotte <martinayotte@gmail.com> wrote: > > > > Fine for me too. > > > > Thanks . > > > > -----Message d'origine----- > > De : Sunil Mohan Adapa [mailto:sunil@medhas.org] > > Envoyé : Monday, August 05, 2019 1:25 AM > > À : Chen-Yu Tsai > > Cc : Maxime Ripard; Martin Ayotte; linux-arm-kernel > > Objet : Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on > > A64-OLinuXino > > > > On 04/08/19 8:33 pm, Chen-Yu Tsai wrote: > > > On Fri, Aug 2, 2019 at 2:47 AM Sunil Mohan Adapa <sunil@medhas.org> wrote: > > >> > > >> On 01/08/19 6:49 am, Martin Ayotte wrote: > > >>> If my SOB could help here, I don't mind since I've done the commit > > >>> more than a year ago for Armbian ... > > >>> > > >>> Signed-off-by: Martin Ayotte <martinayotte@gmai.com> > > >>> Tested-by: Martin Ayotte <martinayotte@gmai.com> > > >> gmai.com is likely a typo. > > >> > > >>> On Wed, Jul 31, 2019 at 10:42 PM Chen-Yu Tsai <wens@csie.org > > >>> > > >>>> Thanks. The patch looks good overall. The authorship is a little > > >>>> confusing though. If it was initially done by Martin (CC-ed), then > > >>>> he should be the author, and we should get his Signed-off-by if > > >>>> possible. > > >> > > >> Martin is indeed the original author of the patch. Thank you for > > reviewing. > > > > > > I'd like to apply this patch with Martin as the author, if that's OK with > > you > > > both? > > > > That is completely okay with me. > > Applied for 5.4. > > I reordered the tags so they make more sense: > > https://git.kernel.org/pub/scm/linux/kernel/git/sunxi/linux.git/commit/?h=sunxi/dt-for-5.4&id=0834887732df5af41b59b2e4d530fc1f5478965f Sorry for being late on this, but it looks like the eMMC, NAND and SPI pins are conflicting on the A64-Olinuxino design. There's no configuration with a NAND, so we don't really need to worry about that, but if we merge this in the main DT, we'll prevent anyone from using that DT on an olinuxino with a SPI flash. I think we should just create emmc and SPI-flash variants of that DT. Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino 2019-08-07 12:01 ` Maxime Ripard @ 2019-08-07 12:09 ` Chen-Yu Tsai 2019-08-07 12:41 ` Martin Ayotte 2019-08-07 14:45 ` Maxime Ripard 0 siblings, 2 replies; 19+ messages in thread From: Chen-Yu Tsai @ 2019-08-07 12:09 UTC (permalink / raw) To: Maxime Ripard; +Cc: Sunil Mohan Adapa, Martin Ayotte, linux-arm-kernel On Wed, Aug 7, 2019 at 8:01 PM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > > On Tue, Aug 06, 2019 at 02:25:17PM +0800, Chen-Yu Tsai wrote: > > On Mon, Aug 5, 2019 at 8:58 PM Martin Ayotte <martinayotte@gmail.com> wrote: > > > > > > Fine for me too. > > > > > > Thanks . > > > > > > -----Message d'origine----- > > > De : Sunil Mohan Adapa [mailto:sunil@medhas.org] > > > Envoyé : Monday, August 05, 2019 1:25 AM > > > Ą : Chen-Yu Tsai > > > Cc : Maxime Ripard; Martin Ayotte; linux-arm-kernel > > > Objet : Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on > > > A64-OLinuXino > > > > > > On 04/08/19 8:33 pm, Chen-Yu Tsai wrote: > > > > On Fri, Aug 2, 2019 at 2:47 AM Sunil Mohan Adapa <sunil@medhas.org> wrote: > > > >> > > > >> On 01/08/19 6:49 am, Martin Ayotte wrote: > > > >>> If my SOB could help here, I don't mind since I've done the commit > > > >>> more than a year ago for Armbian ... > > > >>> > > > >>> Signed-off-by: Martin Ayotte <martinayotte@gmai.com> > > > >>> Tested-by: Martin Ayotte <martinayotte@gmai.com> > > > >> gmai.com is likely a typo. > > > >> > > > >>> On Wed, Jul 31, 2019 at 10:42 PM Chen-Yu Tsai <wens@csie.org > > > >>> > > > >>>> Thanks. The patch looks good overall. The authorship is a little > > > >>>> confusing though. If it was initially done by Martin (CC-ed), then > > > >>>> he should be the author, and we should get his Signed-off-by if > > > >>>> possible. > > > >> > > > >> Martin is indeed the original author of the patch. Thank you for > > > reviewing. > > > > > > > > I'd like to apply this patch with Martin as the author, if that's OK with > > > you > > > > both? > > > > > > That is completely okay with me. > > > > Applied for 5.4. > > > > I reordered the tags so they make more sense: > > > > https://git.kernel.org/pub/scm/linux/kernel/git/sunxi/linux.git/commit/?h=sunxi/dt-for-5.4&id=0834887732df5af41b59b2e4d530fc1f5478965f > > Sorry for being late on this, but it looks like the eMMC, NAND and SPI > pins are conflicting on the A64-Olinuxino design. > > There's no configuration with a NAND, so we don't really need to worry > about that, but if we merge this in the main DT, we'll prevent anyone > from using that DT on an olinuxino with a SPI flash. > > I think we should just create emmc and SPI-flash variants of that DT. Actually they aren't. Olimex specifically uses eMMC modules that don't use the data strobe line, so SPI can be used together. ChenYu _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 19+ messages in thread
* RE: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino 2019-08-07 12:09 ` Chen-Yu Tsai @ 2019-08-07 12:41 ` Martin Ayotte 2019-08-07 14:45 ` Maxime Ripard 1 sibling, 0 replies; 19+ messages in thread From: Martin Ayotte @ 2019-08-07 12:41 UTC (permalink / raw) To: 'Chen-Yu Tsai', 'Maxime Ripard' Cc: 'Sunil Mohan Adapa', 'linux-arm-kernel' Right ! I've even stuffed the missing SPINOR on my board and used an overlay to access it without any issue ... -----Message d'origine----- De : Chen-Yu Tsai [mailto:wens@csie.org] Envoyé : Wednesday, August 07, 2019 8:09 AM À : Maxime Ripard Cc : Martin Ayotte; Sunil Mohan Adapa; linux-arm-kernel Objet : Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino On Wed, Aug 7, 2019 at 8:01 PM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > > On Tue, Aug 06, 2019 at 02:25:17PM +0800, Chen-Yu Tsai wrote: > > On Mon, Aug 5, 2019 at 8:58 PM Martin Ayotte <martinayotte@gmail.com> wrote: > > > > > > Fine for me too. > > > > > > Thanks . > > > > > > -----Message d'origine----- > > > De : Sunil Mohan Adapa [mailto:sunil@medhas.org] > > > Envoyé : Monday, August 05, 2019 1:25 AM > > > Ą : Chen-Yu Tsai > > > Cc : Maxime Ripard; Martin Ayotte; linux-arm-kernel > > > Objet : Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on > > > A64-OLinuXino > > > > > > On 04/08/19 8:33 pm, Chen-Yu Tsai wrote: > > > > On Fri, Aug 2, 2019 at 2:47 AM Sunil Mohan Adapa <sunil@medhas.org> wrote: > > > >> > > > >> On 01/08/19 6:49 am, Martin Ayotte wrote: > > > >>> If my SOB could help here, I don't mind since I've done the commit > > > >>> more than a year ago for Armbian ... > > > >>> > > > >>> Signed-off-by: Martin Ayotte <martinayotte@gmai.com> > > > >>> Tested-by: Martin Ayotte <martinayotte@gmai.com> > > > >> gmai.com is likely a typo. > > > >> > > > >>> On Wed, Jul 31, 2019 at 10:42 PM Chen-Yu Tsai <wens@csie.org > > > >>> > > > >>>> Thanks. The patch looks good overall. The authorship is a little > > > >>>> confusing though. If it was initially done by Martin (CC-ed), then > > > >>>> he should be the author, and we should get his Signed-off-by if > > > >>>> possible. > > > >> > > > >> Martin is indeed the original author of the patch. Thank you for > > > reviewing. > > > > > > > > I'd like to apply this patch with Martin as the author, if that's OK with > > > you > > > > both? > > > > > > That is completely okay with me. > > > > Applied for 5.4. > > > > I reordered the tags so they make more sense: > > > > https://git.kernel.org/pub/scm/linux/kernel/git/sunxi/linux.git/commit/?h=sunxi/dt-for-5.4&id=0834887732df5af41b59b2e4d530fc1f5478965f > > Sorry for being late on this, but it looks like the eMMC, NAND and SPI > pins are conflicting on the A64-Olinuxino design. > > There's no configuration with a NAND, so we don't really need to worry > about that, but if we merge this in the main DT, we'll prevent anyone > from using that DT on an olinuxino with a SPI flash. > > I think we should just create emmc and SPI-flash variants of that DT. Actually they aren't. Olimex specifically uses eMMC modules that don't use the data strobe line, so SPI can be used together. ChenYu _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino 2019-08-07 12:09 ` Chen-Yu Tsai 2019-08-07 12:41 ` Martin Ayotte @ 2019-08-07 14:45 ` Maxime Ripard 2019-08-08 5:12 ` Chen-Yu Tsai 1 sibling, 1 reply; 19+ messages in thread From: Maxime Ripard @ 2019-08-07 14:45 UTC (permalink / raw) To: Chen-Yu Tsai; +Cc: Sunil Mohan Adapa, Martin Ayotte, linux-arm-kernel Hi, On Wed, Aug 07, 2019 at 08:09:19PM +0800, Chen-Yu Tsai wrote: > On Wed, Aug 7, 2019 at 8:01 PM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > > > > On Tue, Aug 06, 2019 at 02:25:17PM +0800, Chen-Yu Tsai wrote: > > > On Mon, Aug 5, 2019 at 8:58 PM Martin Ayotte <martinayotte@gmail.com> wrote: > > > > > > > > Fine for me too. > > > > > > > > Thanks . > > > > > > > > -----Message d'origine----- > > > > De : Sunil Mohan Adapa [mailto:sunil@medhas.org] > > > > Envoyé : Monday, August 05, 2019 1:25 AM > > > > Ą : Chen-Yu Tsai > > > > Cc : Maxime Ripard; Martin Ayotte; linux-arm-kernel > > > > Objet : Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on > > > > A64-OLinuXino > > > > > > > > On 04/08/19 8:33 pm, Chen-Yu Tsai wrote: > > > > > On Fri, Aug 2, 2019 at 2:47 AM Sunil Mohan Adapa <sunil@medhas.org> wrote: > > > > >> > > > > >> On 01/08/19 6:49 am, Martin Ayotte wrote: > > > > >>> If my SOB could help here, I don't mind since I've done the commit > > > > >>> more than a year ago for Armbian ... > > > > >>> > > > > >>> Signed-off-by: Martin Ayotte <martinayotte@gmai.com> > > > > >>> Tested-by: Martin Ayotte <martinayotte@gmai.com> > > > > >> gmai.com is likely a typo. > > > > >> > > > > >>> On Wed, Jul 31, 2019 at 10:42 PM Chen-Yu Tsai <wens@csie.org > > > > >>> > > > > >>>> Thanks. The patch looks good overall. The authorship is a little > > > > >>>> confusing though. If it was initially done by Martin (CC-ed), then > > > > >>>> he should be the author, and we should get his Signed-off-by if > > > > >>>> possible. > > > > >> > > > > >> Martin is indeed the original author of the patch. Thank you for > > > > reviewing. > > > > > > > > > > I'd like to apply this patch with Martin as the author, if that's OK with > > > > you > > > > > both? > > > > > > > > That is completely okay with me. > > > > > > Applied for 5.4. > > > > > > I reordered the tags so they make more sense: > > > > > > https://git.kernel.org/pub/scm/linux/kernel/git/sunxi/linux.git/commit/?h=sunxi/dt-for-5.4&id=0834887732df5af41b59b2e4d530fc1f5478965f > > > > Sorry for being late on this, but it looks like the eMMC, NAND and SPI > > pins are conflicting on the A64-Olinuxino design. > > > > There's no configuration with a NAND, so we don't really need to worry > > about that, but if we merge this in the main DT, we'll prevent anyone > > from using that DT on an olinuxino with a SPI flash. > > > > I think we should just create emmc and SPI-flash variants of that DT. > > Actually they aren't. Olimex specifically uses eMMC modules that don't > use the data strobe line, so SPI can be used together. Ah, right. Still, this creates a precedent that I'm not really comfortable with. Three actually. Merging this in the main DT means three things: - We're not consistent anymore, including within the olinuxino boards only. A20 Olinuxino is pretty much in the same situation, yet we dealt with it differently. - This means that this will create a spurious device and report errors in the kernel message and whenever someone will try to access the device on boards that don't have it wired. This shouldn't happen and we really shouldn't expose devices that just aren't there, just like you don't have all the devices that are not connected on your USB connector. - Finally, this means that in order to keep it somewhat consistent, we would have to merge the SPI flash in the main DT too. This will prevent people without a SPI flash to use the SPI signals on the UEXT connector for something else, which again goes against the policy we've had for basically any other board. Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino 2019-08-07 14:45 ` Maxime Ripard @ 2019-08-08 5:12 ` Chen-Yu Tsai 2019-08-08 6:25 ` Sunil Mohan Adapa 2019-08-12 7:00 ` Maxime Ripard 0 siblings, 2 replies; 19+ messages in thread From: Chen-Yu Tsai @ 2019-08-08 5:12 UTC (permalink / raw) To: Maxime Ripard; +Cc: Sunil Mohan Adapa, Martin Ayotte, linux-arm-kernel On Wed, Aug 7, 2019 at 10:45 PM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > > Hi, > > On Wed, Aug 07, 2019 at 08:09:19PM +0800, Chen-Yu Tsai wrote: > > On Wed, Aug 7, 2019 at 8:01 PM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > > > > > > On Tue, Aug 06, 2019 at 02:25:17PM +0800, Chen-Yu Tsai wrote: > > > > On Mon, Aug 5, 2019 at 8:58 PM Martin Ayotte <martinayotte@gmail.com> wrote: > > > > > > > > > > Fine for me too. > > > > > > > > > > Thanks . > > > > > > > > > > -----Message d'origine----- > > > > > De : Sunil Mohan Adapa [mailto:sunil@medhas.org] > > > > > Envoyé : Monday, August 05, 2019 1:25 AM > > > > > Ą : Chen-Yu Tsai > > > > > Cc : Maxime Ripard; Martin Ayotte; linux-arm-kernel > > > > > Objet : Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on > > > > > A64-OLinuXino > > > > > > > > > > On 04/08/19 8:33 pm, Chen-Yu Tsai wrote: > > > > > > On Fri, Aug 2, 2019 at 2:47 AM Sunil Mohan Adapa <sunil@medhas.org> wrote: > > > > > >> > > > > > >> On 01/08/19 6:49 am, Martin Ayotte wrote: > > > > > >>> If my SOB could help here, I don't mind since I've done the commit > > > > > >>> more than a year ago for Armbian ... > > > > > >>> > > > > > >>> Signed-off-by: Martin Ayotte <martinayotte@gmai.com> > > > > > >>> Tested-by: Martin Ayotte <martinayotte@gmai.com> > > > > > >> gmai.com is likely a typo. > > > > > >> > > > > > >>> On Wed, Jul 31, 2019 at 10:42 PM Chen-Yu Tsai <wens@csie.org > > > > > >>> > > > > > >>>> Thanks. The patch looks good overall. The authorship is a little > > > > > >>>> confusing though. If it was initially done by Martin (CC-ed), then > > > > > >>>> he should be the author, and we should get his Signed-off-by if > > > > > >>>> possible. > > > > > >> > > > > > >> Martin is indeed the original author of the patch. Thank you for > > > > > reviewing. > > > > > > > > > > > > I'd like to apply this patch with Martin as the author, if that's OK with > > > > > you > > > > > > both? > > > > > > > > > > That is completely okay with me. > > > > > > > > Applied for 5.4. > > > > > > > > I reordered the tags so they make more sense: > > > > > > > > https://git.kernel.org/pub/scm/linux/kernel/git/sunxi/linux.git/commit/?h=sunxi/dt-for-5.4&id=0834887732df5af41b59b2e4d530fc1f5478965f > > > > > > Sorry for being late on this, but it looks like the eMMC, NAND and SPI > > > pins are conflicting on the A64-Olinuxino design. > > > > > > There's no configuration with a NAND, so we don't really need to worry > > > about that, but if we merge this in the main DT, we'll prevent anyone > > > from using that DT on an olinuxino with a SPI flash. > > > > > > I think we should just create emmc and SPI-flash variants of that DT. > > > > Actually they aren't. Olimex specifically uses eMMC modules that don't > > use the data strobe line, so SPI can be used together. > > Ah, right. > > Still, this creates a precedent that I'm not really comfortable > with. Three actually. > > Merging this in the main DT means three things: > - We're not consistent anymore, including within the olinuxino > boards only. A20 Olinuxino is pretty much in the same situation, > yet we dealt with it differently. > - This means that this will create a spurious device and report > errors in the kernel message and whenever someone will try to > access the device on boards that don't have it wired. This > shouldn't happen and we really shouldn't expose devices that just > aren't there, just like you don't have all the devices that are > not connected on your USB connector. > - Finally, this means that in order to keep it somewhat consistent, > we would have to merge the SPI flash in the main DT too. This will > prevent people without a SPI flash to use the SPI signals on the > UEXT connector for something else, which again goes against the > policy we've had for basically any other board. OK. Shall we back it out and figure out something else? ChenYu _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino 2019-08-08 5:12 ` Chen-Yu Tsai @ 2019-08-08 6:25 ` Sunil Mohan Adapa 2019-08-08 6:57 ` Chen-Yu Tsai 2019-08-12 7:00 ` Maxime Ripard 1 sibling, 1 reply; 19+ messages in thread From: Sunil Mohan Adapa @ 2019-08-08 6:25 UTC (permalink / raw) To: Chen-Yu Tsai, Maxime Ripard; +Cc: Martin Ayotte, linux-arm-kernel [-- Attachment #1.1.1: Type: text/plain, Size: 4460 bytes --] On 07/08/19 10:12 pm, Chen-Yu Tsai wrote: > On Wed, Aug 7, 2019 at 10:45 PM Maxime Ripard <maxime.ripard@bootlin.com> wrote: >> >> Hi, >> >> On Wed, Aug 07, 2019 at 08:09:19PM +0800, Chen-Yu Tsai wrote: >>> On Wed, Aug 7, 2019 at 8:01 PM Maxime Ripard <maxime.ripard@bootlin.com> wrote: >>>> >>>> On Tue, Aug 06, 2019 at 02:25:17PM +0800, Chen-Yu Tsai wrote: >>>>> On Mon, Aug 5, 2019 at 8:58 PM Martin Ayotte <martinayotte@gmail.com> wrote: >>>>>> >>>>>> Fine for me too. >>>>>> >>>>>> Thanks . >>>>>> >>>>>> -----Message d'origine----- >>>>>> De : Sunil Mohan Adapa [mailto:sunil@medhas.org] >>>>>> Envoyé : Monday, August 05, 2019 1:25 AM >>>>>> Ą : Chen-Yu Tsai >>>>>> Cc : Maxime Ripard; Martin Ayotte; linux-arm-kernel >>>>>> Objet : Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on >>>>>> A64-OLinuXino >>>>>> >>>>>> On 04/08/19 8:33 pm, Chen-Yu Tsai wrote: >>>>>>> On Fri, Aug 2, 2019 at 2:47 AM Sunil Mohan Adapa <sunil@medhas.org> wrote: >>>>>>>> >>>>>>>> On 01/08/19 6:49 am, Martin Ayotte wrote: >>>>>>>>> If my SOB could help here, I don't mind since I've done the commit >>>>>>>>> more than a year ago for Armbian ... >>>>>>>>> >>>>>>>>> Signed-off-by: Martin Ayotte <martinayotte@gmai.com> >>>>>>>>> Tested-by: Martin Ayotte <martinayotte@gmai.com> >>>>>>>> gmai.com is likely a typo. >>>>>>>> >>>>>>>>> On Wed, Jul 31, 2019 at 10:42 PM Chen-Yu Tsai <wens@csie.org >>>>>>>>> >>>>>>>>>> Thanks. The patch looks good overall. The authorship is a little >>>>>>>>>> confusing though. If it was initially done by Martin (CC-ed), then >>>>>>>>>> he should be the author, and we should get his Signed-off-by if >>>>>>>>>> possible. >>>>>>>> >>>>>>>> Martin is indeed the original author of the patch. Thank you for >>>>>> reviewing. >>>>>>> >>>>>>> I'd like to apply this patch with Martin as the author, if that's OK with >>>>>> you >>>>>>> both? >>>>>> >>>>>> That is completely okay with me. >>>>> >>>>> Applied for 5.4. >>>>> >>>>> I reordered the tags so they make more sense: >>>>> >>>>> https://git.kernel.org/pub/scm/linux/kernel/git/sunxi/linux.git/commit/?h=sunxi/dt-for-5.4&id=0834887732df5af41b59b2e4d530fc1f5478965f >>>> >>>> Sorry for being late on this, but it looks like the eMMC, NAND and SPI >>>> pins are conflicting on the A64-Olinuxino design. >>>> >>>> There's no configuration with a NAND, so we don't really need to worry >>>> about that, but if we merge this in the main DT, we'll prevent anyone >>>> from using that DT on an olinuxino with a SPI flash. >>>> >>>> I think we should just create emmc and SPI-flash variants of that DT. >>> >>> Actually they aren't. Olimex specifically uses eMMC modules that don't >>> use the data strobe line, so SPI can be used together. >> >> Ah, right. >> >> Still, this creates a precedent that I'm not really comfortable >> with. Three actually. >> >> Merging this in the main DT means three things: >> - We're not consistent anymore, including within the olinuxino >> boards only. A20 Olinuxino is pretty much in the same situation, >> yet we dealt with it differently. >> - This means that this will create a spurious device and report >> errors in the kernel message and whenever someone will try to >> access the device on boards that don't have it wired. This >> shouldn't happen and we really shouldn't expose devices that just >> aren't there, just like you don't have all the devices that are >> not connected on your USB connector. >> - Finally, this means that in order to keep it somewhat consistent, >> we would have to merge the SPI flash in the main DT too. This will >> prevent people without a SPI flash to use the SPI signals on the >> UEXT connector for something else, which again goes against the >> policy we've had for basically any other board. > > OK. Shall we back it out and figure out something else? I can try to help with alternate implementation with some guidance. I can also test any patches on an A64-OLinuXino model with eMMC. So, do we create a new -emmc.dts like in case of A20 OLinuXino? BTW, a basic question: how does u-boot know which variant of dtb (with -emmc or without) it has load before booting Linux? Does this need to be hardcoded now into the boot script? Currently, in Debian the DTB name is chosen by u-boot and not present in the boot script. Thanks, -- Sunil [-- Attachment #1.2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 833 bytes --] [-- Attachment #2: Type: text/plain, Size: 176 bytes --] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino 2019-08-08 6:25 ` Sunil Mohan Adapa @ 2019-08-08 6:57 ` Chen-Yu Tsai 0 siblings, 0 replies; 19+ messages in thread From: Chen-Yu Tsai @ 2019-08-08 6:57 UTC (permalink / raw) To: Sunil Mohan Adapa; +Cc: Maxime Ripard, Martin Ayotte, linux-arm-kernel On Thu, Aug 8, 2019 at 2:25 PM Sunil Mohan Adapa <sunil@medhas.org> wrote: > > > > On 07/08/19 10:12 pm, Chen-Yu Tsai wrote: > > On Wed, Aug 7, 2019 at 10:45 PM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > >> > >> Hi, > >> > >> On Wed, Aug 07, 2019 at 08:09:19PM +0800, Chen-Yu Tsai wrote: > >>> On Wed, Aug 7, 2019 at 8:01 PM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > >>>> > >>>> On Tue, Aug 06, 2019 at 02:25:17PM +0800, Chen-Yu Tsai wrote: > >>>>> On Mon, Aug 5, 2019 at 8:58 PM Martin Ayotte <martinayotte@gmail.com> wrote: > >>>>>> > >>>>>> Fine for me too. > >>>>>> > >>>>>> Thanks . > >>>>>> > >>>>>> -----Message d'origine----- > >>>>>> De : Sunil Mohan Adapa [mailto:sunil@medhas.org] > >>>>>> Envoyé : Monday, August 05, 2019 1:25 AM > >>>>>> Ą : Chen-Yu Tsai > >>>>>> Cc : Maxime Ripard; Martin Ayotte; linux-arm-kernel > >>>>>> Objet : Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on > >>>>>> A64-OLinuXino > >>>>>> > >>>>>> On 04/08/19 8:33 pm, Chen-Yu Tsai wrote: > >>>>>>> On Fri, Aug 2, 2019 at 2:47 AM Sunil Mohan Adapa <sunil@medhas.org> wrote: > >>>>>>>> > >>>>>>>> On 01/08/19 6:49 am, Martin Ayotte wrote: > >>>>>>>>> If my SOB could help here, I don't mind since I've done the commit > >>>>>>>>> more than a year ago for Armbian ... > >>>>>>>>> > >>>>>>>>> Signed-off-by: Martin Ayotte <martinayotte@gmai.com> > >>>>>>>>> Tested-by: Martin Ayotte <martinayotte@gmai.com> > >>>>>>>> gmai.com is likely a typo. > >>>>>>>> > >>>>>>>>> On Wed, Jul 31, 2019 at 10:42 PM Chen-Yu Tsai <wens@csie.org > >>>>>>>>> > >>>>>>>>>> Thanks. The patch looks good overall. The authorship is a little > >>>>>>>>>> confusing though. If it was initially done by Martin (CC-ed), then > >>>>>>>>>> he should be the author, and we should get his Signed-off-by if > >>>>>>>>>> possible. > >>>>>>>> > >>>>>>>> Martin is indeed the original author of the patch. Thank you for > >>>>>> reviewing. > >>>>>>> > >>>>>>> I'd like to apply this patch with Martin as the author, if that's OK with > >>>>>> you > >>>>>>> both? > >>>>>> > >>>>>> That is completely okay with me. > >>>>> > >>>>> Applied for 5.4. > >>>>> > >>>>> I reordered the tags so they make more sense: > >>>>> > >>>>> https://git.kernel.org/pub/scm/linux/kernel/git/sunxi/linux.git/commit/?h=sunxi/dt-for-5.4&id=0834887732df5af41b59b2e4d530fc1f5478965f > >>>> > >>>> Sorry for being late on this, but it looks like the eMMC, NAND and SPI > >>>> pins are conflicting on the A64-Olinuxino design. > >>>> > >>>> There's no configuration with a NAND, so we don't really need to worry > >>>> about that, but if we merge this in the main DT, we'll prevent anyone > >>>> from using that DT on an olinuxino with a SPI flash. > >>>> > >>>> I think we should just create emmc and SPI-flash variants of that DT. > >>> > >>> Actually they aren't. Olimex specifically uses eMMC modules that don't > >>> use the data strobe line, so SPI can be used together. > >> > >> Ah, right. > >> > >> Still, this creates a precedent that I'm not really comfortable > >> with. Three actually. > >> > >> Merging this in the main DT means three things: > >> - We're not consistent anymore, including within the olinuxino > >> boards only. A20 Olinuxino is pretty much in the same situation, > >> yet we dealt with it differently. > >> - This means that this will create a spurious device and report > >> errors in the kernel message and whenever someone will try to > >> access the device on boards that don't have it wired. This > >> shouldn't happen and we really shouldn't expose devices that just > >> aren't there, just like you don't have all the devices that are > >> not connected on your USB connector. > >> - Finally, this means that in order to keep it somewhat consistent, > >> we would have to merge the SPI flash in the main DT too. This will > >> prevent people without a SPI flash to use the SPI signals on the > >> UEXT connector for something else, which again goes against the > >> policy we've had for basically any other board. > > > > OK. Shall we back it out and figure out something else? > > I can try to help with alternate implementation with some guidance. I > can also test any patches on an A64-OLinuXino model with eMMC. So, do we > create a new -emmc.dts like in case of A20 OLinuXino? > > BTW, a basic question: how does u-boot know which variant of dtb (with > -emmc or without) it has load before booting Linux? Does this need to be > hardcoded now into the boot script? Currently, in Debian the DTB name is > chosen by u-boot and not present in the boot script. The default name is set using Kconfig. It is present in the per-board defconfig files. ChenYu _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino 2019-08-08 5:12 ` Chen-Yu Tsai 2019-08-08 6:25 ` Sunil Mohan Adapa @ 2019-08-12 7:00 ` Maxime Ripard 1 sibling, 0 replies; 19+ messages in thread From: Maxime Ripard @ 2019-08-12 7:00 UTC (permalink / raw) To: Chen-Yu Tsai; +Cc: Sunil Mohan Adapa, Martin Ayotte, linux-arm-kernel On Thu, Aug 08, 2019 at 01:12:37PM +0800, Chen-Yu Tsai wrote: > On Wed, Aug 7, 2019 at 10:45 PM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > > On Wed, Aug 07, 2019 at 08:09:19PM +0800, Chen-Yu Tsai wrote: > > > On Wed, Aug 7, 2019 at 8:01 PM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > > > > > > > > On Tue, Aug 06, 2019 at 02:25:17PM +0800, Chen-Yu Tsai wrote: > > > > > On Mon, Aug 5, 2019 at 8:58 PM Martin Ayotte <martinayotte@gmail.com> wrote: > > > > > > > > > > > > Fine for me too. > > > > > > > > > > > > Thanks . > > > > > > > > > > > > -----Message d'origine----- > > > > > > De : Sunil Mohan Adapa [mailto:sunil@medhas.org] > > > > > > Envoyé : Monday, August 05, 2019 1:25 AM > > > > > > Ą : Chen-Yu Tsai > > > > > > Cc : Maxime Ripard; Martin Ayotte; linux-arm-kernel > > > > > > Objet : Re: [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on > > > > > > A64-OLinuXino > > > > > > > > > > > > On 04/08/19 8:33 pm, Chen-Yu Tsai wrote: > > > > > > > On Fri, Aug 2, 2019 at 2:47 AM Sunil Mohan Adapa <sunil@medhas.org> wrote: > > > > > > >> > > > > > > >> On 01/08/19 6:49 am, Martin Ayotte wrote: > > > > > > >>> If my SOB could help here, I don't mind since I've done the commit > > > > > > >>> more than a year ago for Armbian ... > > > > > > >>> > > > > > > >>> Signed-off-by: Martin Ayotte <martinayotte@gmai.com> > > > > > > >>> Tested-by: Martin Ayotte <martinayotte@gmai.com> > > > > > > >> gmai.com is likely a typo. > > > > > > >> > > > > > > >>> On Wed, Jul 31, 2019 at 10:42 PM Chen-Yu Tsai <wens@csie.org > > > > > > >>> > > > > > > >>>> Thanks. The patch looks good overall. The authorship is a little > > > > > > >>>> confusing though. If it was initially done by Martin (CC-ed), then > > > > > > >>>> he should be the author, and we should get his Signed-off-by if > > > > > > >>>> possible. > > > > > > >> > > > > > > >> Martin is indeed the original author of the patch. Thank you for > > > > > > reviewing. > > > > > > > > > > > > > > I'd like to apply this patch with Martin as the author, if that's OK with > > > > > > you > > > > > > > both? > > > > > > > > > > > > That is completely okay with me. > > > > > > > > > > Applied for 5.4. > > > > > > > > > > I reordered the tags so they make more sense: > > > > > > > > > > https://git.kernel.org/pub/scm/linux/kernel/git/sunxi/linux.git/commit/?h=sunxi/dt-for-5.4&id=0834887732df5af41b59b2e4d530fc1f5478965f > > > > > > > > Sorry for being late on this, but it looks like the eMMC, NAND and SPI > > > > pins are conflicting on the A64-Olinuxino design. > > > > > > > > There's no configuration with a NAND, so we don't really need to worry > > > > about that, but if we merge this in the main DT, we'll prevent anyone > > > > from using that DT on an olinuxino with a SPI flash. > > > > > > > > I think we should just create emmc and SPI-flash variants of that DT. > > > > > > Actually they aren't. Olimex specifically uses eMMC modules that don't > > > use the data strobe line, so SPI can be used together. > > > > Ah, right. > > > > Still, this creates a precedent that I'm not really comfortable > > with. Three actually. > > > > Merging this in the main DT means three things: > > - We're not consistent anymore, including within the olinuxino > > boards only. A20 Olinuxino is pretty much in the same situation, > > yet we dealt with it differently. > > - This means that this will create a spurious device and report > > errors in the kernel message and whenever someone will try to > > access the device on boards that don't have it wired. This > > shouldn't happen and we really shouldn't expose devices that just > > aren't there, just like you don't have all the devices that are > > not connected on your USB connector. > > - Finally, this means that in order to keep it somewhat consistent, > > we would have to merge the SPI flash in the main DT too. This will > > prevent people without a SPI flash to use the SPI signals on the > > UEXT connector for something else, which again goes against the > > policy we've had for basically any other board. > > OK. Shall we back it out and figure out something else? I guess we can send a new patch on top to move it to a separate file Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 19+ messages in thread
end of thread, other threads:[~2019-08-12 7:01 UTC | newest] Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-07-27 21:03 [PATCH v2] arm64: dts: allwinner: a64: Enable eMMC on A64-OLinuXino Sunil Mohan Adapa 2019-08-01 2:42 ` Chen-Yu Tsai 2019-08-01 14:05 ` Martin Ayotte [not found] ` <CAKQ8BtjLmhDgA8woY5NqaifODLUh_w_K4QYOUuqc4Six5Amerg@mail.gmail.com> 2019-08-01 18:47 ` Sunil Mohan Adapa 2019-08-02 12:45 ` Martin Ayotte 2019-08-05 3:33 ` Chen-Yu Tsai 2019-08-05 5:24 ` Sunil Mohan Adapa 2019-08-05 12:58 ` Martin Ayotte 2019-08-06 6:25 ` Chen-Yu Tsai 2019-08-06 12:49 ` Martin Ayotte 2019-08-07 6:18 ` Chen-Yu Tsai 2019-08-07 12:01 ` Maxime Ripard 2019-08-07 12:09 ` Chen-Yu Tsai 2019-08-07 12:41 ` Martin Ayotte 2019-08-07 14:45 ` Maxime Ripard 2019-08-08 5:12 ` Chen-Yu Tsai 2019-08-08 6:25 ` Sunil Mohan Adapa 2019-08-08 6:57 ` Chen-Yu Tsai 2019-08-12 7:00 ` Maxime Ripard
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).