Received: by 2002:ab2:60d1:0:b0:1f7:5705:b850 with SMTP id i17csp1688226lqm; Fri, 3 May 2024 03:38:33 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCWxRxcNV6sTnTt2jFDcheaql70N9/EIR4qoq7oowdL5tX1ib6KOTozSmFJyCktpxQS+7ANRs8a9DYEbxuLQvDFnxNtA+xDLr4bScsQmnw== X-Google-Smtp-Source: AGHT+IFE3aEmmTwC5Z1x4GdEp9YjWvPa+hoZb1seKe/pWXF6qDTqPkvkIgAO6fJixunF+BOXQpqu X-Received: by 2002:a50:8d58:0:b0:56f:e5dc:e6e8 with SMTP id t24-20020a508d58000000b0056fe5dce6e8mr1319278edt.27.1714732713697; Fri, 03 May 2024 03:38:33 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1714732713; cv=pass; d=google.com; s=arc-20160816; b=uUjYHTc8qWFndggDOh0r0Otxs2WVmiQLFBxm1N+GUfR7wQ1lSmRRe/lHJZ2SgVensE uukyArFExpV7SAff1jmYHlcT/ob5RMS0C5A6SOsBEpkEwnkPfYWnqDJvBxcblTLSQ5hD 8x5BEH+2fO2d564den9VKOgit8LZMxjNBYwRk2OjJ/XgznRrrFkWU9S15LCCcwvLDa8T wPDXcV5ryB62dWqK0ZRNgc1AoydFa4BcYtDhxN9Odq9cUU+P94evDQoo/0rjb8GkIZIU JMCPEibVB8qXnnd3bdEKDWdzKYmpvlY0K2K4s/2YVpUK94mw9A1Ff3bROROZD2d9KZRc fO/w== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:list-unsubscribe:list-subscribe :list-id:precedence:dkim-signature; bh=Ayrnvvh3IgapDx256XNOB94m6G2IphNzT61aOxS4Ki8=; fh=WJrVtCgwQFx3Dsq7b+C7kIe5EK8kHa1kgFF12DcjKEY=; b=e6WY4lqJZVGVq6ITyhEyJ5hWcHgphKtu8FsHU+Rgj+cSks8ev04eMu97f+hAk/hCNq AsqYeICx4gfwWaoLMuo29CKPXIlpPnMy6kf0uvsgQeVMizUoB+6yunCnYcpWveXbZSiV YKR9NzoFwQLIoYModNHFtZje+42g2esRQ/4gnlt69mFJSTJK9inhsEkAHzkQgK6tH4bn TQoxQCcHfQzgfb0Qm36/9Sdgg4KfZOJ32Jes5MCORPG066DRLI9f+7RQf85V++Q0MUWI jsgYJdX2RyUszIW1oklHzaVidejg7IHPdUYxBcmFzPW7LFlQhDu3l3fK9Ke7SK6D2Ghv RhFQ==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=UBMDKcLn; arc=pass (i=1 spf=pass spfdomain=gmail.com dkim=pass dkdomain=gmail.com dmarc=pass fromdomain=gmail.com); spf=pass (google.com: domain of linux-kernel+bounces-167525-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-167525-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [147.75.80.249]) by mx.google.com with ESMTPS id w6-20020a056402070600b00572a07b01a5si1385613edx.650.2024.05.03.03.38.33 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 03 May 2024 03:38:33 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-167525-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=UBMDKcLn; arc=pass (i=1 spf=pass spfdomain=gmail.com dkim=pass dkdomain=gmail.com dmarc=pass fromdomain=gmail.com); spf=pass (google.com: domain of linux-kernel+bounces-167525-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-167525-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by am.mirrors.kernel.org (Postfix) with ESMTPS id 142BF1F233F6 for ; Fri, 3 May 2024 10:38:33 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 5A1051514C2; Fri, 3 May 2024 10:38:26 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="UBMDKcLn" Received: from mail-vs1-f45.google.com (mail-vs1-f45.google.com [209.85.217.45]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9CA061E528; Fri, 3 May 2024 10:38:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.217.45 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1714732703; cv=none; b=psNjdAKkRiQot8WZDmu9GcSutfsuihqAKLI/H8nBg42xO4DONeT0BuJhnC/pweUr4DHOObrwdsCRVfyOa1P3CPpekWyqAK/5HvLYTjNhF/q8facD3LI82cCTxpGOcXAEKRf+koPaYubKJdmJZ6wZ8QtGXM/1qAiTicK0WRnm48M= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1714732703; c=relaxed/simple; bh=1kEB4U4QiaMitsAR9ZWhEPqZMlSf4/JcdSs/iwJyDuo=; h=MIME-Version:References:In-Reply-To:From:Date:Message-ID:Subject: To:Cc:Content-Type; b=my+V4TP2tCqLyPoorhz5nK3x40w8A3GAXgf8rIZDEYVC05bTPApfmIQGAcQ42+1Hor6yq7VTeB8P3cpYqtItgHvtZTUrckAkx3RISBELP5fVEW0FEZtzszB92YOEzbYqnqMvJ9LJnuh9eYA5Cr/Zd13lDAXXNKOZZY4GQrTgy3U= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=UBMDKcLn; arc=none smtp.client-ip=209.85.217.45 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Received: by mail-vs1-f45.google.com with SMTP id ada2fe7eead31-47eeb8efb6fso826894137.1; Fri, 03 May 2024 03:38:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1714732699; x=1715337499; darn=vger.kernel.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=Ayrnvvh3IgapDx256XNOB94m6G2IphNzT61aOxS4Ki8=; b=UBMDKcLnjbvMds1L4nMbbGqQOJ9if4mb1mmGOAA68zgypIC5pjPYImcmZzAx+g4MNP c12AVSknmP8dkXstAc3+/agJ6wJE5qF2gBzw5Dy4XSPjjhQ3MHVQAdtL0dSs5lzemVWc Rehuj/YNdRCnrm4t8MUyiFvBw5FO2/3djdO0asY7W16SVYzCsDVvHXoWGU9Da6rAQgnz /+f5IT5EalZzCE5T/EcOdi5Zmk17VERkC1CKkCDPmxzJYp6tdz6SEUYGlgqBmatQid9w MccsAKLWSihHRxe03I6f145EEW08LyD4Krs69wS055poGXQ4JsLYSljuEbA4dwBckd3F /18w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1714732699; x=1715337499; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=Ayrnvvh3IgapDx256XNOB94m6G2IphNzT61aOxS4Ki8=; b=PrTEGZaQ2JFay16A8j3/LSeduWkCqk4OkCd9L3zec+EV7hGwGOyO+qPMZsN2Y5ExHK FIr5aZuktH3VLLnuy4JmRVXL50K02o9kftS8awiVUfBITITK7rtPFDsR7vVhwEjoUQ9F B4xFW4cMcUTz6rjRdVJ87WfiUOZ0Qb/QfRKGlylYOg1c5SSSYfSOq9UkFDX2zqe9XoKq z0myEMVYGmZ6HoW2M8qNjh9ar1ToDpdEF+dhrHbEa3rmr2TugHSjBT5rAcCXY27C3MDc xG5Uh042PW3jd1iJXlsZkksUE0d/SPfOP1gRrSZYxDLZLf5WyIAzffsrA41HLHNpCX8t cZrA== X-Forwarded-Encrypted: i=1; AJvYcCXYWygG8Fp0I88MaITOIOrccbzr9ma4MoUN6Rq/pq2CYqI7mWcOVMS/SGnb+7pLa/+C11Qu81LD8ghErZm3iMdpGplmr2dzwAv7XU95 X-Gm-Message-State: AOJu0Yy6MFX3soKUnrYz1LQ6OEZ2aFg8jklQwJ2d7wTH2wIEx8S46VlW HLyQSeWIOdQikmnlP+ag76R0A2MBzpSvUGKwB8i7CD7hTBYlWJOsF5YvWVmViaPNv5OtlaT7IKy SmMCxY7k5IcX7flN2yIadqQxF9ur/rVkG X-Received: by 2002:a67:c58e:0:b0:47b:bda4:c30d with SMTP id h14-20020a67c58e000000b0047bbda4c30dmr2416903vsk.3.1714732699172; Fri, 03 May 2024 03:38:19 -0700 (PDT) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 References: <20240422103745.14725-1-victorshihgli@gmail.com> In-Reply-To: <20240422103745.14725-1-victorshihgli@gmail.com> From: Victor Shih Date: Fri, 3 May 2024 18:38:06 +0800 Message-ID: Subject: Re: [PATCH V15 00/22] Add support UHS-II for GL9755 and GL9767 To: ulf.hansson@linaro.org, adrian.hunter@intel.com Cc: linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org, benchuanggli@gmail.com, HL.Liu@genesyslogic.com.tw, Greg.tu@genesyslogic.com.tw, takahiro.akashi@linaro.org, dlunev@chromium.org, Victor Shih Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hi, Ulf and Adrian Excuse me, may I know about your opinion on this series of patches? If you already have a closer look at a paragraph in the series, could you let me know your comments first, let me check it first. I look forward to your reply. Thanks, Victor Shih On Mon, Apr 22, 2024 at 6:37=E2=80=AFPM Victor Shih wrote: > > From: Victor Shih > > Summary > =3D=3D=3D=3D=3D=3D=3D > These patches[1] support UHS-II and fix GL9755 and GL9767 > UHS-II compatibility. > > About UHS-II, roughly deal with the following three parts: > 1) A UHS-II detection and initialization: > - Host setup to support UHS-II (Section 3.13.1 Host Controller Setup > Sequence[2]). > - Detect a UHS-II I/F (Section 3.13.2 Card Interface Detection Sequence > [2]). > - In step(9) of Section 3.13.2 in [2], UHS-II initialization is include > Section 3.13.3 UHS-II Card Initialization and Section 3.13.4 UHS-II > Setting Register Setup Sequence. > > 2) Send Legacy SD command through SD-TRAN > - Encapsulated SD packets are defined in SD-TRAN in order to ensure Legac= y > SD compatibility and preserve Legacy SD infrastructures (Section 7.1.1 > Packet Types and Format Overview[3]). > - Host issue a UHS-II CCMD packet or a UHS-II DCMD (Section 3.13.5 UHS-II > CCMD Packet issuing and Section 3.13.6 UHS-II DCMD Packet issuing[2]). > > 3) UHS-II Interrupt > - Except for UHS-II error interrupts, most interrupts share the original > interrupt registers. > > Patch structure > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > patch#1-#7: for core > patch#8-#20: for sdhci > patch#21: for GL9755 > patch#22: for GL9767 > > Changes in v15 (April. 22, 2024) > * rebased to the linux-kernel-v6.9.0-rc2 in Ulf Hansson next branch. > * according to the comments provided by Adrian Hunter to modify the > patches base on the [V14 00/21] Add support UHS-II for GL9755. > * Patch#07: Move struct uhs2_command uhs2_cmd to struct mmc_request and > modify whatever other changers to make it work. > Refer the SD Host Controller Standard Specification > Section 3.10 to add Error Recovery mechanism to recover > the command error. > * Patch#11: Refer the SD Host Controller Standard Specification > Section 3.10 to add reset command data mechanism. > * Patch#15: Resolve merge conflicts and reduce unnecessary line breaks. > * Patch#16: Adjust the parameters used in the __sdhci_uhs2_finish_command= () > to match changes in the Patch#7. > * Patch#21: Adjust gl9755_vendor_init() to the correct function. > * Patch#22: Add gl9767 to support uhs2 function. > > Reference > =3D=3D=3D=3D=3D=3D=3D=3D=3D > [1] https://gitlab.com/VictorShih/linux-uhs2.git > [2] SD Host Controller Simplified Specification 4.20 > [3] UHS-II Simplified Addendum 1.02 > [4] https://patchwork.kernel.org/project/linux-mmc/cover/20240123062827.8= 525-1-victorshihgli@gmail.com/ > > ----------------- original cover letter from v14 ----------------- > Summary > =3D=3D=3D=3D=3D=3D=3D > These patches[1] support UHS-II and fix GL9755 UHS-II compatibility. > > About UHS-II, roughly deal with the following three parts: > 1) A UHS-II detection and initialization: > - Host setup to support UHS-II (Section 3.13.1 Host Controller Setup > Sequence[2]). > - Detect a UHS-II I/F (Section 3.13.2 Card Interface Detection Sequence > [2]). > - In step(9) of Section 3.13.2 in [2], UHS-II initialization is include > Section 3.13.3 UHS-II Card Initialization and Section 3.13.4 UHS-II > Setting Register Setup Sequence. > > 2) Send Legacy SD command through SD-TRAN > - Encapsulated SD packets are defined in SD-TRAN in order to ensure Legac= y > SD compatibility and preserve Legacy SD infrastructures (Section 7.1.1 > Packet Types and Format Overview[3]). > - Host issue a UHS-II CCMD packet or a UHS-II DCMD (Section 3.13.5 UHS-II > CCMD Packet issuing and Section 3.13.6 UHS-II DCMD Packet issuing[2]). > > 3) UHS-II Interrupt > - Except for UHS-II error interrupts, most interrupts share the original > interrupt registers. > > Patch structure > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > patch#1-#7: for core > patch#8-#20: for sdhci > patch#21: for GL9755 > > Changes in v14 (January. 23, 2024) > * rebased to the linux-kernel-v6.7.0-rc4 in Ulf Hansson next branch. > * according to the comments provided by Adrian Hunter to modify the > patches base on the [V13 00/21] Add support UHS-II for GL9755. > * Patch#03: Move mmc_card_uhs2() to include/linux/mmc/host.h, so that > mmc_card_uhs2() can be available for host drivers. > * Patch#10: Use mmc_card_uhs2() to stead sdhci_uhs2_mode() in the > sdhci_uhs2_dump_regs(). > * Patch#11: Since mmc_card_uhs2() is the same as sdhci_uhs2_mode(), so > drop sdhci_uhs2_mode() and use mmc_card_uhs2() instead of > sdhci_uhs2_mode(). > * Patch#13: Use mmc_card_uhs2() to stead sdhci_uhs2_mode() in the > sdhci_uhs2_set_timeout(). > * Patch#14: Use mmc_card_uhs2() to stead sdhci_uhs2_mode() in the > sdhci_uhs2_set_ios(). > * Patch#16: Use mmc_card_uhs2() to stead sdhci_uhs2_mode() in the > sdhci_uhs2_request(). > * Patch#17: Use mmc_card_uhs2() to stead sdhci_uhs2_mode() in the > sdhci_uhs2_complete_work(), sdhci_uhs2_irq() and > sdhci_uhs2_thread_irq(). > * Patch#18: Use mmc_card_uhs2() to stead sdhci_uhs2_mode() in the > __sdhci_uhs2_remove_host(). > > Reference > =3D=3D=3D=3D=3D=3D=3D=3D=3D > [1] https://gitlab.com/VictorShih/linux-uhs2.git > [2] SD Host Controller Simplified Specification 4.20 > [3] UHS-II Simplified Addendum 1.02 > [4] https://patchwork.kernel.org/project/linux-mmc/cover/20230915094351.1= 1120-1-victorshihgli@gmail.com/ > > ----------------- original cover letter from v13 ----------------- > Summary > =3D=3D=3D=3D=3D=3D=3D > These patches[1] support UHS-II and fix GL9755 UHS-II compatibility. > > About UHS-II, roughly deal with the following three parts: > 1) A UHS-II detection and initialization: > - Host setup to support UHS-II (Section 3.13.1 Host Controller Setup > Sequence[2]). > - Detect a UHS-II I/F (Section 3.13.2 Card Interface Detection Sequence > [2]). > - In step(9) of Section 3.13.2 in [2], UHS-II initialization is include > Section 3.13.3 UHS-II Card Initialization and Section 3.13.4 UHS-II > Setting Register Setup Sequence. > > 2) Send Legacy SD command through SD-TRAN > - Encapsulated SD packets are defined in SD-TRAN in order to ensure Legac= y > SD compatibility and preserve Legacy SD infrastructures (Section 7.1.1 > Packet Types and Format Overview[3]). > - Host issue a UHS-II CCMD packet or a UHS-II DCMD (Section 3.13.5 UHS-II > CCMD Packet issuing and Section 3.13.6 UHS-II DCMD Packet issuing[2]). > > 3) UHS-II Interrupt > - Except for UHS-II error interrupts, most interrupts share the original > interrupt registers. > > Patch structure > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > patch#1-#7: for core > patch#8-#20: for sdhci > patch#21: for GL9755 > > Changes in v13 (November. 17, 2023) > * rebased to the linux-kernel-v6.7.0-rc1 in Ulf Hansson next branch. > * according to the comments provided by Adrian Hunter to modify the > patches base on the [V12 00/23] Add support UHS-II for GL9755. > * according to the comments provided by Ulf Hansson to modify the > patches base on the [V12 00/23] Add support UHS-II for GL9755. > * Add new patch#6 to re-factoring the code. > * Patch#7: Separate __mmc_go_idle() into one patch for re-factorring the = code. > Move mmc_decode_scr declaration to sd.h. > Ues uhs2_sd_tran to stead MMC_UHS2_SD_TRAN. > Drop unnecessary comment. > * Patch#11: Use ios timing to stead MMC_UHS2_SUPPORT for indicate the UHS= 2 mode. > * Patch#12: Drop use vmmc2. > Modify comment message. > * Patch#13: Modify comment message. > * Patch#14: Add judgment condition for power mode in the __sdhci_uhs2_set= _ios(). > Modify comment message. > * Patch#15: Merge Patch#15, Patch#16 and Patch#17 of v12 version into Pat= ch#15 in v13 version. > Use definitions to simplify code. > Modify comment message. > * Patch#16: Re-order function to avoid declaration. > Remove unnecessary function. > * Patch#17: Re-order function to avoid declaration. > Remove unnecessary definitions. > * Patch#18: Re-order function to avoid declaration. > Use vqmmc2 to stead vmmc2. > * Patch#21: Ues uhs2_sd_tran to stead MMC_UHS2_SD_TRAN. > > Reference > =3D=3D=3D=3D=3D=3D=3D=3D=3D > [1] https://gitlab.com/VictorShih/linux-uhs2.git > [2] SD Host Controller Simplified Specification 4.20 > [3] UHS-II Simplified Addendum 1.02 > [4] https://patchwork.kernel.org/project/linux-mmc/cover/20230915094351.1= 1120-1-victorshihgli@gmail.com/ > > ----------------- original cover letter from v12 ----------------- > Summary > =3D=3D=3D=3D=3D=3D=3D > These patches[1] support UHS-II and fix GL9755 UHS-II compatibility. > > About UHS-II, roughly deal with the following three parts: > 1) A UHS-II detection and initialization: > - Host setup to support UHS-II (Section 3.13.1 Host Controller Setup > Sequence[2]). > - Detect a UHS-II I/F (Section 3.13.2 Card Interface Detection Sequence > [2]). > - In step(9) of Section 3.13.2 in [2], UHS-II initialization is include > Section 3.13.3 UHS-II Card Initialization and Section 3.13.4 UHS-II > Setting Register Setup Sequence. > > 2) Send Legacy SD command through SD-TRAN > - Encapsulated SD packets are defined in SD-TRAN in order to ensure Legac= y > SD compatibility and preserve Legacy SD infrastructures (Section 7.1.1 > Packet Types and Format Overview[3]). > - Host issue a UHS-II CCMD packet or a UHS-II DCMD (Section 3.13.5 UHS-II > CCMD Packet issuing and Section 3.13.6 UHS-II DCMD Packet issuing[2]). > > 3) UHS-II Interrupt > - Except for UHS-II error interrupts, most interrupts share the original > interrupt registers. > > Patch structure > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > patch#1-#6: for core > patch#7-#22: for sdhci > patch#23: for GL9755 > > Changes in v12 (September. 15, 2023) > * rebased to the linux-kernel-v6.6.0-rc1 in Ulf Hansson next branch. > * according to the comments provided by Adrian Hunter to modify the > patches base on the [V11 00/23] Add support UHS-II for GL9755. > * according to the comments provided by Ulf Hansson to modify the > patches base on the [V11 00/23] Add support UHS-II for GL9755. > * Patch#5: Remove unused max_current_180_vdd2. > * Patch#6: Use mmc_op_multi() to check DCMD which supports multi > read/write in mmc_uhs2_prepare_cmd(). > > Reference > =3D=3D=3D=3D=3D=3D=3D=3D=3D > [1] https://gitlab.com/VictorShih/linux-uhs2.git > [2] SD Host Controller Simplified Specification 4.20 > [3] UHS-II Simplified Addendum 1.02 > [4] https://patchwork.kernel.org/project/linux-mmc/cover/20230908095330.1= 2075-1-victorshihgli@gmail.com/ > > ----------------- original cover letter from v11 ----------------- > Summary > =3D=3D=3D=3D=3D=3D=3D > These patches[1] support UHS-II and fix GL9755 UHS-II compatibility. > > About UHS-II, roughly deal with the following three parts: > 1) A UHS-II detection and initialization: > - Host setup to support UHS-II (Section 3.13.1 Host Controller Setup > Sequence[2]). > - Detect a UHS-II I/F (Section 3.13.2 Card Interface Detection Sequence > [2]). > - In step(9) of Section 3.13.2 in [2], UHS-II initialization is include > Section 3.13.3 UHS-II Card Initialization and Section 3.13.4 UHS-II > Setting Register Setup Sequence. > > 2) Send Legacy SD command through SD-TRAN > - Encapsulated SD packets are defined in SD-TRAN in order to ensure Legac= y > SD compatibility and preserve Legacy SD infrastructures (Section 7.1.1 > Packet Types and Format Overview[3]). > - Host issue a UHS-II CCMD packet or a UHS-II DCMD (Section 3.13.5 UHS-II > CCMD Packet issuing and Section 3.13.6 UHS-II DCMD Packet issuing[2]). > > 3) UHS-II Interrupt > - Except for UHS-II error interrupts, most interrupts share the original > interrupt registers. > > Patch structure > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > patch#1-#6: for core > patch#7-#22: for sdhci > patch#23: for GL9755 > > Changes in v11 (September. 08, 2023) > * rebased to the linux-kernel-v6.5.0-rc5 in Ulf Hansson next branch. > * according to the comments provided by Adrian Hunter to modify the > patches base on the [V10 00/23] Add support UHS-II for GL9755. > * Patch#18: Drop the check mmc_card_uhs2_hd_mode(host->mmc) > in sdhci_uhs2_set_transfer_mode(). > * Patch#20: Remove unused ocr_avail_uhs2. > > Reference > =3D=3D=3D=3D=3D=3D=3D=3D=3D > [1] https://gitlab.com/VictorShih/linux-uhs2.git > [2] SD Host Controller Simplified Specification 4.20 > [3] UHS-II Simplified Addendum 1.02 > [4] https://patchwork.kernel.org/project/linux-mmc/cover/20230721101349.1= 2387-1-victorshihgli@gmail.com/ > > ----------------- original cover letter from v10 ----------------- > Summary > =3D=3D=3D=3D=3D=3D=3D > These patches[1] support UHS-II and fix GL9755 UHS-II compatibility. > > About UHS-II, roughly deal with the following three parts: > 1) A UHS-II detection and initialization: > - Host setup to support UHS-II (Section 3.13.1 Host Controller Setup > Sequence[2]). > - Detect a UHS-II I/F (Section 3.13.2 Card Interface Detection Sequence > [2]). > - In step(9) of Section 3.13.2 in [2], UHS-II initialization is include > Section 3.13.3 UHS-II Card Initialization and Section 3.13.4 UHS-II > Setting Register Setup Sequence. > > 2) Send Legacy SD command through SD-TRAN > - Encapsulated SD packets are defined in SD-TRAN in order to ensure Legac= y > SD compatibility and preserve Legacy SD infrastructures (Section 7.1.1 > Packet Types and Format Overview[3]). > - Host issue a UHS-II CCMD packet or a UHS-II DCMD (Section 3.13.5 UHS-II > CCMD Packet issuing and Section 3.13.6 UHS-II DCMD Packet issuing[2]). > > 3) UHS-II Interrupt > - Except for UHS-II error interrupts, most interrupts share the original > interrupt registers. > > Patch structure > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > patch#1-#6: for core > patch#7-#22: for sdhci > patch#23: for GL9755 > > Changes in v10 (August. 18, 2023) > * rebased to the linux-kernel-v6.5.0-rc5 in Ulf Hansson next branch. > * according to the comments provided by Ulf Hansson to modify the > patches base on the [V9 00/23] Add support UHS-II for GL9755. > * Patch#2: Drop unnecessary definitions and code. > * Patch#3: Modify the commit message. > * Patch#4: Modify the commit message. > * Patch#5: Drop unnecessary definitions. > * Patch#6: Move some definitions of PatchV9[02/23] to PatchV10[06/23]. > Move some definitions of PatchV9[05/23] to PatchV10[06/23]. > Drop do_multi in the mmc_blk_rw_rq_prep(). > Use tmode_half_duplex to instead of uhs2_tmode0_flag. > Move entire control of the tmode into mmc_uhs2_prepare_cmd(). > * Patch#11: Move some definitions of PatchV9[05/23] to PatchV10[11/23]. > * Patch#18: Use tmode_half_duplex to instead of uhs2_tmode0_flag > in sdhci_uhs2_set_transfer_mode(). > * Patch#20: Move some definitions of PatchV9[05/23] to PatchV10[20/23]. > > Reference > =3D=3D=3D=3D=3D=3D=3D=3D=3D > [1] https://gitlab.com/VictorShih/linux-uhs2.git > [2] SD Host Controller Simplified Specification 4.20 > [3] UHS-II Simplified Addendum 1.02 > [4] https://patchwork.kernel.org/project/linux-mmc/cover/20230721101349.1= 2387-1-victorshihgli@gmail.com/ > > ----------------- original cover letter from v9 ----------------- > Summary > =3D=3D=3D=3D=3D=3D=3D > These patches[1] support UHS-II and fix GL9755 UHS-II compatibility. > > About UHS-II, roughly deal with the following three parts: > 1) A UHS-II detection and initialization: > - Host setup to support UHS-II (Section 3.13.1 Host Controller Setup > Sequence[2]). > - Detect a UHS-II I/F (Section 3.13.2 Card Interface Detection Sequence > [2]). > - In step(9) of Section 3.13.2 in [2], UHS-II initialization is include > Section 3.13.3 UHS-II Card Initialization and Section 3.13.4 UHS-II > Setting Register Setup Sequence. > > 2) Send Legacy SD command through SD-TRAN > - Encapsulated SD packets are defined in SD-TRAN in order to ensure Legac= y > SD compatibility and preserve Legacy SD infrastructures (Section 7.1.1 > Packet Types and Format Overview[3]). > - Host issue a UHS-II CCMD packet or a UHS-II DCMD (Section 3.13.5 UHS-II > CCMD Packet issuing and Section 3.13.6 UHS-II DCMD Packet issuing[2]). > > 3) UHS-II Interrupt > - Except for UHS-II error interrupts, most interrupts share the original > interrupt registers. > > Patch structure > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > patch#1-#6: for core > patch#7-#22: for sdhci > patch#23: for GL9755 > > Changes in v9 (July. 21, 2023) > * rebased to the linux-kernel-v6.5.0-rc1 in Ulf Hansson next branch. > * according to the comments provided by Adrian Hunter to modify the > patches base on the [V8 00/23] Add support UHS-II for GL9755. > * Patch#2: move sd_uhs2_operation definition of PatchV8[05/23] > to PatchV9[02/23] for avoid compilation errors. > move uhs2_control definition of PatchV8[05/23] > to PatchV9[02/23] for avoid compilation errors. > move mmc_host flags definition of PatchV8[05/23] > to PatchV9[02/23] for avoid compilation errors. > move mmc_host flags MMC_UHS2_SUPPORT definition of > PatchV8[05/23] to PatchV9[02/23] for avoid compilation errors. > move mmc_host flags MMC_UHS2_SD_TRAN definition of > PatchV8[05/23] to PatchV9[02/23] for avoid compilation errors. > * Patch#7: Modify the commit message. > * Patch#8: Modify the commit message. > * Patch#11: Modify annotations in sdhci_get_vdd_value(). > * Patch#14: Simplity the turning_on_clk in sdhci_set_ios(). > * Patch#18: Modify the annotations in __sdhci_uhs2_send_command(). > * Patch#19: Cancel export state of sdhci_set_mrq_done() function. > * Patch#23: Rename gl9755_pre_detect_init() to sdhci_gli_pre_detect_init(= ). > Rename gl9755_uhs2_reset_sd_tran() to > sdhci_gli_uhs2_reset_sd_tran(). > > Reference > =3D=3D=3D=3D=3D=3D=3D=3D=3D > [1] https://gitlab.com/VictorShih/linux-uhs2.git > [2] SD Host Controller Simplified Specification 4.20 > [3] UHS-II Simplified Addendum 1.02 > [4] https://patchwork.kernel.org/project/linux-mmc/cover/20230621100151.6= 329-1-victorshihgli@gmail.com/ > > ----------------- original cover letter from v8 ----------------- > Summary > =3D=3D=3D=3D=3D=3D=3D > These patches[1] support UHS-II and fix GL9755 UHS-II compatibility. > > About UHS-II, roughly deal with the following three parts: > 1) A UHS-II detection and initialization: > - Host setup to support UHS-II (Section 3.13.1 Host Controller Setup > Sequence[2]). > - Detect a UHS-II I/F (Section 3.13.2 Card Interface Detection Sequence > [2]). > - In step(9) of Section 3.13.2 in [2], UHS-II initialization is include > Section 3.13.3 UHS-II Card Initialization and Section 3.13.4 UHS-II > Setting Register Setup Sequence. > > 2) Send Legacy SD command through SD-TRAN > - Encapsulated SD packets are defined in SD-TRAN in order to ensure Legac= y > SD compatibility and preserve Legacy SD infrastructures (Section 7.1.1 > Packet Types and Format Overview[3]). > - Host issue a UHS-II CCMD packet or a UHS-II DCMD (Section 3.13.5 UHS-II > CCMD Packet issuing and Section 3.13.6 UHS-II DCMD Packet issuing[2]). > > 3) UHS-II Interrupt > - Except for UHS-II error interrupts, most interrupts share the original > interrupt registers. > > Patch structure > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > patch#1-#6: for core > patch#7-#22: for sdhci > patch#23: for GL9755 > > Changes in v8 (June. 21, 2023) > * rebased to the linux-kernel-v6.4.0-rc6 in Ulf Hansson next branch. > * fix most of checkpatch warnings/errors. > * according to the comments provided by Adrian Hunter to modify the > patches base on the [V7 00/23] Add support UHS-II for GL9755. > * Patch#6: Add MMC_UHS2_SUPPORT to be cleared in sd_uhs2_detect(). > Modify return value in sd_uhs2_attach(). > * Patch#7: Use tabs instead of spaces. > * Patch#8: Modify MODULE_LICENSE from "GPL v2" to "GPL". > * Patch#10: Adjust the position of matching brackets. > * Patch#11: Adjust the position of matching brackets. > Add the initial value of the pwr in sdhci_uhs2_set_power(). > * Patch#13: Initialization be combined with declaration and realigned > in sdhci_calc_timeout_uhs2(). > Forward declare struct mmc_command in sdhci_uhs2.h. > * Patch#14: Add the judgment formula for MMC_TIMING_SPEED_A_HD, > MMC_TIMING_SPEED_B and MMC_TIMING_SPEED_B_HD in > __sdhci_uhs2_set_ios(). > Add the switch case for MMC_TIMING_SPEED_A_HD, > MMC_TIMING_SPEED_B and MMC_TIMING_SPEED_B_HD in > sdhci_get_preset_value(). > mmc_opt_regulator_set_ocr() to instead of > mmc_regulator_set_ocr() in sdhci_uhs2_set_ios(). > * Patch#15: usleep_range() to instead of udelay() in > sdhci_uhs2_interface_detect(). > read_poll_timeout() to instead of read_poll_timeout_atomic() > in sdhci_uhs2_interface_detect(). > Modify return value in sdhci_uhs2_do_detect_init(). > * Patch#16: Remove unnecessary include file. > read_poll_timeout() to instead of read_poll_timeout_atomic() > in sdhci_uhs2_enable_clk(). > Put the comment on the end and put the lines in descending > line length in sdhci_uhs2_enable_clk(). > Modify return value in sdhci_uhs2_enable_clk(). > * Patch#17: Reorder the definitions and lose the parentheses in > sdhci_uhs2_set_config(). > read_poll_timeout() to instead of read_poll_timeout_atomic() > in sdhci_uhs2_check_dormant(). > * Patch#18: Adjust the position of matching brackets in > sdhci_uhs2_send_command_retry(). > Modify CameCase definition in __sdhci_uhs2_finish_command(). > Modify error message in __sdhci_uhs2_finish_command(). > sdhci_uhs2_send_command_retry() to instead of > sdhci_uhs2_send_command() in sdhci_uhs2_request(). > Use sdhci_uhs2_mode() to simplify code in > sdhci_uhs2_request_atomic(). > Add forward declaration for sdhci_send_command(). > * Patch#19: Forward declare struct mmc_request in sdhci_uhs2.h. > Remove forward declaration of sdhci_send_command(). > Use mmc_dev() to simplify code in sdhci_request_done_dma(). > * Patch#20: Change return type to void for __sdhci_uhs2_add_host_v4(). > Remove unused variables in __sdhci_uhs2_add_host_v4(). > * Patch#22: Add config select MMC_SDHCI_UHS2 in Kconfig. > * Patch#23: Use sdhci_get_vdd_value() to simplify code in > gl9755_set_power(). > Use read_poll_timeout_atomic() to simplify code in > sdhci_wait_clock_stable(). > Use read_poll_timeout_atomic() to simplify code in > sdhci_gl9755_reset(). > > Reference > =3D=3D=3D=3D=3D=3D=3D=3D=3D > [1] https://gitlab.com/VictorShih/linux-uhs2.git > [2] SD Host Controller Simplified Specification 4.20 > [3] UHS-II Simplified Addendum 1.02 > [4] https://patchwork.kernel.org/project/linux-mmc/cover/20230331105546.1= 3607-1-victor.shih@genesyslogic.com.tw/ > > ----------------- original cover letter from v7 ----------------- > Summary > =3D=3D=3D=3D=3D=3D=3D > These patches[1] support UHS-II and fix GL9755 UHS-II compatibility. > > About UHS-II, roughly deal with the following three parts: > 1) A UHS-II detection and initialization: > - Host setup to support UHS-II (Section 3.13.1 Host Controller Setup Sequ= ence > [2]). > - Detect a UHS-II I/F (Section 3.13.2 Card Interface Detection Sequence[2= ]). > - In step(9) of Section 3.13.2 in [2], UHS-II initialization is include S= ection > 3.13.3 UHS-II Card Initialization and Section 3.13.4 UHS-II Setting Reg= ister > Setup Sequence. > > 2) Send Legacy SD command through SD-TRAN > - Encapsulated SD packets are defined in SD-TRAN in order to ensure Legac= y SD > compatibility and preserve Legacy SD infrastructures (Section 7.1.1 Pac= ket > Types and Format Overview[3]). > - Host issue a UHS-II CCMD packet or a UHS-II DCMD (Section 3.13.5 UHS-II > CCMD Packet issuing and Section 3.13.6 UHS-II DCMD Packet issuing[2]). > > 3) UHS-II Interrupt > - Except for UHS-II error interrupts, most interrupts share the original > interrupt registers. > > Patch structure > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > patch#1-#6: for core > patch#7-#22: for sdhci > patch#23: for GL9755 > > Changes in v7 (Mar. 31, 2023) > * rebased to the linux-kernel-v6.3.0-rc3 in Ulf Hansson next branch. > * according to the guidance and overall architecture provided > by Ulf Hansson, Ben Chuang and Jason Lai to implement the > UHS-2 Core function based on the patches of the [V4,0/6] > Preparations to support SD UHS-II cards[5]. > * according to the guidance and comments provided by > Adrian Hunter, Ben Chuang and AKASHI Takahiro to implement > the UHS-2 Host function based on the patches of the > [RFC,v3.1,00/27] Add support UHS-II for GL9755[4]. > * implement the necessary function to let the UHS-2 Core/Host > work properly. > * fix most of checkpatch warnings/errors. > * according to the guidance and comments provided by > Adrian Hunter, Ben Chuang to implement the UHS-2 > Host function based on the patches of the > [V5,00/26] Add support UHS-II for GL9755[6]. > * according to the guidance and comments provided by > Ulf Hanssion, Adrian Hunter, Ben Chuang to implement the UHS-2 > Host function based on the patches of the > [V6,00/24] Add support UHS-II for GL9755[7]. > * The uhs2_post_attach_sd() function is no longer needed so drop > the V6 version of the Patch#22. > * Modifies the usage of the flags used by the sdhci host for > MMC_UHS2_INITIALIZED. > * Patch#1: Drop unnecessary bracket. > * Patch#2: Drop sd_uhs2_set_ios function. > Used ->uhs2_control() callback for uhs2_set_ios > in sd_uhs2_power_up(). > Used ->uhs2_control() callback for uhs2_set_ios > in sd_uhs2_power_off(). > Drop MMC_TIMING_SD_UHS2 in favor of MMC_TIMING_UHS2_SPEED_A. > Modify sd_uhs2_legacy_init to avoid the > sd_uhs2_reinit cycle issue. > * Patch#5: Drop unnecessary definitions. > * Patch#6: Drop unnecessary function. > Drop uhs2_state in favor of ios->timing. > * Patch#7: Reorder values and positions of definitions. > * Patch#9: Used sdhci_uhs2_mode function to simplify. > * Patch#11: Drop pwr variable in sdhci_uhs2_set_power function. > * Patch#14: Modify some descriptions. > Drop unnecessary function. > * Patch#15: Drop using uhs2_reset ops and use sdhci_uhs2_reset function > in the sdhci_do_detect_init function. > * Patch#17: Drop unnecessary function. > * Patch#18: Drop unnecessary whitespace changes. > Cancel the export state of some functions. > * Patch#19: Drop unnecessary function. > Used sdhci_uhs2_mode function to simplify. > Modify some descriptions. > Cancel the export state of some functions. > * Patch#20: Drop using __sdhci_uhs2_host function and use > __sdhci_add_host function in sdhci_uhs2_add_host function. > Cancel the export state of some functions. > * Patch#23: Drop using uhs2_post_attach_sd function. > > Reference > =3D=3D=3D=3D=3D=3D=3D=3D=3D > [1] https://gitlab.com/ben.chuang/linux-uhs2-gl9755.git > [2] SD Host Controller Simplified Specification 4.20 > [3] UHS-II Simplified Addendum 1.02 > [4] https://patchwork.kernel.org/project/linux-mmc/cover/20201106022726.1= 9831-1-takahiro.akashi@linaro.org/ > [5] https://patchwork.kernel.org/project/linux-mmc/cover/20220418115833.1= 0738-1-jasonlai.genesyslogic@gmail.com/ > [6] https://patchwork.kernel.org/project/linux-mmc/cover/20221019110647.1= 1076-1-victor.shih@genesyslogic.com.tw/ > [7] https://patchwork.kernel.org/project/linux-mmc/cover/20221213090047.3= 805-1-victor.shih@genesyslogic.com.tw/ > > ----------------- original cover letter from v6 ----------------- > Summary > =3D=3D=3D=3D=3D=3D=3D > These patches[1] support UHS-II and fix GL9755 UHS-II compatibility. > > About UHS-II, roughly deal with the following three parts: > 1) A UHS-II detection and initialization: > - Host setup to support UHS-II (Section 3.13.1 Host Controller Setup Sequ= ence > [2]). > - Detect a UHS-II I/F (Section 3.13.2 Card Interface Detection Sequence[2= ]). > - In step(9) of Section 3.13.2 in [2], UHS-II initialization is include S= ection > 3.13.3 UHS-II Card Initialization and Section 3.13.4 UHS-II Setting Reg= ister > Setup Sequence. > > 2) Send Legacy SD command through SD-TRAN > - Encapsulated SD packets are defined in SD-TRAN in order to ensure Legac= y SD > compatibility and preserve Legacy SD infrastructures (Section 7.1.1 Pac= ket > Types and Format Overview[3]). > - Host issue a UHS-II CCMD packet or a UHS-II DCMD (Section 3.13.5 UHS-II > CCMD Packet issuing and Section 3.13.6 UHS-II DCMD Packet issuing[2]). > > 3) UHS-II Interrupt > - Except for UHS-II error interrupts, most interrupts share the original > interrupt registers. > > Patch structure > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > patch#1-#6: for core > patch#7-#23: for sdhci > patch#24: for GL9755 > > Changes in v6 (Dec. 12, 2022) > * rebased to the linux-kernel-v6.1.0-rc8 in Ulf Hansson next branch. > * according to the guidance and overall architecture provided > by Ulf Hansson, Ben Chuang and Jason Lai to implement the > UHS-2 Core function based on the patches of the [V4,0/6] > Preparations to support SD UHS-II cards[5]. > * according to the guidance and comments provided by > Adrian Hunter, Ben Chuang and AKASHI Takahiro to implement > the UHS-2 Host function based on the patches of the > [RFC,v3.1,00/27] Add support UHS-II for GL9755[4]. > * implement the necessary function to let the UHS-2 Core/Host > work properly. > * fix most of checkpatch warnings/errors. > * according to the guidance and comments provided by > Adrian Hunter, Ben Chuang to implement the UHS-2 > Host function based on the patches of the > [V5,00/26] Add support UHS-II for GL9755[6]. > * The uhs2_post_attach_sd() has implemented in Patch#6 and > Patch#17 so drop the V5 version of the Patch#23. > * Modifies the usage of the flags used by the sdhci host for > MMC_UHS2_INITIALIZED. > * Patch#5: Drop unused definitions and functions. > * Patch#7: Rename definitions. > Use BIT() GENMASK() in some cases. > * Patch#8: Merge V5 version of Patch[7] and Patch[9] into > V6 version of Patch[8]. > * Patch#9: Drop unnecessary function. > Rename used definitions. > * Patch#10: Drop unnecessary function and simplify some code. > * Patch#11: Drop unnecessary function. > Add new mmc_opt_regulator_set_ocr function. > * Patch#13: Drop unnecessary function. > Use GENMASK() and FIELD_PREP() in some cases. > * Patch#14: Drop unnecessary function. > Modify return value in some function. > Use GENMASK() and FIELD_PREP() in some cases. > * Patch#15: Drop unnecessary function. > Rename used definitions. > Use GENMASK() and FIELD_GET() in some cases. > Wrap at 100 columns in some functions. > * Patch#16: Drop unnecessary function. > * Patch#17: Drop unnecessary function. > Drop the unnecessary parameter when call the DBG() > function. > Rename used definitions. > Cancel the export state of some functions. > Use GENMASK() and FIELD_PREP() in some cases. > * Patch#18: Drop unnecessary function. > Add uhs2_dev_cmd function to simplify some functions. > Rename used definitions. > Cancel the export state of some functions. > Use GENMASK() and FIELD_PREP() in some cases. > * Patch#19: Drop unnecessary function. > Add sdhci_uhs2_mode() in some functions. > Rename used definitions. > Cancel the export state of some functions. > * Patch#20: Add new complete_work_fn/thread_irq_fn variables in > struct sdhci_host. > Use complete_work_fn/thread_irq_fn variables in > sdhci_alloc_host()/sdhci_uhs2_add_host(). > Rename used definitions. > * Patch[24]: Rename used definitions. > > Reference > =3D=3D=3D=3D=3D=3D=3D=3D=3D > [1] https://gitlab.com/ben.chuang/linux-uhs2-gl9755.git > [2] SD Host Controller Simplified Specification 4.20 > [3] UHS-II Simplified Addendum 1.02 > [4] https://patchwork.kernel.org/project/linux-mmc/cover/20201106022726.1= 9831-1-takahiro.akashi@linaro.org/ > [5] https://patchwork.kernel.org/project/linux-mmc/cover/20220418115833.1= 0738-1-jasonlai.genesyslogic@gmail.com/ > [6] https://patchwork.kernel.org/project/linux-mmc/cover/20221019110647.1= 1076-1-victor.shih@genesyslogic.com.tw/ > > ----------------- original cover letter from v5 ----------------- > Summary > =3D=3D=3D=3D=3D=3D=3D > These patches[1] support UHS-II and fix GL9755 UHS-II compatibility. > > About UHS-II, roughly deal with the following three parts: > 1) A UHS-II detection and initialization: > - Host setup to support UHS-II (Section 3.13.1 Host Controller Setup Sequ= ence > [2]). > - Detect a UHS-II I/F (Section 3.13.2 Card Interface Detection Sequence[2= ]). > - In step(9) of Section 3.13.2 in [2], UHS-II initialization is include S= ection > 3.13.3 UHS-II Card Initialization and Section 3.13.4 UHS-II Setting Reg= ister > Setup Sequence. > > 2) Send Legacy SD command through SD-TRAN > - Encapsulated SD packets are defined in SD-TRAN in order to ensure Legac= y SD > compatibility and preserve Legacy SD infrastructures (Section 7.1.1 Pac= ket > Types and Format Overview[3]). > - Host issue a UHS-II CCMD packet or a UHS-II DCMD (Section 3.13.5 UHS-II > CCMD Packet issuing and Section 3.13.6 UHS-II DCMD Packet issuing[2]). > > 3) UHS-II Interrupt > - Except for UHS-II error interrupts, most interrupts share the original > interrupt registers. > > Patch structure > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > patch#1-#6: for core > patch#7-#25: for sdhci > patch#26: for GL9755 > > Changes in v5 (Oct. 19, 2022) > * rebased to the linux-kernel-v6.1-rc1 in Ulf Hansson next branch. > * according to the guidance and overall architecture provided > by Ulf Hansson, Ben Chuang and Jason Lai to implement the > UHS-2 Core function based on the patches of the [V4,0/6] > Preparations to support SD UHS-II cards[5]. > * according to the guidance and comments provided by > Adrian Hunter, Ben Chuang and AKASHI Takahiro to implement > the UHS-2 Host function based on the patches of the > [RFC,v3.1,00/27] Add support UHS-II for GL9755[4]. > * implement the necessary function to let the UHS-2 Core/Host > work properly. > * fix most of checkpatch warnings/errors > > Reference > =3D=3D=3D=3D=3D=3D=3D=3D=3D > [1] https://gitlab.com/ben.chuang/linux-uhs2-gl9755.git > [2] SD Host Controller Simplified Specification 4.20 > [3] UHS-II Simplified Addendum 1.02 > [4] https://patchwork.kernel.org/project/linux-mmc/cover/20201106022726.1= 9831-1-takahiro.akashi@linaro.org/ > [5] https://patchwork.kernel.org/project/linux-mmc/cover/20220418115833.1= 0738-1-jasonlai.genesyslogic@gmail.com/ > > ----------------- original cover letter from v3.1 ----------------- > This is an interim snapshot of our next version, v4, for enabling > UHS-II on MMC/SD. > > It is focused on 'sdhci' side to address Adrian's comments regarding > "modularising" sdhci-uhs2.c. > The whole aim of this version is to get early feedback from Adrian (and > others) on this issue. Without any consensus about the code structure, > it would make little sense to go further ahead on sdhci side. > (Actually, Adrian has made no comments other than "modularising" so far.) > > I heavily reworked/refactored sdhci-uhs2.c and re-organised the patch > set to meet what I believe Adrian expects; no UHS-II related code in > Legacy (UHS-I) code or sdhci.c. > > Nevertheless, almost of all changes I made are trivial and straightforwar= d > in this direction, and I believe that there is no logic changed since v3 > except sdhci_uhs2_irq(), as ops->irq hook, where we must deal with UHS-II > command sequences in addition to UHS-II errors. So I added extra handling= s. > > I admit that there is plenty of room for improvements (for example, > handling host->flags), but again the focal point here is how sdhci-uhs2.c > should be built as a module. > > Please review this series (particularly Patch#8-#26 and #27) from this > viewpoint in the first place. > (Ben is working on 'host' side but there is no change on 'host' side > in this submission except a minor tweak.) > > Thanks, > -Takahiro Akashi > > ------ original cover letter from v3 ------ > Summary > =3D=3D=3D=3D=3D=3D=3D > These patches[1] support UHS-II and fix GL9755 UHS-II compatibility. > > About UHS-II, roughly deal with the following three parts: > 1) A UHS-II detection and initialization: > - Host setup to support UHS-II (Section 3.13.1 Host Controller Setup Sequ= ence > [2]). > - Detect a UHS-II I/F (Section 3.13.2 Card Interface Detection Sequence[2= ]). > - In step(9) of Section 3.13.2 in [2], UHS-II initialization is include S= ection > 3.13.3 UHS-II Card Initialization and Section 3.13.4 UHS-II Setting Reg= ister > Setup Sequence. > > 2) Send Legacy SD command through SD-TRAN > - Encapsulated SD packets are defined in SD-TRAN in order to ensure Legac= y SD > compatibility and preserve Legacy SD infrastructures (Section 7.1.1 Pac= ket > Types and Format Overview[3]). > - Host issue a UHS-II CCMD packet or a UHS-II DCMD (Section 3.13.5 UHS-II > CCMD Packet issuing and Section 3.13.6 UHS-II DCMD Packet issuing[2]). > > 3) UHS-II Interrupt > - Except for UHS-II error interrupts, most interrupts share the original > interrupt registers. > > Patch structure > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > patch#1-#7: for core > patch#8-#17: for sdhci > patch#18-#21: for GL9755 > > Tests > =3D=3D=3D=3D=3D > Ran 'dd' command to evaluate the performance: > (SanDisk UHS-II card on GL9755 controller) > Read Write > UHS-II disabled (UHS-I): 88.3MB/s 60.7MB/s > UHS-II enabled : 206MB/s 80MB/s > > TODO > =3D=3D=3D=3D > - replace some define with BIT macro > > Reference > =3D=3D=3D=3D=3D=3D=3D=3D=3D > [1] https://gitlab.com/ben.chuang/linux-uhs2-gl9755.git > [2] SD Host Controller Simplified Specification 4.20 > [3] UHS-II Simplified Addendum 1.02 > > Changes in v3 (Jul. 10, 2020) > * rebased to v5.8-rc4 > * add copyright notice > * reorganize the patch set and split some commits into smaller ones > * separate uhs-2 headers from others > * correct wrong spellings > * fix most of checkpatch warnings/errors > * remove all k[cz]alloc() from the code > * guard sdhci-uhs2 specific code with > 'if (IS_ENABLED(CONFIG_MMC_SDHCI_UHS2))' > * make sdhci-uhs2.c as a module > * trivial changes, including > - rename back sdhci-core.c to sdhci.c > - allow vendor code to disable uhs2 if v4_mode =3D=3D 0 > in __sdhci_add_host() > - merge uhs2_power_up() into mmc_power_up() > - remove flag_uhs2 from mmc_attach_sd() > - add function descriptions to EXPORT'ed functions > - other minor code optimization > > Changes in v2 (Jan. 9, 2020) > * rebased to v5.5-rc5 > > Ben Chuang (1): > mmc: sdhci-uhs2: add pre-detect_init hook > > Ulf Hansson (4): > mmc: core: Cleanup printing of speed mode at card insertion > mmc: core: Prepare to support SD UHS-II cards > mmc: core: Announce successful insertion of an SD UHS-II card > mmc: core: Extend support for mmc regulators with a vqmmc2 > > Victor Shih (17): > mmc: core: Add definitions for SD UHS-II cards > mmc: core: Add New function to re-factoring the code > mmc: core: Support UHS-II card control and access > mmc: sdhci: add UHS-II related definitions in headers > mmc: sdhci: add UHS-II module and add a kernel configuration > mmc: sdhci-uhs2: dump UHS-II registers > mmc: sdhci-uhs2: add reset function function > mmc: sdhci-uhs2: add set_power() to support vdd2 > mmc: sdhci-uhs2: add set_timeout() > mmc: sdhci-uhs2: add set_ios() > mmc: sdhci-uhs2: add uhs2_control() to initialise the interface > mmc: sdhci-uhs2: add request() and others > mmc: sdhci-uhs2: add irq() and others > mmc: sdhci-uhs2: add add_host() and others to set up the driver > mmc: sdhci-pci: add UHS-II support framework > mmc: sdhci-pci-gli: enable UHS-II mode for GL9755 > mmc: sdhci-pci-gli: enable UHS-II mode for GL9767 > > drivers/mmc/core/Makefile | 2 +- > drivers/mmc/core/bus.c | 38 +- > drivers/mmc/core/core.c | 27 +- > drivers/mmc/core/core.h | 2 + > drivers/mmc/core/mmc_ops.c | 24 +- > drivers/mmc/core/mmc_ops.h | 1 + > drivers/mmc/core/regulator.c | 34 + > drivers/mmc/core/sd.c | 10 +- > drivers/mmc/core/sd.h | 5 + > drivers/mmc/core/sd_ops.c | 9 + > drivers/mmc/core/sd_ops.h | 13 + > drivers/mmc/core/sd_uhs2.c | 1403 +++++++++++++++++++++++++++++ > drivers/mmc/host/Kconfig | 10 + > drivers/mmc/host/Makefile | 1 + > drivers/mmc/host/sdhci-pci-core.c | 16 +- > drivers/mmc/host/sdhci-pci-gli.c | 385 +++++++- > drivers/mmc/host/sdhci-pci.h | 3 + > drivers/mmc/host/sdhci-uhs2.c | 1269 ++++++++++++++++++++++++++ > drivers/mmc/host/sdhci-uhs2.h | 190 ++++ > drivers/mmc/host/sdhci.c | 277 +++--- > drivers/mmc/host/sdhci.h | 75 +- > include/linux/mmc/card.h | 36 + > include/linux/mmc/core.h | 17 + > include/linux/mmc/host.h | 86 ++ > include/linux/mmc/sd_uhs2.h | 240 +++++ > 25 files changed, 4024 insertions(+), 149 deletions(-) > create mode 100644 drivers/mmc/core/sd_uhs2.c > create mode 100644 drivers/mmc/host/sdhci-uhs2.c > create mode 100644 drivers/mmc/host/sdhci-uhs2.h > create mode 100644 include/linux/mmc/sd_uhs2.h > > -- > 2.25.1 >