ADXL sensor drivers attempt to modify config struct at run-time

[ubieda]

Describe the bug
Drivers for ADXL345 and ADXL372 attempt to modify the config struct when setting ODR attribute:

• ADXL345:

  zephyr/drivers/sensor/adi/adxl345/adxl345.c

  Line 260 in 99a63a7

  cfg->odr = odr;

• ADXL372:

  zephyr/drivers/sensor/adi/adxl372/adxl372.c

  Line 578 in 99a63a7

  cfg->odr = odr;

Since config struct is constant, modifying it triggers a fault.

This is present since 4.0.0:

• ADXL345: drivers: sensor: adxl345: Updated ADXL345 driver with RTIO stream and Trigger functionality #80004
• ADXL372: drivers: sensor: adxl372: Updated driver with RTIO stream functionality #78941

To Reproduce

Build samples/drivers/sensor/accel_polling with either of these sensors, and it will crash, since this sample sets the ODR.

Expected behavior
Config struct is not modified at run-time.

[glenn-andrews]

Should ODR be in the data struct instead of config?

[ubieda]

Should ODR be in the data struct instead of config?

I hadn't seen any references in the file (get_attr() is not implemented), henced I had the PR removing it. I did take another look and I see it's referenced in process_fifo_samples_cb in Streaming mode. I'll add it back up in data. Thanks!