Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Subject: Re: [PATCH 3/3] docs: mips: remove no longer needed au1xxx_ide.rst
 documentation
To:     Jonathan Corbet <corbet@lwn.net>
Cc:     "David S . Miller" <davem@davemloft.net>,
        Ralf Baechle <ralf@linux-mips.org>,
        Paul Burton <paul.burton@mips.com>,
        James Hogan <jhogan@kernel.org>, linux-ide@vger.kernel.org,
        linux-mips@vger.kernel.org, linux-doc@vger.kernel.org,
        linux-kernel@vger.kernel.org
From:   Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Message-ID: <a3c6522f-ab0e-8212-a049-a31286b36ad7@samsung.com>
Date:   Mon, 2 Mar 2020 15:53:58 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101
        Thunderbird/60.8.0
MIME-Version: 1.0
In-Reply-To: <20200116122938.20789-4-b.zolnierkie@samsung.com>
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Content-Type: text/plain; charset="utf-8"
CMS-TYPE: 201P
References: <20200116122938.20789-1-b.zolnierkie@samsung.com>
        <CGME20200116122952eucas1p2060adca6613ae02096ecc80319e5d197@eucas1p2.samsung.com>
        <20200116122938.20789-4-b.zolnierkie@samsung.com>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


Hi Jonathan,

Gentle ping.

Can we get your ACK to merge this patch through MIPS (or IDE) tree?

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics

On 1/16/20 1:29 PM, Bartlomiej Zolnierkiewicz wrote:
> Since the au1xxx-ide IDE host driver is no longer needed its documentation
> can be removed.
> 
> Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> ---
>  Documentation/mips/au1xxx_ide.rst | 130 ------------------------------
>  Documentation/mips/index.rst      |   2 -
>  2 files changed, 132 deletions(-)
>  delete mode 100644 Documentation/mips/au1xxx_ide.rst
> 
> diff --git a/Documentation/mips/au1xxx_ide.rst b/Documentation/mips/au1xxx_ide.rst
> deleted file mode 100644
> index 2f9c2cff6738..000000000000
> --- a/Documentation/mips/au1xxx_ide.rst
> +++ /dev/null
> @@ -1,130 +0,0 @@
> -.. include:: <isonum.txt>
> -
> -======================
> -MIPS AU1XXX IDE driver
> -======================
> -
> -Released 2005-07-15
> -
> -About
> -=====
> -
> -This file describes the 'drivers/ide/au1xxx-ide.c', related files and the
> -services they provide.
> -
> -If you are short in patience and just want to know how to add your hard disc to
> -the white or black list, go to the 'ADD NEW HARD DISC TO WHITE OR BLACK LIST'
> -section.
> -
> -
> -License
> -=======
> -
> -:Copyright: |copy| 2003-2005 AMD, Personal Connectivity Solutions
> -
> -This program is free software; you can redistribute it and/or modify it under
> -the terms of the GNU General Public License as published by the Free Software
> -Foundation; either version 2 of the License, or (at your option) any later
> -version.
> -
> -THIS SOFTWARE IS PROVIDED ``AS IS`` AND ANY EXPRESS OR IMPLIED WARRANTIES,
> -INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND
> -FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHOR
> -BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
> -CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
> -SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
> -INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
> -CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
> -ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
> -POSSIBILITY OF SUCH DAMAGE.
> -
> -You should have received a copy of the GNU General Public License along with
> -this program; if not, write to the Free Software Foundation, Inc.,
> -675 Mass Ave, Cambridge, MA 02139, USA.
> -
> -Note:
> -      for more information, please refer "AMD Alchemy Au1200/Au1550 IDE
> -      Interface and Linux Device Driver" Application Note.
> -
> -
> -Files, Configs and Compatibility
> -================================
> -
> -Two files are introduced:
> -
> -  a) 'arch/mips/include/asm/mach-au1x00/au1xxx_ide.h'
> -     contains : struct _auide_hwif
> -
> -                - timing parameters for PIO mode 0/1/2/3/4
> -                - timing parameters for MWDMA 0/1/2
> -
> -  b) 'drivers/ide/mips/au1xxx-ide.c'
> -     contains the functionality of the AU1XXX IDE driver
> -
> -Following extra configs variables are introduced:
> -
> -  CONFIG_BLK_DEV_IDE_AU1XXX_PIO_DBDMA
> -	- enable the PIO+DBDMA mode
> -  CONFIG_BLK_DEV_IDE_AU1XXX_MDMA2_DBDMA
> -	- enable the MWDMA mode
> -
> -
> -Supported IDE Modes
> -===================
> -
> -The AU1XXX IDE driver supported all PIO modes - PIO mode 0/1/2/3/4 - and all
> -MWDMA modes - MWDMA 0/1/2 -. There is no support for SWDMA and UDMA mode.
> -
> -To change the PIO mode use the program hdparm with option -p, e.g.
> -'hdparm -p0 [device]' for PIO mode 0. To enable the MWDMA mode use the option
> --X, e.g. 'hdparm -X32 [device]' for MWDMA mode 0.
> -
> -
> -Performance Configurations
> -==========================
> -
> -If the used system doesn't need USB support enable the following kernel
> -configs::
> -
> -    CONFIG_IDE=y
> -    CONFIG_BLK_DEV_IDE=y
> -    CONFIG_IDE_GENERIC=y
> -    CONFIG_BLK_DEV_IDEPCI=y
> -    CONFIG_BLK_DEV_GENERIC=y
> -    CONFIG_BLK_DEV_IDEDMA_PCI=y
> -    CONFIG_BLK_DEV_IDE_AU1XXX=y
> -    CONFIG_BLK_DEV_IDE_AU1XXX_MDMA2_DBDMA=y
> -    CONFIG_BLK_DEV_IDEDMA=y
> -
> -Also define 'IDE_AU1XXX_BURSTMODE' in 'drivers/ide/mips/au1xxx-ide.c' to enable
> -the burst support on DBDMA controller.
> -
> -If the used system need the USB support enable the following kernel configs for
> -high IDE to USB throughput.
> -
> -::
> -
> -    CONFIG_IDE_GENERIC=y
> -    CONFIG_BLK_DEV_IDEPCI=y
> -    CONFIG_BLK_DEV_GENERIC=y
> -    CONFIG_BLK_DEV_IDEDMA_PCI=y
> -    CONFIG_BLK_DEV_IDE_AU1XXX=y
> -    CONFIG_BLK_DEV_IDE_AU1XXX_MDMA2_DBDMA=y
> -    CONFIG_BLK_DEV_IDEDMA=y
> -
> -Also undefine 'IDE_AU1XXX_BURSTMODE' in 'drivers/ide/mips/au1xxx-ide.c' to
> -disable the burst support on DBDMA controller.
> -
> -
> -Acknowledgments
> -===============
> -
> -These drivers wouldn't have been done without the base of kernel 2.4.x AU1XXX
> -IDE driver from AMD.
> -
> -Additional input also from:
> -Matthias Lenk <matthias.lenk@amd.com>
> -
> -Happy hacking!
> -
> -Enrico Walther <enrico.walther@amd.com>
> diff --git a/Documentation/mips/index.rst b/Documentation/mips/index.rst
> index a93c2f65884c..d5ad8c00f0bd 100644
> --- a/Documentation/mips/index.rst
> +++ b/Documentation/mips/index.rst
> @@ -10,8 +10,6 @@ MIPS-specific Documentation
>  
>     ingenic-tcu
>  
> -   au1xxx_ide
> -
>  .. only::  subproject and html
>  
>     Indices
>