2 comments:
File src/mainboard/siemens/mc_apl1/variants/mc_apl2/mainboard.c:
Patch Set #1, Line 47: dev = dev_find_slot(0, PCH_DEVFN_SDCARD);
Would pcidev_path_on_root(PCH_DEVFN_SDCARD) work here instead?
There are some aspects of dev_find_slot() why I would like to see it deprecated. One of them is that not only does the 'slot' aka 'dev' section of the PCI path BDF has to match, but also the '.fn' part.
Patch Set #1, Line 55: write32(base + SD_CAP_BYP, SD_CAP_BYP_EN);
Looks like arithmetics performed on void pointer. AFAIR this is GCC extension not in the C standard but I don't remember if we generally approve or disapprove these.
To view, visit change 32542. To unsubscribe, or for help writing mail filters, visit settings.