[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 0/7] floppy: build as modules.
From: |
Gerd Hoffmann |
Subject: |
Re: [PATCH 0/7] floppy: build as modules. |
Date: |
Thu, 5 Aug 2021 09:11:55 +0200 |
On Wed, Aug 04, 2021 at 05:19:02PM +0200, Philippe Mathieu-Daudé wrote:
> +Mark
>
> On 8/4/21 4:27 PM, Gerd Hoffmann wrote:
> > Some code shuffling needed beforehand due to floppy being part of
> > several platforms. While being at it also make floppy optional
> > in pc machine type.
>
> > floppy: move fdctrl_init_sysbus
> > floppy: move sun4m_fdctrl_init
>
> https://www.mail-archive.com/qemu-block@nongnu.org/msg84008.html
>
> Mark suggested:
>
> You may be able to simplify this further by removing the
> global legacy init functions fdctrl_init_sysbus() and
> sun4m_fdctrl_init(): from what I can see fdctrl_init_sysbus()
> is only used in hw/mips/jazz.c and sun4m_fdctrl_init() is only
> used in hw/sparc/sun4m.c so you might as well inline them or
> move the functions to the relevant files.
>
> I did it and plan to send during 6.2. Sounds simpler than module.
> You could easily rebase your series on top (or I can include your
> patches while sending).
Feel free to include them. But I can also rebase when your patches
landed upstream. Your choice ;)
take care,
Gerd
- [PATCH 0/7] floppy: build as modules., Gerd Hoffmann, 2021/08/04
- [PATCH 2/7] floppy: move isa_fdc_init_drives + fdctrl_init_drives, Gerd Hoffmann, 2021/08/04
- [PATCH 1/7] floppy: move isa_fdc_get_drive_type to separate source file., Gerd Hoffmann, 2021/08/04
- [PATCH 4/7] floppy: move sun4m_fdctrl_init, Gerd Hoffmann, 2021/08/04
- [PATCH 5/7] floppy: move cmos_get_fd_drive_type, Gerd Hoffmann, 2021/08/04
- [PATCH 3/7] floppy: move fdctrl_init_sysbus, Gerd Hoffmann, 2021/08/04
- [PATCH 7/7] pc: add floppy=OnOffAuto, Gerd Hoffmann, 2021/08/04
- [PATCH 6/7] floppy: build as modules., Gerd Hoffmann, 2021/08/04
- Re: [PATCH 0/7] floppy: build as modules., Philippe Mathieu-Daudé, 2021/08/04
- Re: [PATCH 0/7] floppy: build as modules.,
Gerd Hoffmann <=