[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v7 0/8] Mac Old World ROM experiment
From: |
Mark Cave-Ayland |
Subject: |
Re: [PATCH v7 0/8] Mac Old World ROM experiment |
Date: |
Tue, 30 Jun 2020 20:30:52 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.9.0 |
On 29/06/2020 19:55, BALATON Zoltan wrote:
> This is now a minimal set of patches needed to make it possible to
> experiment with a firmware ROM from real hardware. After finding out
> that the board firmware does not probe PCI devices but expects them at
> known fixed addresses we only need to change the address of the macio
> device to get the firmware correctly map it. This allows dropping
> workarounds in previous versions for this and now only the minimal set
> of patches are included to get the firmware loaded and do something.
> (Also excluded the grackle revision and machine ID pathes for now that
> may be needed as the firmware accesses these but seems to go further
> without them so until we hit a problem we can live without it,
> although I wonder if this causes us unnecessary debugging later so
> unless they cause regressions they could be merged).
>
> I still don't get video output but at least it talks to the GPU chip
> now so it can be debugged and improved (this will either need
> emulating the correct chip the firmware has a driver for or an OF
> compliant ROM for the emulated card).
>
> As before the I2C part (patches 6-8) is RFC and unfinished but the
> first 5 patches should be good enough now. I hope someone can take
> care of I2C, I can look at the ati-vga side later.
If you can sort out the issue with masking in patches 1 and 2 then I'd be happy
to
take patches 1-5. Obviously there is still some discussion around the i2c part,
so I
can wait a few more days to see what the outcome is there: the patches
generally seem
okay, the one change I would like to see is to add a comment around the SPD
parts
mentioning that these are only used by the real G3 ROM and not OpenBIOS.
My only concern is whether an incomplete i2c implementation could cause OSs that
currently boot to hang, so it is important that you can test a variety of OS
images
from MacOS to Linux and BSD to ensure that it doesn't cause any regression.
ATB,
Mark.
- [PATCH v7 4/8] mac_oldworld: Drop some variables, (continued)
- [PATCH v7 4/8] mac_oldworld: Drop some variables, BALATON Zoltan, 2020/06/29
- [PATCH v7 3/8] mac_oldworld: Drop a variable, use get_system_memory() directly, BALATON Zoltan, 2020/06/29
- [PATCH v7 2/8] mac_newworld: Allow loading binary ROM image, BALATON Zoltan, 2020/06/29
- [PATCH v7 1/8] mac_oldworld: Allow loading binary ROM image, BALATON Zoltan, 2020/06/29
- [PATCH v7 5/8] mac_oldworld: Change PCI address of macio to match real hardware, BALATON Zoltan, 2020/06/29
- [PATCH v7 8/8] mac_oldworld: Add SPD data to cover RAM, BALATON Zoltan, 2020/06/29
- [PATCH v7 6/8] i2c: Match parameters of i2c_start_transfer and i2c_send_recv, BALATON Zoltan, 2020/06/29
- [RFC PATCH v7 7/8] WIP macio/cuda: Attempt to add i2c support, BALATON Zoltan, 2020/06/29
- Re: [PATCH v7 0/8] Mac Old World ROM experiment,
Mark Cave-Ayland <=