All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michael Opdenacker <michael.opdenacker@rootcommit.com>
To: Iker Pedrosa <ikerpedrosam@gmail.com>
Cc: michael.opdenacker@rootcommit.com, Yao Zi <me@ziyao.cc>,
	Yixun Lan <dlan@gentoo.org>,
	Dan Carpenter <dan.carpenter@linaro.org>,
	Binbin Zhou <zhoubinbin@loongson.cn>,
	linux-riscv@lists.infradead.org, spacemit@lists.linux.dev
Subject: Re: [PATCH 1/2] riscv: dts: spacemit: Add i2c buses on OrangePi RV2
Date: Wed, 17 Dec 2025 16:27:16 +0000 (UTC)	[thread overview]
Message-ID: <1626d445-fa7b-4527-b1d9-4d141d8b5ee5@rootcommit.com> (raw)
In-Reply-To: <CABdCQ=Orw0ULhPm43rAhS_iqqNYU4nChcPCqh8UdFhJUumVG5w@mail.gmail.com>

Hi Iker

On 12/17/25 16:26, Iker Pedrosa wrote:
> Hi Michael,
>
> Based on the discussion and Javier's feedback, to make the SD card
> work properly, we need to patch the driver to implement the following:
>
> Define a separate compatible string: the eMMC slot has a PHY, while
> SD/SDIO slots do not. We need to differentiate them to apply the
> correct reset logic (skipping PHY init for SD).
> Extend the pinctrl driver: support is needed for switching the SD
> voltage pins from 3.3V to 1.8V.
> Add tuning logic: this is required at least for SDR modes.
>
> If you don't mind, I'll start working on point #1. Feel free to choose
> any other task.
>
> I'll post a patch once I have it working.

Wow, that would be awesome indeed, all the more as my time is quite 
limited. Please do!
I'll have a look at other aspects if I can, like USB and PCI Express.

Note that I have an eMMC module if you have code to test, and I have the 
BananaPI F3 too which also has eMMC.
Happy to work together :)
Cheers
Michael.

-- 
Michael Opdenacker
Root Commit
Yocto Project and OpenEmbedded Training course - Learn by doing:
https://rootcommit.com/training/yocto/


_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

WARNING: multiple messages have this Message-ID (diff)
From: Michael Opdenacker <michael.opdenacker@rootcommit.com>
To: Iker Pedrosa <ikerpedrosam@gmail.com>
Cc: michael.opdenacker@rootcommit.com, Yao Zi <me@ziyao.cc>,
	Yixun Lan <dlan@gentoo.org>,
	Dan Carpenter <dan.carpenter@linaro.org>,
	Binbin Zhou <zhoubinbin@loongson.cn>,
	linux-riscv@lists.infradead.org, spacemit@lists.linux.dev
Subject: Re: [PATCH 1/2] riscv: dts: spacemit: Add i2c buses on OrangePi RV2
Date: Wed, 17 Dec 2025 16:27:16 +0000 (UTC)	[thread overview]
Message-ID: <1626d445-fa7b-4527-b1d9-4d141d8b5ee5@rootcommit.com> (raw)
In-Reply-To: <CABdCQ=Orw0ULhPm43rAhS_iqqNYU4nChcPCqh8UdFhJUumVG5w@mail.gmail.com>

Hi Iker

On 12/17/25 16:26, Iker Pedrosa wrote:
> Hi Michael,
>
> Based on the discussion and Javier's feedback, to make the SD card
> work properly, we need to patch the driver to implement the following:
>
> Define a separate compatible string: the eMMC slot has a PHY, while
> SD/SDIO slots do not. We need to differentiate them to apply the
> correct reset logic (skipping PHY init for SD).
> Extend the pinctrl driver: support is needed for switching the SD
> voltage pins from 3.3V to 1.8V.
> Add tuning logic: this is required at least for SDR modes.
>
> If you don't mind, I'll start working on point #1. Feel free to choose
> any other task.
>
> I'll post a patch once I have it working.

Wow, that would be awesome indeed, all the more as my time is quite 
limited. Please do!
I'll have a look at other aspects if I can, like USB and PCI Express.

Note that I have an eMMC module if you have code to test, and I have the 
BananaPI F3 too which also has eMMC.
Happy to work together :)
Cheers
Michael.

-- 
Michael Opdenacker
Root Commit
Yocto Project and OpenEmbedded Training course - Learn by doing:
https://rootcommit.com/training/yocto/


  reply	other threads:[~2025-12-17 16:27 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-12-15 10:10 [PATCH 0/2] Attempt to enable MMC on SpacemiT K1 boards Michael Opdenacker
2025-12-15 10:10 ` Michael Opdenacker
2025-12-15 10:10 ` [PATCH 1/2] riscv: dts: spacemit: Add i2c buses on OrangePi RV2 Michael Opdenacker
2025-12-15 10:10   ` Michael Opdenacker
2025-12-15 12:58   ` Yao Zi
2025-12-15 12:58     ` Yao Zi
2025-12-15 15:00     ` Michael Opdenacker
2025-12-15 15:00       ` Michael Opdenacker
2025-12-16  5:37       ` Yao Zi
2025-12-16  5:37         ` Yao Zi
2025-12-16 21:05         ` Michael Opdenacker
2025-12-16 21:05           ` Michael Opdenacker
2025-12-17 15:26           ` Iker Pedrosa
2025-12-17 15:26             ` Iker Pedrosa
2025-12-17 16:27             ` Michael Opdenacker [this message]
2025-12-17 16:27               ` Michael Opdenacker
2025-12-17 22:10               ` Johannes Erdfelt
2025-12-17 22:10                 ` Johannes Erdfelt
2025-12-18  5:19                 ` Michael Opdenacker
2025-12-18  5:25                   ` Michael Opdenacker
2025-12-18  5:25                     ` Michael Opdenacker
2025-12-18  9:49                     ` Iker Pedrosa
2025-12-18  9:49                       ` Iker Pedrosa
2025-12-19 11:29                       ` Yixun Lan
2025-12-19 11:29                         ` Yixun Lan
2025-12-20  2:22                         ` Yao Zi
2025-12-20  2:22                           ` Yao Zi
2025-12-17 20:25             ` Javier Martinez Canillas
2025-12-17 20:25               ` Javier Martinez Canillas
2025-12-16  8:58   ` Javier Martinez Canillas
2025-12-16  8:58     ` Javier Martinez Canillas
2025-12-24 14:00   ` Chukun Pan
2025-12-24 14:00     ` Chukun Pan
2025-12-15 10:10 ` [PATCH 2/2] riscv: dts: spacemit: Add MMC support on BPI-F3 and " Michael Opdenacker
2025-12-15 10:10   ` Michael Opdenacker
2025-12-15 12:46   ` Yao Zi
2025-12-15 12:46     ` Yao Zi
2025-12-15 13:58     ` Michael Opdenacker
2025-12-15 13:58       ` Michael Opdenacker
2025-12-24 15:00   ` Chukun Pan
2025-12-24 15:00     ` Chukun Pan
2025-12-29 10:14     ` Javier Martinez Canillas
2025-12-29 10:14       ` Javier Martinez Canillas

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1626d445-fa7b-4527-b1d9-4d141d8b5ee5@rootcommit.com \
    --to=michael.opdenacker@rootcommit.com \
    --cc=dan.carpenter@linaro.org \
    --cc=dlan@gentoo.org \
    --cc=ikerpedrosam@gmail.com \
    --cc=linux-riscv@lists.infradead.org \
    --cc=me@ziyao.cc \
    --cc=spacemit@lists.linux.dev \
    --cc=zhoubinbin@loongson.cn \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.