From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753246AbdJPOgA (ORCPT ); Mon, 16 Oct 2017 10:36:00 -0400 Received: from mail-eopbgr20083.outbound.protection.outlook.com ([40.107.2.83]:28576 "EHLO EUR02-VE1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751802AbdJPOfy (ORCPT ); Mon, 16 Oct 2017 10:35:54 -0400 From: Liming Sun To: Liming Sun , Ulf Hansson , "Rob Herring" , Mark Rutland , "Jaehoon Chung" , Kukjin Kim , "Krzysztof Kozlowski" CC: "linux-mmc@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-samsung-soc@vger.kernel.org" Subject: RE: [PATCH 0/9] Enable dw-mmc multi-card support Thread-Topic: [PATCH 0/9] Enable dw-mmc multi-card support Thread-Index: AQHTPthtXKpZ5PmCS0OG3cLvz1lwFqLml1zA Date: Mon, 16 Oct 2017 14:35:48 +0000 Message-ID: References: In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: spf=none (sender IP is ) smtp.mailfrom=lsun@mellanox.com; x-originating-ip: [216.156.69.42] x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1;AM2PR05MB0756;6:nl/pJCbis50qSDS9zApghrreWsmPo2gvR8Y9Qe/onVdUT/fqgl2MlO1Y9ZdyEbzuCB2X+qEfYev0xyvzxeYGLX/+v546MmmD/ETGHrLNhs/p7S4HIURnbYczIW3szxcesZRIMmfXCd0wL07uJKBXBuuotsfigbx8FP0CIjM0r6oVCEZPJit4+SOW/A8bKvL63u0vm18IhtAajkSbi1nKmuPXkKNH5uKq8S+MxSGTxOyJqPqGIfhOaa+lO0MYv+EtAuMVJ2c7wJsNw9buWQpUp4A7GpHGzIxD15282Hf+EFTIYpAew/+e/hCeyMftbNQatNe5lyfFdLSQXOj1z1tm6A==;5:deOhU00rfwuZKI8Onv7G4Ey64RfXXIjoVXKlcEab9iQJmlSianJyM8uZZQ7h9EvsqmSrtRWC/P2h4x4bkwg5al23O6xS6D8jVnLQoUuFI2rInaxmJKW8m5fTnNAPP+CtWm+EtqitSbXXry6KRq5xsvMTWoodpB30QvVKiZFPWDU=;24:XX8WHZLlW8XdCVRuwRuWulLFxAwFZI/wuUiScvf2LvqvMrYDSTKqy++dxAf62O07PlRFLjAoWvEoQJ2iwCFGzB1mBmNg+OMWO394kPmIJaI=;7:2+YNWCDx/iAhHtgTcetxLdn8M3WwK+p/tOPkQctrSC0+VPCPJznQXth1jo8cUZ8jh4ELEGKf513XQ9R0kaL2onG4QipX96FSzL/hwXHuJwYI6uBzJRk7CjEuDK+Ksj1wyOVZNC8LKKZNOBgs3hCdZEHDUSSsIJ4KGn53dM+4TWx519ML4VGHuv7acga77IaYUKFrB7eJd3p9m7S4DbvcvVR7SESIPtsiEkNkVn4v/SY= x-ms-exchange-antispam-srfa-diagnostics: SSOS;SSOR; x-ms-office365-filtering-correlation-id: ec3890f9-c51a-4e2d-85ed-08d514a33147 x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001)(2017030254152)(48565401081)(2017052603199)(201703131423075)(201703031133081)(201702281549075);SRVR:AM2PR05MB0756; x-ms-traffictypediagnostic: AM2PR05MB0756: x-exchange-antispam-report-test: UriScan:(180628864354917)(9452136761055)(258649278758335)(7411616537696); x-microsoft-antispam-prvs: x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(5005006)(8121501046)(100000703101)(100105400095)(10201501046)(3002001)(93006095)(93001095)(6055026)(6041248)(20161123562025)(20161123560025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123564025)(20161123555025)(20161123558100)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095);SRVR:AM2PR05MB0756;BCL:0;PCL:0;RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);SRVR:AM2PR05MB0756; x-forefront-prvs: 0462918D61 x-forefront-antispam-report: SFV:NSPM;SFS:(10009020)(6009001)(376002)(39860400002)(346002)(377454003)(199003)(13464003)(189002)(6116002)(102836003)(3846002)(25786009)(2950100002)(54356999)(76176999)(14454004)(2906002)(6506006)(6436002)(99286003)(54906003)(50986999)(53546010)(316002)(110136005)(229853002)(9686003)(5250100002)(105586002)(7736002)(101416001)(106356001)(53936002)(2900100001)(74316002)(4326008)(189998001)(66066001)(478600001)(68736007)(86362001)(33656002)(6246003)(5660300001)(7416002)(7696004)(8676002)(305945005)(3280700002)(81156014)(81166006)(8936002)(3660700001)(97736004)(55016002);DIR:OUT;SFP:1101;SCL:1;SRVR:AM2PR05MB0756;H:AM2PR05MB0753.eurprd05.prod.outlook.com;FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-originalarrivaltime: 16 Oct 2017 14:35:48.0382 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: a652971c-7d2e-4d9b-a6a4-d149256f461b X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM2PR05MB0756 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by nfs id v9GEa7OU020847 So far I saw one comment (an ACK) on PATCH 1/9. There seems no comments on other patches yet. Ulf, (or other maintainers), any comments or suggestions how to move forward for this patch series? Thanks, Liming -----Original Message----- From: linux-kernel-owner@vger.kernel.org [mailto:linux-kernel-owner@vger.kernel.org] On Behalf Of Liming Sun Sent: Friday, October 6, 2017 3:21 PM To: Ulf Hansson ; Rob Herring ; Mark Rutland ; Jaehoon Chung ; Kukjin Kim ; Krzysztof Kozlowski Cc: Liming Sun ; linux-mmc@vger.kernel.org; devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-samsung-soc@vger.kernel.org Subject: [PATCH 0/9] Enable dw-mmc multi-card support This series of commits enables the multi-card support for the dw-mmc controller. It includes two parts as below. The first part (patches 1-7) reverts the series of recent commits that removed the multi-card support with comments saying there was no such use case in the real world. Actually this feature is being used in Mellanox Bluefield SoC and has been requested by customers. The second part (patches 8-9) fixes the DesignWare multi-card support according to the dw-mmc databook (synnopsys: DesignWare Cores Mobile Storage Host Databook, 2.70a). It has changes to set the card number into the CMD register to multiplex requests to different cards when working in SD_MMC_CEATA mode, set the CTYPE / CLKENA / CDTHRCTL registers properly according to the spec, and parse the per-card configuration to match the Linux Documentation (bindings/mmc/synopsys-dw-mshc.txt). Liming Sun (9): Revert "Documentation: dw-mshc: deprecate num-slots" Revert "mmc: dw_mmc: remove the unnecessary slot variable" Revert "mmc: dw_mmc: use the 'slot' instead of 'cur_slot'" Revert "mmc: dw_mmc: remove the 'id' arguments about functions relevant to slot" Revert "mmc: dw_mmc: change the array of slots" Revert "mmc: dw_mmc: remove the loop about finding slots" Revert "mmc: dw_mmc: deprecated the "num-slots" property" mmc: dw_mmc: Support two SD_MMC_CE-ATA cards mmc: dw_mmc: Parse slot-specific configuration .../devicetree/bindings/mmc/synopsys-dw-mshc.txt | 16 +- drivers/mmc/host/dw_mmc-exynos.c | 4 +- drivers/mmc/host/dw_mmc.c | 277 ++++++++++++++++----- drivers/mmc/host/dw_mmc.h | 17 +- 4 files changed, 236 insertions(+), 78 deletions(-) -- 1.8.3.1 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Liming Sun Subject: RE: [PATCH 0/9] Enable dw-mmc multi-card support Date: Mon, 16 Oct 2017 14:35:48 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: Liming Sun , Ulf Hansson , Rob Herring , Mark Rutland , Jaehoon Chung , Kukjin Kim , Krzysztof Kozlowski Cc: "linux-mmc@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-samsung-soc@vger.kernel.org" List-Id: devicetree@vger.kernel.org So far I saw one comment (an ACK) on PATCH 1/9. There seems no comments on = other patches yet.=20 Ulf, (or other maintainers), any comments or suggestions how to move forwar= d for this patch series? Thanks, Liming -----Original Message----- From: linux-kernel-owner@vger.kernel.org [mailto:linux-kernel-owner@vger.ke= rnel.org] On Behalf Of Liming Sun Sent: Friday, October 6, 2017 3:21 PM To: Ulf Hansson ; Rob Herring ;= Mark Rutland ; Jaehoon Chung ; Kukjin Kim ; Krzysztof Kozlowski Cc: Liming Sun ; linux-mmc@vger.kernel.org; devicetree@v= ger.kernel.org; linux-kernel@vger.kernel.org; linux-arm-kernel@lists.infrad= ead.org; linux-samsung-soc@vger.kernel.org Subject: [PATCH 0/9] Enable dw-mmc multi-card support This series of commits enables the multi-card support for the dw-mmc contro= ller. It includes two parts as below. The first part (patches 1-7) reverts the series of recent commits that remo= ved the multi-card support with comments saying there was no such use case = in the real world. Actually this feature is being used in Mellanox Bluefiel= d SoC and has been requested by customers. The second part (patches 8-9) fixes the DesignWare multi-card support accor= ding to the dw-mmc databook (synnopsys: DesignWare Cores Mobile Storage Hos= t Databook, 2.70a). It has changes to set the card number into the CMD regi= ster to multiplex requests to different cards when working in SD_MMC_CEATA = mode, set the CTYPE / CLKENA / CDTHRCTL registers properly according to the= spec, and parse the per-card configuration to match the Linux Documentatio= n (bindings/mmc/synopsys-dw-mshc.txt). Liming Sun (9): Revert "Documentation: dw-mshc: deprecate num-slots" Revert "mmc: dw_mmc: remove the unnecessary slot variable" Revert "mmc: dw_mmc: use the 'slot' instead of 'cur_slot'" Revert "mmc: dw_mmc: remove the 'id' arguments about functions relevant to slot" Revert "mmc: dw_mmc: change the array of slots" Revert "mmc: dw_mmc: remove the loop about finding slots" Revert "mmc: dw_mmc: deprecated the "num-slots" property" mmc: dw_mmc: Support two SD_MMC_CE-ATA cards mmc: dw_mmc: Parse slot-specific configuration .../devicetree/bindings/mmc/synopsys-dw-mshc.txt | 16 +- drivers/mmc/host/dw_mmc-exynos.c | 4 +- drivers/mmc/host/dw_mmc.c | 277 ++++++++++++++++-= ---- drivers/mmc/host/dw_mmc.h | 17 +- 4 files changed, 236 insertions(+), 78 deletions(-) -- 1.8.3.1 From mboxrd@z Thu Jan 1 00:00:00 1970 From: lsun@mellanox.com (Liming Sun) Date: Mon, 16 Oct 2017 14:35:48 +0000 Subject: [PATCH 0/9] Enable dw-mmc multi-card support In-Reply-To: References: Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org So far I saw one comment (an ACK) on PATCH 1/9. There seems no comments on other patches yet. Ulf, (or other maintainers), any comments or suggestions how to move forward for this patch series? Thanks, Liming -----Original Message----- From: linux-kernel-owner@vger.kernel.org [mailto:linux-kernel-owner at vger.kernel.org] On Behalf Of Liming Sun Sent: Friday, October 6, 2017 3:21 PM To: Ulf Hansson ; Rob Herring ; Mark Rutland ; Jaehoon Chung ; Kukjin Kim ; Krzysztof Kozlowski Cc: Liming Sun ; linux-mmc at vger.kernel.org; devicetree at vger.kernel.org; linux-kernel at vger.kernel.org; linux-arm-kernel at lists.infradead.org; linux-samsung-soc at vger.kernel.org Subject: [PATCH 0/9] Enable dw-mmc multi-card support This series of commits enables the multi-card support for the dw-mmc controller. It includes two parts as below. The first part (patches 1-7) reverts the series of recent commits that removed the multi-card support with comments saying there was no such use case in the real world. Actually this feature is being used in Mellanox Bluefield SoC and has been requested by customers. The second part (patches 8-9) fixes the DesignWare multi-card support according to the dw-mmc databook (synnopsys: DesignWare Cores Mobile Storage Host Databook, 2.70a). It has changes to set the card number into the CMD register to multiplex requests to different cards when working in SD_MMC_CEATA mode, set the CTYPE / CLKENA / CDTHRCTL registers properly according to the spec, and parse the per-card configuration to match the Linux Documentation (bindings/mmc/synopsys-dw-mshc.txt). Liming Sun (9): Revert "Documentation: dw-mshc: deprecate num-slots" Revert "mmc: dw_mmc: remove the unnecessary slot variable" Revert "mmc: dw_mmc: use the 'slot' instead of 'cur_slot'" Revert "mmc: dw_mmc: remove the 'id' arguments about functions relevant to slot" Revert "mmc: dw_mmc: change the array of slots" Revert "mmc: dw_mmc: remove the loop about finding slots" Revert "mmc: dw_mmc: deprecated the "num-slots" property" mmc: dw_mmc: Support two SD_MMC_CE-ATA cards mmc: dw_mmc: Parse slot-specific configuration .../devicetree/bindings/mmc/synopsys-dw-mshc.txt | 16 +- drivers/mmc/host/dw_mmc-exynos.c | 4 +- drivers/mmc/host/dw_mmc.c | 277 ++++++++++++++++----- drivers/mmc/host/dw_mmc.h | 17 +- 4 files changed, 236 insertions(+), 78 deletions(-) -- 1.8.3.1