Cookies op Tweakers

Tweakers maakt gebruik van cookies, onder andere om de website te analyseren, het gebruiksgemak te vergroten en advertenties te tonen. Door gebruik te maken van deze website, of door op 'Ga verder' te klikken, geef je toestemming voor het gebruik van cookies. Wil je meer informatie over cookies en hoe ze worden gebruikt, bekijk dan ons cookiebeleid.

Meer informatie

Door , , 8 reacties
Bron: MBM Homepage

Daneel schrijft: "Op dezelfde dag als de release van Motherboard Monitor 4.14 wordt ook versie 4.15 uitgebracht . Alex blijkt een probleempje met de Asus K7M over het hoofd te hebben gezien":

Released MBM 4.15, Jay Carter pointed out a problem with 4.14 and the Asus KM7, I am very sorry for those of you who already spend time on downloading and installing mbm 4.14. I was 100% a dump programming error, for testing I // a line and forgot to put it back on..... sorry
Motherboard Monitor

MBM 4.15 kun je hier naar binnen tanken.

Moderatie-faq Wijzig weergave

Reacties (8)

// Dit vind ik wel een beetje raar hoor. Gister 4.14 en vandaag 4.15. Geeft niet zo'n goede indruk. }>
Hoezo Bl@de?
Nooit een keertje aan het debuggen geweest?
Is vrij normaal alleen staat idd niet zo mooi als je commentaarquotes vergeet weg te halen...
#if 0
Dit doet het dus niet
#endif

werkt i.h.a. beter. Kan je op greppen en je ziet het beter staan ook...
</div><div class=b4>Is vrij normaal alleen staat idd niet zo mooi als je commentaarquotes vergeet weg te halen...</div><div class=b1>

Ehhh.... compilers slaan commentaar over. En dat is maar goed ook, anders zou iemand anders (of jezelf na een maand of wat) er veel te lang over moeten doen om de code te begrijpen.

En nog iets (goed lezen hoor) er staat dat Alex vergeten is om juist // weg te halen, zodat de compiler die line nog steeds als commentaar zag wat dus niet de bedoeling is. En in die regel stond juist een soort code dat het proggie het ook op de Asus KM7 werkt.

;) Martijn
(* Hey , niet zoveel commentaar *)
// Dit moet je gewoon niet vergeten
{Weg te halen}
Het is juist positief dat ie dezelfde dag nog een nieuwere versie uitgebracht heeft. Dat hoef je van de meeste bedrijven (incl m$) niet te verwachten.

Prima prestatie
Martijn heeft gelijk maar voor die mensen die zich afvragen wat er precies aan de hand was:

zowel de Asus K7M als de FIC SD-11 gebruiken de VIA 686a chipset, FIC heeft met de laatste bios update de monitor optie uitgezet.

MBM kijkt naar een register in de PCI bus om te kijken of de monitor optie aanstaat.

Na de nieuwe bios update van FIC kon MBM de monitor niet meer vinden, dit kwam door dat de bit in een register niet meer op 1 stond.

Om nu te kijken of de monitor toch gewoon werkte ondanks die bit heb ik de controlle van deze bit even "gerem'd" zoals dat vroegen in Basic werd genoemd.

dus de code:
PCIReg:=ReadPciReg(Bus, Device, 4, $74);
if PCIReg and $1 = 1 then
begin
PCIReg:=ReadPciReg(Bus, Device, 4, $70);
IsaBase:=PCIReg and $FFFE;
ChipType:=Via686A;
UsedBus:=VIA686ABus;
end;

veranderde in:

PCIReg:=ReadPciReg(Bus, Device, 4, $74);
// if PCIReg and $1 = 1 then
begin
PCIReg:=ReadPciReg(Bus, Device, 4, $70);
IsaBase:=PCIReg and $FFFE;
ChipType:=Via686A;
UsedBus:=VIA686ABus;
end;

dus de controlle werdt niet meer uitgevoerd, beta ging naar tester en later bleek dat dit niet de juiste oplossing was en dus moest er net voor deze code een stukje bij die de bit in kwestie op 1 zet voor de FIC SD-11

Op zich geen probleem en MBM werkte zoals het hoort, maar de Asus K7M gebruikt een extra toegevoegde sensor chip, en dus is bij Asus de bit in kwestie altijd een 0. Daar ik echter vergeten was de // er uit te halen werdt ook bij de Asus de 686A opgestart, en dus niet de juist (W83782D) chip.

Hoop dat het nu wat duidelijker is

Groet,

Alex
Puik stukje software, snelle bugfix, en een eerlijke programmeur die toegeeft dat er een fout in zat.

Masta-T is tevreden.

Op dit item kan niet meer gereageerd worden.



Apple iOS 10 Google Pixel Apple iPhone 7 Sony PlayStation VR AMD Radeon RX 480 4GB Battlefield 1 Google Android Nougat Watch Dogs 2

© 1998 - 2016 de Persgroep Online Services B.V. Tweakers vormt samen met o.a. Autotrack en Carsom.nl de Persgroep Online Services B.V. Hosting door True