[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 2/3] usb/hcd-xhci: Move qemu-xhci device to hcd-xhci-pci.c
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v2 2/3] usb/hcd-xhci: Move qemu-xhci device to hcd-xhci-pci.c |
Date: |
Thu, 25 Jun 2020 10:17:38 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0 |
On 6/25/20 10:06 AM, Markus Armbruster wrote:
> Sai Pavan Boddu <sai.pavan.boddu@xilinx.com> writes:
>
>> Move pci specific devices to new file. This set the environment to move all
>> pci specific hooks in hcd-xhci.c to hcd-xhci-pci.c.
>>
>> Signed-off-by: Sai Pavan Boddu <sai.pavan.boddu@xilinx.com>
>> ---
>> hw/usb/hcd-xhci-pci.c | 64
>> +++++++++++++++++++++++++++++++++++++++++++++++++++
>> hw/usb/hcd-xhci.c | 32 ++------------------------
>> hw/usb/hcd-xhci.h | 2 ++
>> 3 files changed, 68 insertions(+), 30 deletions(-)
>> create mode 100644 hw/usb/hcd-xhci-pci.c
>>
>> diff --git a/hw/usb/hcd-xhci-pci.c b/hw/usb/hcd-xhci-pci.c
>> new file mode 100644
>> index 0000000..26af683
>> --- /dev/null
>> +++ b/hw/usb/hcd-xhci-pci.c
>> @@ -0,0 +1,64 @@
>> +/*
>> + * USB xHCI controller with PCI system bus emulation
>
> Scratch "system".
>
>> + *
>> + * Copyright (c) 2011 Securiforest
>> + * Date: 2011-05-11 ; Author: Hector Martin <hector@marcansoft.com>
>
> Let's use the opportunity to drop the "Date: " part, because we don't
> have it anywhere else.
Good opportunity to suggest the SPDX tags again :P
/*
* SPDX-FileCopyrightText: 2011 Securiforest
* SPDX-FileContributor: Hector Martin <hector@marcansoft.com>
* SPDX-FileCopyrightText: 2020 Xilinx Inc.
* SPDX-FileContributor: Sai Pavan Boddu <sai.pavan.boddu@xilinx.com>
* SPDX-License-Identifier: GPL-2.0-or-later
*/
https://spdx.org/rdf/ontology/spdx-2-0-rc/dataproperties/fileContributor___-1635717172.html
>
>> + * Based on usb-ohci.c, emulates Renesas NEC USB 3.0
>> + * Date: 2020-03-01; Author: Sai Pavan Boddu <sai.pavan.boddu@xilinx.com>
>
> And no new "Date: " parts, please.
>
>> + * Moved the pci specific content for hcd-xhci.c to hcd-xhci-pci.c
>> + *
>> + * This library is free software; you can redistribute it and/or
>> + * modify it under the terms of the GNU Lesser General Public
>> + * License as published by the Free Software Foundation; either
>> + * version 2 of the License, or (at your option) any later version.
>> + *
>> + * This library is distributed in the hope that it will be useful,
>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
>> + * Lesser General Public License for more details.
>> + *
>> + * You should have received a copy of the GNU Lesser General Public
>> + * License along with this library; if not, see
>> <http://www.gnu.org/licenses/>.
>> + */
> [...]
>
- [PATCH v2 0/3] Make hcd-xhci independent of pci hooks, Sai Pavan Boddu, 2020/06/24
- [PATCH v2 2/3] usb/hcd-xhci: Move qemu-xhci device to hcd-xhci-pci.c, Sai Pavan Boddu, 2020/06/24
- [PATCH v2 1/3] usb/hcd-xhci: Make dma read/writes hooks pci free, Sai Pavan Boddu, 2020/06/24
- [PATCH v2 3/3] usb/hcd-xhci: Split pci wrapper for xhci base model, Sai Pavan Boddu, 2020/06/24
- Re: [PATCH v2 3/3] usb/hcd-xhci: Split pci wrapper for xhci base model, Markus Armbruster, 2020/06/25
- RE: [PATCH v2 3/3] usb/hcd-xhci: Split pci wrapper for xhci base model, Sai Pavan Boddu, 2020/06/25
- Re: [PATCH v2 3/3] usb/hcd-xhci: Split pci wrapper for xhci base model, Markus Armbruster, 2020/06/26
- RE: [PATCH v2 3/3] usb/hcd-xhci: Split pci wrapper for xhci base model, Sai Pavan Boddu, 2020/06/26
- Re: [PATCH v2 3/3] usb/hcd-xhci: Split pci wrapper for xhci base model, Markus Armbruster, 2020/06/27
- Re: [PATCH v2 3/3] usb/hcd-xhci: Split pci wrapper for xhci base model, Gerd Hoffmann, 2020/06/29
- RE: [PATCH v2 3/3] usb/hcd-xhci: Split pci wrapper for xhci base model, Sai Pavan Boddu, 2020/06/30