[PATCH] bcm63xx: t-com w303v typ B: detect wifi chip & fix default failsafe mode [21.02.0]

[hontz1]

sprom fallback need to intialize the wlan pci wifi-bridge correctly.
and reset button corrected - no regular boot without pressing the reset key fixed

• broken wifi->fixed
• default boot into failsafe->fixed

i compiled and tested only with the kernel 5.4 version and it is working now (wifi+boot),
kernel 5.10 should be similar, but is untested.
i also made a wiki at https://github.com/hontz1/openwrt_readme_t-com-w303v-B#readme

[adschm]

This has formal issues regarding the commit title:

Use "bcm63xx:" prefix and describe the specific change you do, not just which file you touch. (Essentially a shorter version of the commit messages you already have).

https://openwrt.org/submitting-patches

@Noltari should be able to judge the content.

[hontz1]

it is only a setup up option of existing code, that's why you can apply this, rewrite my sentences.
my name must not be included due to this.
thanks

[adschm]

This still needs to be brought in proper shape. Please update or we will close this due to inactivity.

[hontz1]

today i wrote email to noltari as suggested. i am waiting for answer.
i did not know, that he was not contacted already

[hontz1]

hello @adschm ,
@Noltari does not answer, do you have an other idea besides closing and dropping?
thanks

[adschm]

You still have formal issues, i.e. generic titles without proper prefix etc., commit message not wrapped properly.

Apart from that, you could describe the reasons for your changes better.

[hontz1]

danke adschm. hier verlässt mich eine Schulenglisch. wenn du mir bitte gern helfen könntest. :)

[adschm]

danke adschm. hier verlässt mich eine Schulenglisch. wenn du mir bitte gern helfen könntest. :)

Deine Commit-Titel entsprechen nicht den Konventionen. Der Commit-Titel hat immer ein Prefix (in diesem Fall den Target-Namen) und sollte darüber hinaus spezifisch das Problem beschreiben, nicht welche Datei du änderst.

Ein guter Name für den ersten Commit wäre also z.B. "bcm63xx: fix active state for reset button on Speedport W-303V". Dadurch weiß der Leser sofort, worum es geht.

In der Commit-Message erklärst du dann, warum diese Änderung notwendig ist:
Der bisherige Wert "0" bzw. "ACTIVE_HIGH" für den Reset-Button ist falsch. Dies hat bisher dazu geführt, dass der Router in den Failsafe-Mode gebootet hat. Mit der Änderung bootet der Router normal.

Nach dem gleichen Prinzip updatest du dann die anderen beiden Commits.

Beachte auch, dass es mir um die Commits geht, nicht um den PR.

Im Wesentlichen sind deine Commit-Messages ja okay, man versteht zumindest grundsätzlich das Problem; man könnte das aber noch etwas besser aufbereiten. Im Wesentlichen reicht es wohl aus, wenn du den Inhalt deines ersten Posts nimmst und ihn auf die Commits aufteilst.

[hontz1]

danke, dass du den Inhalt verstehst. gern würde ich den commit-text ändern, aber hier auf GitHub.com kann ich das nicht mehr editieren.
finde den edit button nicht.
eh du mir jetzt alles schritt für schritt erklärst, kannst du es auch bitte machen, da es komplizierter scheint.
sonst komme ich hier nicht mehr weiter. wie gesagt, immer im sinne derer, die noch so ein alten router haben. mir ist die persönliche Erwähnung nicht so wichtig.
die Dateien von meinem Rechner sind leider bereits gelöscht, wegen Platzmangel. wäre also komplett neu aufzusetzen.
ich kann dir auch mein pw geben.
hier noch ein Screenshot, falls es doch nur ein button ist. danke

gerade noch GitHub desktop installiert, kann dort aber auch nichts mehr machen

[hontz1]

sonst lassen wir es eben, steht jetzt alles auch im wiki.

[hontz1]

alles klar kann geschlossen werden, vielen dank und ist wie auch schon vor 20 jahren .....