Jief_Machak Posted September 27, 2021 Share Posted September 27, 2021 3 hours ago, SavageAUS said: Has the CCPV been updated with new changes? Sources are always up to date, as they are shared, but you might need to re compile it yourself. Link to comment Share on other sites More sharing options...
Guest 5T33Z0 Posted September 27, 2021 Share Posted September 27, 2021 (edited) 5 hours ago, chris1111 said: Your Sample Config.plist https://github.com/CloverHackyColor/CloverBootloader/commit/f0a20247e9edc34f9eb55a3d9e14bbda858734f5 is not good for CloverConfigValidator The first Warning can be ignored because the SMBIOS section is empty The 2nd one is a kernel quirk which exists since OC 062. Edited September 27, 2021 by 5T33Z0 Link to comment Share on other sites More sharing options...
Jief_Machak Posted September 27, 2021 Share Posted September 27, 2021 On 8/10/2021 at 10:31 PM, kushwavez said: Yes, it is swapped for me too That should unswap the left and right arrow keys : AppleKeyFeeder.efi 1 Link to comment Share on other sites More sharing options...
Slice Posted September 27, 2021 Share Posted September 27, 2021 4 hours ago, joevt said: I use BBEdit.app. It can do regular expression searches. It has worksheets so you could also use grep while inside BBEdit.app. I use the worksheets for git and building as well. It's like Terminal.app but you can select commands and press enter to execute them and edit the output. Worksheets are an idea from the MPW days. MPW was cool because you could execute scripts to modify all the menus and add menu commands. It had commands to manipulate text windows, the current selection, etc. For example, MPW didn't have text block selection capabilities but you could make scripts/menu commands to add that functionality. For EFI projects, I like using -save-temps in the compiler commands. Then I can search the .i files to find the include file where a struct or function is defined. They can be used to find compile problems (it shows what the preprocessor produces). It's one way to find the include file for an EFI protocol for example. BBEdit is not free. I'll be happy with some open-source solution. Thanks for your suggestion about -save-temps. 11 hours ago, 5T33Z0 said: @yapan4 So 0x3E7 is the new CsrActiveConfig for Big Sur/Monterey? For beeing a "new" sample, I think it has an awful lot of obsolete options, patches, features and quirks for old OSes and CPUs enabled… Nothing new. The system can work with a value == 0. It is your choice to cancel some limitations, for example if you use unsigned kexts or unapproved kext. You may set 0xFFF to cancel all limitations but sure about bit 0x10 which prevents SystemUpdate. So the working mask can be 0xFEF. See explanations what each bit means and for a what. For example DTRACE. Are you really going to use it? 2 Link to comment Share on other sites More sharing options...
Slice Posted September 27, 2021 Share Posted September 27, 2021 On 9/25/2021 at 2:34 PM, Jief_Machak said: @Slice Thing to have a look : in CommonPatches.c, importing KernelCPUIDPatch() if needed (maybe there is a kernel cpuid patch mechanism we can use instead ?). We can retranslate our settings for FakeCPUID to OC corresponding patch. May be it will be enough. Link to comment Share on other sites More sharing options...
Jief_Machak Posted September 27, 2021 Share Posted September 27, 2021 10 minutes ago, Slice said: We can retranslate our settings for FakeCPUID to OC corresponding patch. May be it will be enough. It's definitely not my area of expertise, but if you tell me what to do, I can help. It's always best to avoid modifying OC to ease upgrade of OpenCorePkg. Although in that case, a whole new function is easy to import and won't generate conflict. Except maybe the line where that new function is called, but that's one line. 2 Link to comment Share on other sites More sharing options...
Slice Posted September 27, 2021 Share Posted September 27, 2021 12 hours ago, Jief_Machak said: I can see that hebuild.sh and ebuild.sh are almost the same. One difference, for example, is that they don't use the same mtoc binary. Shouldn't we update and keep only one ? Redundancy is always dangerous... mtoc difference should be excluded. But the main difference that hebuild doesn't recompile all sources. It is needed for me to recompile a few changed files. 1 minute ago, Jief_Machak said: It's definitely not my area of expertise, but if you tell me what to do, I can help. It's always best to avoid modifying OC to ease upgrade of OpenCorePkg. Although in that case, a whole new function is easy to import and won't generate conflict. Except maybe the line where that new function is called, but that's one line. Make it #if CLOVER_BUILD I don't remember OC syntax so I may propose CPUIDpatch should be same as Clover's FakeCPUID, and mask should be 0xffffff 1 Link to comment Share on other sites More sharing options...
chris1111 Posted September 27, 2021 Share Posted September 27, 2021 (edited) 4 hours ago, 5T33Z0 said: The first Warning can be ignored because the SMBIOS section is empty The 2nd one is a kernel quirk which exists since OC 062. 😀 If this is how it works exemple from my config.plist so this one it's not good And the secound one its the Sample Config.plist from source using by the Package Clover I am just report a bug Edited September 27, 2021 by chris1111 1 Link to comment Share on other sites More sharing options...
Matgen84 Posted September 27, 2021 Share Posted September 27, 2021 48 minutes ago, chris1111 said: 😀 If this is how it works exemple from my config.plist so this one it's not good And the secound one its the Sample Config.plist from source using by the Package Clover I don't have the same warnings with config-sample.plist using CCPV r5139 4be6350 Warning: FixHeaders exists in ACPI and ACPI/DSDT/Fixes. Delete FixHeaders from ACPI/DSDT/Fixes. 1 Link to comment Share on other sites More sharing options...
chris1111 Posted September 27, 2021 Share Posted September 27, 2021 6 minutes ago, Matgen84 said: I don't have the same warnings with config-sample.plist using CCPV r5139 4be6350 Warning: FixHeaders exists in ACPI and ACPI/DSDT/Fixes. Delete FixHeaders from ACPI/DSDT/Fixes. Here Latest commit (6a6c66a4a) with latest CCPV Warning: ProductName is not defined, the whole SMBIOS dict is ignored at line 953. Warning: FixHeaders exists in ACPI and ACPI/DSDT/Fixes. Delete FixHeaders from ACPI/DSDT/Fixes. Link to comment Share on other sites More sharing options...
Matgen84 Posted September 27, 2021 Share Posted September 27, 2021 (edited) 6 minutes ago, chris1111 said: Here Latest commit (6a6c66a4a) with latest CCPV Warning: ProductName is not defined, the whole SMBIOS dict is ignored at line 953. Warning: FixHeaders exists in ACPI and ACPI/DSDT/Fixes. Delete FixHeaders from ACPI/DSDT/Fixes. I have the two lines. If I uncommented ProductName, warning for ProductName disappears. Right, for CCPV, there is a duplicate key for fix headers in config-sample.plist. Edited September 27, 2021 by Matgen84 1 Link to comment Share on other sites More sharing options...
Slice Posted September 27, 2021 Share Posted September 27, 2021 Will be deleted 1 Link to comment Share on other sites More sharing options...
Jief_Machak Posted September 27, 2021 Share Posted September 27, 2021 5 hours ago, Slice said: But the main difference that hebuild doesn't recompile all sources Try ebuild.sh, it doesn't recompile all either. The cmd generated for compiling at hebuild.sh:567 is the same as ebuild.sh:579. Use a filediff and you'll see that the only difference is -DLESS_DEBUG It looks like hebuild.sh is outdate and does nothing more. Check with a file diff software. 5 hours ago, Slice said: 5 hours ago, Jief_Machak said: It's definitely not my area of expertise, but if you tell me what to do, I can help. It's always best to avoid modifying OC to ease upgrade of OpenCorePkg. Although in that case, a whole new function is easy to import and won't generate conflict. Except maybe the line where that new function is called, but that's one line. Make it #if CLOVER_BUILD I need everything enclosed with #if CLOVER_BUILD, so I can still compile a standalone OC from OpenCorePkg. But it's not what I meant. I meant : every OC line changed maybe a conflict when upgrading OpenCorePkg. Conflict happens when we change a line inside a function. Not so much when adding a function. So using OC mechanism is always a bit better if possible. 1 Link to comment Share on other sites More sharing options...
Slice Posted September 27, 2021 Share Posted September 27, 2021 1 hour ago, Jief_Machak said: Try ebuild.sh, it doesn't recompile all either. The cmd generated for compiling at hebuild.sh:567 is the same as ebuild.sh:579. Use a filediff and you'll see that the only difference is -DLESS_DEBUG It looks like hebuild.sh is outdate and does nothing more. Check with a file diff software. No, there are differences. I created hebuild.sh only for me because it is comfortable for me. Link to comment Share on other sites More sharing options...
Jief_Machak Posted September 27, 2021 Share Posted September 27, 2021 1 minute ago, Slice said: No, there are differences. I created hebuild.sh only for me because it is comfortable for me. I just said : there isn't any difference in the build command line. I'm pretty sure we can find a way to merge both in a way that it will still do what you want... In the meantime, hebuild.sh has a problem with mtoc binary name (line 215) Link to comment Share on other sites More sharing options...
Slice Posted September 27, 2021 Share Posted September 27, 2021 There is interesting TextEditor 😉 instead of Xcode 30 minutes ago, Jief_Machak said: I just said : there isn't any difference in the build command line. I'm pretty sure we can find a way to merge both in a way that it will still do what you want... In the meantime, hebuild.sh has a problem with mtoc binary name (line 215) Yes mtoc can be corrected. I use gcc53. Can the ebuild.sh script be converted into makefile? To type make all... Link to comment Share on other sites More sharing options...
Slice Posted September 27, 2021 Share Posted September 27, 2021 FakeCPIID works currently. If no more suggestion I can release 5140. 1 1 Link to comment Share on other sites More sharing options...
chris1111 Posted September 27, 2021 Share Posted September 27, 2021 (edited) try to use themes that looks like apple, is what people like 😀 Edited September 27, 2021 by chris1111 Link to comment Share on other sites More sharing options...
kushwavez Posted September 27, 2021 Share Posted September 27, 2021 12 hours ago, Jief_Machak said: That should unswap the left and right arrow keys : AppleKeyFeeder.efi yes, it’s good now with that! Link to comment Share on other sites More sharing options...
Jief_Machak Posted September 27, 2021 Share Posted September 27, 2021 2 hours ago, Slice said: There is interesting TextEditor 😉 instead of Xcode Eclipse is what I use. I didn't suggest it because I thought you wanted something small. Very handy for git. Link to comment Share on other sites More sharing options...
SavageAUS Posted September 28, 2021 Share Posted September 28, 2021 FakeCPIID works currently. If no more suggestion I can release 5140.Will ccpv be release with 5140? Is it updated with the changes?Sent from my iPhone using Tapatalk Link to comment Share on other sites More sharing options...
Slice Posted September 28, 2021 Share Posted September 28, 2021 9 hours ago, Jief_Machak said: Eclipse is what I use. I didn't suggest it because I thought you wanted something small. Very handy for git. Eclipse is 1000 times smaller then XCode13 and very powerful. 3 hours ago, SavageAUS said: Will ccpv be release with 5140? Is it updated with the changes? Sent from my iPhone using Tapatalk I hope it will be updated. 1 Link to comment Share on other sites More sharing options...
Jief_Machak Posted September 28, 2021 Share Posted September 28, 2021 57 minutes ago, Slice said: Eclipse is 1000 times smaller then XCode13 and very powerful. I agree, that's why I use it. I will share soon my project files that allows to use gdb from inside eclipse : not to know a single gdb command... 1 Link to comment Share on other sites More sharing options...
Matgen84 Posted September 28, 2021 Share Posted September 28, 2021 (edited) Hi @Slice @Jief_Machak @all My stupid question: I want to replace a USBMap.kext, to use instead a SSDT for my Z390 USB ports with a specific name. Is it necessary to drop tables (length) the original one, in config.plist ? I read something about that somewhere. Sorry for my bad english. Edited September 28, 2021 by Matgen84 Link to comment Share on other sites More sharing options...
Slice Posted September 28, 2021 Share Posted September 28, 2021 2 hours ago, Matgen84 said: Hi @Slice @Jief_Machak @all My stupid question: I want to replace a USBMap.kext, to use instead a SSDT for my Z390 USB ports with a specific name. Is it necessary to drop tables (length) the original one, in config.plist ? I read something about that somewhere. Sorry for my bad english. Yes, you have to drop similar tables to avoid symbol duplication. 1 Link to comment Share on other sites More sharing options...
Recommended Posts