Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31225 )
Change subject: src/mainboard/lenovo/g505s/Kconfig: disable DRIVERS_INTEL_WIFI by default ......................................................................
Patch Set 2:
Patch Set 2:
I think that until we find at least one G505S user who needs Intel WiFi we could safely default it to "n". Why default this to "y" if we don't know anyone who needs it?
My g505s now has Intel WiFi. Previously it had none.
Patch Set 2:
I'm not aware of any G505s's shipped with an Intel wifi adapter either, but I can't really prove a negative.
Irrelevant what it shipped with. It was alreydy open up once for flashing?
There is a hardware maintenance manual PDF at https://github.com/g505s-opensource-researcher/g505s-proprietary/tree/master... , which among others lists only Intel 135 as the supported replacement, but if you search the Internet you could not find any G505S with it - only some other Ideapad G-series Intel laptops which have been mentioned at this manual. To me it's a good enough proof.
It's a slot, making WiFi card user changeable.
My point is that the default settings should suit the needs of majority, especially considering that in this particular case it's likely that minority doesn't exist. Meanwhile I've already found myself a couple of times in a situation where I had to disable DRIVERS_INTEL_WIFI in order to save a few kilobytes to make enough remaining room for the last floppy image of my interest
You are in the minority with your need to tune up last kilobytes from ramstage. You need to adopt, probably should be building from local branch anyways. Default build is clearly not for you.