mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Ahmad Fatoum <a.fatoum@pengutronix.de>
To: Steffen Trumtrar <s.trumtrar@pengutronix.de>,
	barebox@lists.infradead.org
Subject: Re: [PATCH 04/12] mci: arasan: read clk phases from DT
Date: Mon, 11 Mar 2024 09:44:51 +0100	[thread overview]
Message-ID: <0014f54a-7359-44a1-85ea-be451ef7a733@pengutronix.de> (raw)
In-Reply-To: <20240308-v2024-02-0-topic-arasan-hs200-support-v1-4-6d50c90485f3@pengutronix.de>

On 08.03.24 12:17, Steffen Trumtrar wrote:
> Depending on the used SDHCI mode the clock phases are different.
> Import the helper function to get these values from the DT from linux v6.7.
> 
> Signed-off-by: Steffen Trumtrar <s.trumtrar@pengutronix.de>
> ---
>  drivers/mci/arasan-sdhci.c | 121 +++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 121 insertions(+)
> 
> diff --git a/drivers/mci/arasan-sdhci.c b/drivers/mci/arasan-sdhci.c
> index cc1bad8b62..76022708cd 100644
> --- a/drivers/mci/arasan-sdhci.c
> +++ b/drivers/mci/arasan-sdhci.c
> @@ -30,10 +30,33 @@
>  #define SDHCI_ARASAN_BUS_WIDTH			4
>  #define TIMEOUT_VAL				0xE
>  
> +#define ZYNQMP_CLK_PHASES			10
> +#define ZYNQMP_CLK_PHASE_UHS_SDR104		6
> +#define ZYNQMP_CLK_PHASE_HS200			9
> +/* Default settings for ZynqMP Clock Phases */
> +#define ZYNQMP_ICLK_PHASE {0, 63, 63, 0, 63,  0,   0, 183, 54,  0, 0}
> +#define ZYNQMP_OCLK_PHASE {0, 72, 60, 0, 60, 72, 135, 48, 72, 135, 0}

"include: mci: sync mci_timing with linux" should be ordered before this
patch, because the above two lines depend on it for correctness.

