Add QMC5883P driver

[BortEngineerDude]

User description

This should close #10744
I made this driver looking into the datasheet, and I don't have the QMC5833P device, so testing is mandatory before merging.

---

PR Type

Enhancement

---

Description

This description is generated by an AI tool. It may have inaccuracies

• Add QMC5883P magnetometer driver support

• Implement axis transformation for QMC5883L compatibility

• Register new compass hardware in detection system

• Configure I2C bus and device initialization

---

Diagram Walkthrough

flowchart LR
  A["QMC5883P Driver"] --> B["Device Detection"]
  B --> C["I2C Communication"]
  C --> D["Data Reading"]
  D --> E["Axis Transformation"]
  E --> F["QMC5883L Compatibility"]

Loading

File Walkthrough

	Relevant files
Enhancement	5 files bus.h Add QMC5883P device hardware identifier                                    +1/-0      compass_qmc5883p.c Implement complete QMC5883P magnetometer driver                    +202/-0  compass_qmc5883p.h Add QMC5883P driver header file                                                    +27/-0    compass.c Integrate QMC5883P into compass detection system                  +14/-0    compass.h Add QMC5883P to magnetometer sensor enum                                  +1/-0
Configuration changes	4 files common_hardware.c Register QMC5883P I2C bus device configuration                      +7/-0      common_post.h Enable QMC5883P in magnetometer compilation flags                +1/-0      CMakeLists.txt Add QMC5883P source files to build                                              +2/-0      settings.yaml Add QMC5883P to magnetometer hardware settings                      +1/-1

---

[MATEKSYS]

compared QMC5883L and QMC5883P on M10Q-5883 board
Maybe we can set the alignment of QMC5883P to be the same as that of QMC5883L.
This way, users will not be confused when setting it up.

[qodo-merge-pro]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪

⚡ Recommended focus areas for review Typo in Constant Line 63 contains a typo in the constant name QMC5833P_CONF1_MODE_SINGLE which should be QMC5883P_CONF1_MODE_SINGLE to match the chip name pattern used throughout the file. #define QMC5833P_CONF1_MODE_SINGLE 0x02 #define QMC5883P_CONF1_MODE_CONTINUOUS 0x03 Typo in Constant Line 100 contains a typo in the constant name QMC5833P_STATUS_OVFL_MASK which should be QMC5883P_STATUS_OVFL_MASK to match the chip name pattern used throughout the file. #define QMC5833P_STATUS_OVFL_MASK 0x02 Missing Y-axis Reset In the qmc5883pRead function, only X and Z axes are reset to zero on line 132-133, but the Y-axis is not reset. This could leave stale data in magADCRaw[Y] if the read operation fails. mag->magADCRaw[X] = 0; mag->magADCRaw[Z] = 0;

[qodo-merge-pro]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
Possible issue	✅ Reset all magnetometer axes Suggestion Impact: The suggestion was directly implemented - the missing Y-axis reset line was added exactly as suggested code diff: + mag->magADCRaw[Y] = 0; The Y-axis is not being reset to zero in case of failed read, which could leave stale data. All three axes should be reset to maintain consistency and prevent potential issues with outdated magnetometer readings. src/main/drivers/compass/compass_qmc5883p.c [131-133] // set magData to zero for case of failed read mag->magADCRaw[X] = 0; +mag->magADCRaw[Y] = 0; mag->magADCRaw[Z] = 0; [Suggestion processed] Suggestion importance[1-10]: 8 __ Why: The suggestion correctly identifies a bug where only the X and Z axes are cleared, potentially leaving stale data in the Y axis upon a failed read, which could lead to incorrect sensor fusion.	Medium
Update

[MATEKSYS]

QMC5883P alignment is same as QMC5883L now.

[sensei-hacker]

It would be great if someone can test this.