Added compatibility to kernel 6.4

[jagiella]

Änderungen mit kernel 6.4:

• die Funktionssignatur von class_create(...) hat sich geändert
• die Funktion of_modalias_node(...) wurde durch of_alias_from_compatible(...) ersetzt

[alexreinert]

Der PR ist so nicht fertig für einen Merge, da fehlt jegliche Anpassung an den Versionen und Paketen. Durch die vielen inkludierten Merge Commits ist er auch nicht fähig für einen Merge, ein PR sollte im Regelfall nur einen Commit beinhalten bevor er gemerged wird. Aber auch inhaltlich macht eine Umsetzung per Conditionell Macro mehr Sinn, da hier einfach zu viel anderer Code zusätzlich in den Precompiler ifs drin ist.

Lange Rede, kurzer Sinn: Declined

[alexreinert]

Und an sich haben ich einen entsprechenden Change auch bereits seit gestern in meinem Dev System fertig, der läuft grade durch die ganzen Tests (und wird erst danach hier gemerged).

[jagiella]

Der PR ist so nicht fertig für einen Merge, da fehlt jegliche Anpassung an den Versionen und Paketen.

Wie läuft das denn mit anderen PRs? Gibst du nicht die Versionsänderungen vor?

Durch die vielen inkludierten Merge Commits ist er auch nicht fähig für einen Merge, ein PR sollte im Regelfall nur einen Commit beinhalten bevor er gemerged wird.

Normalerweise kann der PR doch als squash commit gemergt werden und dann ist es nur noch ein einzelner Commit auf deinem master.

Aber auch inhaltlich macht eine Umsetzung per Conditionell Macro mehr Sinn, da hier einfach zu viel anderer Code zusätzlich in den Precompiler ifs drin ist.

Ich hab mal einen Gegenvorschlag mit conditional macros in einem Parallel-PR #500 gepusht (nur 1 Commit :wink: )

Lange Rede, kurzer Sinn: Declined

[alexreinert]

Der PR ist so nicht fertig für einen Merge, da fehlt jegliche Anpassung an den Versionen und Paketen.

Wie läuft das denn mit anderen PRs? Gibst du nicht die Versionsänderungen vor?

Das ist einerseits die Modulversionsnummer, die einfach innerhalb der jeweiligen Kernel Modules imkrementiert wird und andererseits die Version des Debian Pakets im Erstellungsschritt. Ich fahre diese Repository so, dass auf dem main branch immer das vorhanden ist, was auch in den APT Repositories liegt und zwar gebaut mit den jeweilgen Commits. Wenn das größere Sachen in der Entwicklung sind, dann wird das in Feature Branches gemacht (die regelmäßig nicht hier auf Github liegen) und wird dann inkl. Version als ein Commit in den main Branch gemerged und das erst, wenn vollständig getestet wurde.

Durch die vielen inkludierten Merge Commits ist er auch nicht fähig für einen Merge, ein PR sollte im Regelfall nur einen Commit beinhalten bevor er gemerged wird.

Normalerweise kann der PR doch als squash commit gemergt werden und dann ist es nur noch ein einzelner Commit auf deinem master.

Grundsätzlich ja, aber ich kenne es so, dass das bereits vom Ersteller des PRs gemacht wird.

Aber auch inhaltlich macht eine Umsetzung per Conditionell Macro mehr Sinn, da hier einfach zu viel anderer Code zusätzlich in den Precompiler ifs drin ist.

Ich hab mal einen Gegenvorschlag mit conditional macros in einem Parallel-PR #500 gepusht (nur 1 Commit 😉 )

Wie ich geschrieben hatte, ich habe das bereits gestern in Code gegossen und bin einfach nur noch nicht mit dem Testing fertig, dein zweiter PR war daher unnötig.

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[dsiggi]

Hi alexreinert, wie ist den dein Status beim Test deines Codes? Ich nutze deine Module auf meinem Homeserver (amd64) und kann diese leider nicht mehr mit dem aktuellen Kernel kompilieren.