> +
> +/**
> + * struct sdhci_arasan_clk_data - Arasan Controller Clock Data.
> + *
> + * @clk_phase_in:	Array of Input Clock Phase Delays for all speed modes
> + * @clk_phase_out:	Array of Output Clock Phase Delays for all speed modes
> + * @set_clk_delays:	Function pointer for setting Clock Delays
> + * @clk_of_data:	Platform specific runtime clock data storage pointer
> + */
> +struct sdhci_arasan_clk_data {
> +	int		clk_phase_in[ZYNQMP_CLK_PHASES + 1];
> +	int		clk_phase_out[ZYNQMP_CLK_PHASES + 1];
> +	void		(*set_clk_delays)(struct sdhci *host);
> +	void		*clk_of_data;
> +};
> +
>  struct arasan_sdhci_host {
>  	struct mci_host		mci;
>  	struct sdhci		sdhci;
>  	unsigned int		quirks; /* Arasan deviations from spec */
> +	struct sdhci_arasan_clk_data clk_data;
>  /* Controller does not have CD wired and will not function normally without */
>  #define SDHCI_ARASAN_QUIRK_FORCE_CDTEST		BIT(0)
>  #define SDHCI_ARASAN_QUIRK_NO_1_8_V		BIT(1)
> @@ -119,6 +142,7 @@ static int arasan_sdhci_init(struct mci_host *mci, struct device *dev)
>  static void arasan_sdhci_set_clock(struct mci_host *mci, unsigned int clock)
>  {
>  	struct arasan_sdhci_host *host = to_arasan_sdhci_host(mci);
> +	struct sdhci_arasan_clk_data *clk_data = &host->clk_data;
>  
>  	if (host->quirks & SDHCI_ARASAN_QUIRK_CLOCK_25_BROKEN) {
>  		/*
> @@ -219,6 +243,101 @@ static int arasan_sdhci_send_cmd(struct mci_host *mci, struct mci_cmd *cmd,
>  	return ret;
>  }
>  
> +static void sdhci_arasan_set_clk_delays(struct sdhci *host)
> +{
> +	struct arasan_sdhci_host *arasan_sdhci = sdhci_to_arasan(host);
> +	struct mci_host *mci = &arasan_sdhci->mci;
> +	struct sdhci_arasan_clk_data *clk_data = &arasan_sdhci->clk_data;
> +
> +}
> +
> +static void arasan_dt_read_clk_phase(struct device *dev,
> +				     struct sdhci_arasan_clk_data *clk_data,
> +				     unsigned int timing, const char *prop)
> +{
> +	struct device_node *np = dev->of_node;
> +
> +	u32 clk_phase[2] = {0};
> +	int ret;
> +
> +	/*
> +	 * Read Tap Delay values from DT, if the DT does not contain the
> +	 * Tap Values then use the pre-defined values.
> +	 */
> +	ret = of_property_read_u32_array(np, prop, &clk_phase[0], 2);
> +	if (ret < 0) {
> +		dev_dbg(dev, "Using predefined clock phase for %s = %d %d\n",
> +			prop, clk_data->clk_phase_in[timing],
> +			clk_data->clk_phase_out[timing]);
> +		return;
> +	}
> +
> +	/* The values read are Input and Output Clock Delays in order */
> +	clk_data->clk_phase_in[timing] = clk_phase[0];
> +	clk_data->clk_phase_out[timing] = clk_phase[1];
> +}
> +
> +/**
> + * arasan_dt_parse_clk_phases - Read Clock Delay values from DT
> + *
> + * @dev:		Pointer to our struct device.
> + * @clk_data:		Pointer to the Clock Data structure
> + *
> + * Called at initialization to parse the values of Clock Delays.
> + */
> +static void arasan_dt_parse_clk_phases(struct device *dev,
> +				       struct sdhci_arasan_clk_data *clk_data)
> +{
> +	u32 mio_bank = 0;
> +	int i;
> +
> +	/*
> +	 * This has been kept as a pointer and is assigned a function here.
> +	 * So that different controller variants can assign their own handling
> +	 * function.
> +	 */
> +	clk_data->set_clk_delays = sdhci_arasan_set_clk_delays;
> +
> +	if (of_device_is_compatible(dev->of_node, "xlnx,zynqmp-8.9a")) {
> +		u32 zynqmp_iclk_phase[ZYNQMP_CLK_PHASES + 1] = ZYNQMP_ICLK_PHASE;
> +		u32 zynqmp_oclk_phase[ZYNQMP_CLK_PHASES + 1] = ZYNQMP_OCLK_PHASE;
> +
> +		of_property_read_u32(dev->of_node, "xlnx,mio-bank", &mio_bank);
> +		if (mio_bank == 2) {
> +			zynqmp_oclk_phase[ZYNQMP_CLK_PHASE_UHS_SDR104] = 90;
> +			zynqmp_oclk_phase[ZYNQMP_CLK_PHASE_HS200] = 90;
> +		}
> +
> +		for (i = 0; i <= ZYNQMP_CLK_PHASES; i++) {
> +			clk_data->clk_phase_in[i] = zynqmp_iclk_phase[i];
> +			clk_data->clk_phase_out[i] = zynqmp_oclk_phase[i];
> +		}
> +	}
> +
> +	arasan_dt_read_clk_phase(dev, clk_data, MMC_TIMING_LEGACY,
> +				 "clk-phase-legacy");
> +	arasan_dt_read_clk_phase(dev, clk_data, MMC_TIMING_MMC_HS,
> +				 "clk-phase-mmc-hs");
> +	arasan_dt_read_clk_phase(dev, clk_data, MMC_TIMING_SD_HS,
> +				 "clk-phase-sd-hs");
> +	arasan_dt_read_clk_phase(dev, clk_data, MMC_TIMING_UHS_SDR12,
> +				 "clk-phase-uhs-sdr12");
> +	arasan_dt_read_clk_phase(dev, clk_data, MMC_TIMING_UHS_SDR25,
> +				 "clk-phase-uhs-sdr25");
> +	arasan_dt_read_clk_phase(dev, clk_data, MMC_TIMING_UHS_SDR50,
> +				 "clk-phase-uhs-sdr50");
> +	arasan_dt_read_clk_phase(dev, clk_data, MMC_TIMING_UHS_SDR104,
> +				 "clk-phase-uhs-sdr104");
> +	arasan_dt_read_clk_phase(dev, clk_data, MMC_TIMING_UHS_DDR50,
> +				 "clk-phase-uhs-ddr50");
> +	arasan_dt_read_clk_phase(dev, clk_data, MMC_TIMING_MMC_DDR52,
> +				 "clk-phase-mmc-ddr52");
> +	arasan_dt_read_clk_phase(dev, clk_data, MMC_TIMING_MMC_HS200,
> +				 "clk-phase-mmc-hs200");
> +	arasan_dt_read_clk_phase(dev, clk_data, MMC_TIMING_MMC_HS400,
> +				 "clk-phase-mmc-hs400");
> +}
> +
>  static int arasan_sdhci_probe(struct device *dev)
>  {
>  	struct device_node *np = dev->of_node;
> @@ -281,6 +400,8 @@ static int arasan_sdhci_probe(struct device *dev)
>  	mci->f_max = clk_get_rate(clk_xin);
>  	mci->f_min = 50000000 / 256;
>  
> +	arasan_dt_parse_clk_phases(dev, &arasan_sdhci->clk_data);
> +
>  	/* parse board supported bus width capabilities */
>  	mci_of_parse(&arasan_sdhci->mci);
>  
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |




  parent reply	other threads:[~2024-03-11  8:45 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-08 11:16 [PATCH 00/12] mci: add HS200 support for eMMCs Steffen Trumtrar
2024-03-08 11:16 ` [PATCH 01/12] ARM: zynqmp: add sd_dll_reset call Steffen Trumtrar
2024-03-11  8:43   ` Ahmad Fatoum
2024-03-08 11:17 ` [PATCH 02/12] zynqmp: firmware: add functions to set tap delay Steffen Trumtrar
2024-03-11  8:43   ` Ahmad Fatoum
2024-03-08 11:17 ` [PATCH 03/12] mci: arasan: implement 25MHz quirk for zynqmp Steffen Trumtrar
2024-03-12  8:34   ` Ahmad Fatoum
2024-03-08 11:17 ` [PATCH 04/12] mci: arasan: read clk phases from DT Steffen Trumtrar
2024-03-11  8:16   ` Ahmad Fatoum
2024-03-11  8:44   ` Ahmad Fatoum [this message]
2024-03-08 11:17 ` [PATCH 05/12] mci: arasan: register sdcard/sampleclk Steffen Trumtrar
2024-03-11  8:14   ` Ahmad Fatoum
2024-03-08 11:17 ` [PATCH 06/12] include: mci: sync mci_timing with linux Steffen Trumtrar
2024-03-11  8:11   ` Ahmad Fatoum
2024-03-08 11:17 ` [PATCH 07/12] include: mci: add more EXT_CSD_CARD_TYPE_* Steffen Trumtrar
2024-03-11  8:17   ` Ahmad Fatoum
2024-03-08 11:17 ` [PATCH 08/12] mci: core: parse more host capabilities from DT Steffen Trumtrar
2024-03-11  8:23   ` Ahmad Fatoum
2024-03-08 11:17 ` [PATCH 09/12] mci: mci-core: add HS200 support Steffen Trumtrar
2024-03-12  8:20   ` Ahmad Fatoum
2024-03-08 11:17 ` [PATCH 10/12] mci: sdhci: add tuning support Steffen Trumtrar
2024-03-12  8:32   ` Ahmad Fatoum
2024-03-08 11:17 ` [PATCH 11/12] mci: arasan-sdhci: add HS200 tuning support on ZynqMP Steffen Trumtrar
2024-03-12  8:33   ` Ahmad Fatoum
2024-03-08 11:17 ` [PATCH 12/12] mci: arasan: use sdhci_wait_idle2 Steffen Trumtrar
2024-03-11  8:42   ` Ahmad Fatoum

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=0014f54a-7359-44a1-85ea-be451ef7a733@pengutronix.de \
    --to=a.fatoum@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    --cc=s.trumtrar@pengutronix.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox