Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964875AbcKXLKj (ORCPT ); Thu, 24 Nov 2016 06:10:39 -0500 Received: from mout.kundenserver.de ([212.227.17.10]:56767 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S937225AbcKXLKg (ORCPT ); Thu, 24 Nov 2016 06:10:36 -0500 From: Arnd Bergmann To: linux-arm-kernel@lists.infradead.org Cc: Ziji Hu , Gregory CLEMENT , Hilbert Zhang , Andrew Lunn , Ulf Hansson , Romain Perier , Liuliu Zhao , Peng Zhu , linux-kernel@vger.kernel.org, Nadav Haklai , "Jack(SH) Zhu" , Victor Gu , Doug Jones , Jisheng Zhang , Yehuda Yitschak , Marcin Wojtas , Xueping Liu , Shiwu Zhang , Yu Cao , Sebastian Hesselbarth , devicetree@vger.kernel.org, Jason Cooper , Hanna Hawa , Kostya Porotchkin , Rob Herring , Ryan Gao , "Wei(SOCP) Liu" , Thomas Petazzoni , Keji Zhang , linux-mmc@vger.kernel.org, Adrian Hunter , Jimmy Xu , Wilson Ding Subject: Re: [PATCH 7/10] mmc: sdhci-xenon: Add support to PHYs of Marvell Xenon SDHC Date: Thu, 24 Nov 2016 12:09:41 +0100 Message-ID: <23084629.DpfWxNEgoY@wuerfel> User-Agent: KMail/5.1.3 (Linux/4.4.0-34-generic; KDE/5.18.0; x86_64; ; ) In-Reply-To: References: <2204525.IWIYQVjIXl@wuerfel> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:diZ/ruI2rQ7yGhoo5Vj5aDtieQtyn67MZlMw58o3g8ifq7Ueolj 9StZMFpE1MmFwV4mGlnAkIm8V58DfobL2vI+OvuQPNUI+XgC7uR9JItmiqPa0JSJHbbiGWF dLgDlXx0/xGXKYinWBd7XfjKXLbksC7OYzA/V4Nr6G4beXreqgIPscuFKDTgxE1cweQlgpJ X/pdozn0bOOJUHzF/wqvg== X-UI-Out-Filterresults: notjunk:1;V01:K0:+mwlYqynZwc=:ZSSZsGdFDIiKItFdvgVc0J Wu0w4Agvwcf4JjTpul9c/k8XzZwiCwqaXDOFWb4FUc5XZ583Cafx/N7faVGNjw3jOGRvXTFuV L6bYx+7dWTh4zVXKDRMA3zDA1mJDGKjhFBE4so3jBAaa0PQl89w5qSyRQkexLIpi6+E/V7F2T KfUFYAHe2PeTlSMsKVgMYZia76MNDfrfOlA+sjyyVzMUS2V+sTMPnw12GaVxjssgUyJibmU3m gAbeH6ejedPFJ3+e/y5zXA5GkSlouWnG7mLHy/6uPalID+r26F5o2zKe4043V0ZDE7KxMSxxQ SLPNTzRS8E8YuKT6fNAldViaopES8FllAPkiKRhL36x3OyRRklOmZUmyA10bfT3srO54gtsW7 +btr4LbymLqxn+pJImNGwEoqc+wxj9MAJrZhArkNLiAoOxGeC6+yDBb4WnS4sNJ2DW3o68CnK XD3eIxJN/yiX5s9k/qy4P1NgSVlwHE9prXC3GhENx/M/UN1Cj16bJXo030uvYjO+o3pryh+7n msTTuEjpjnPqy/93E5aOxwAMfH28Ozm/y7eyYPgC/XxzReg/HOkShbvht6HXxIvaQ3Px60dZC /J35uDqMcI0wsYoopeOaSjfu6QFxPuulUOTrX9PfUN/HMc5vuGbAnY77BCplxHA+pjn+uqQad DmEyE7DSKQpGE1LNC9WyHO+n1SF0VSY/AVXTUgrPm3tp/CpGcSF7/Wp8ePVz0x29Us6A89nYW ZIQdn1defuhJB6Wd Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 809 Lines: 17 On Thursday, November 24, 2016 6:57:18 PM CET Ziji Hu wrote: > > > > Please explain in the changelog why this is not a generic > > phy driver (or three of them). > > > Actually we tried to put the PHY code into Linux PHY framework. > But it cannot fit in Linux common PHY framework. > > Our Xenon SDHC PHY register is a part of Xenon SDHC register set. > Besides, during MMC initialization, MMC sequence has to call several PHY functions to complete timing setting. > In those PHY setting functions, they have to access SDHC register and know current MMC setting, such as bus width, clock frequency and speed mode. > As a result, we have to implement PHY under MMC directory. > Ok, that makes sense, just put the same text in the changelog comment. Arnd