Hi Laurentiu, Thank you for replying. On 11/09/15 09:57, Laurentiu Palcu wrote: > Hi Violeta, > > On Thu, Sep 10, 2015 at 04:02:05PM +0100, Violeta Menendez Gonzalez wrote: >> Hi, >> >> I've been taking a look at the accelerometer drivers meaning to add some >> features for the chip BMA250. > Just a heads-up, BMA250 is not recommended for new designs: > https://www.bosch-sensortec.com/en/homepage/products_3/3_axis_sensors/acceleration_sensors/acceleration_sensors_1 > Yes, I'm aware, thank you. Not trying to do new designs, just porting some features from an out-of-tree driver. >> >> I've seen that this chip is supported by the bma180 driver [1], and that >> there's another driver for bmc150 [2] that supports BMA250E. Looking at >> the datasheets [3][4][5] I can't really understand why it is that way, >> as BMA250 and BMA250E register map looks exactly the same except for the >> FIFO memory, and BMA250 doesn't look similar to BMA180 to me. >> >> After some research on the internet and talking to Ben Dooks (who is >> guiding me through this) we couldn't arrive to a conclusion as to why >> this is this way. Could anyone clarify this a bit more? Could it make >> sense to move support of BMA250 to bmc180 driver? > BMA250 and BMA250E are slightly different, register wise. However, the E > version is completely register compatible with BMC150, except the > resolution. So, adding support to it to existing bmc150_accel driver was > the logical thing to do. > Yes, I agree with that. All the chips in the bmc150_accel driver are very similar. But my idea was that the BMA250 is also very similar (register compatible) to them (except FIFO), so I wonder why the support for the BMA250 was added to bmc180 instead of adding it directly to the bmc150_accel driver. > I'm not sure it's worth the trouble to change bmc150_accel driver to > support BMA250 too. I would recommend to add the features you need to > the existing bmc180 driver which already supports it. > I thought it would be easier for me to add features to a driver where the chips are more homogeneous. BMA250 and BMC180 don't look very register compatible to me, so I thought it may be worth just making the effort of changing the support first and then trying to add the features. I'm pretty inexperienced on this, that's why I'm asking for opinion on it, I may be missing something. Talking to Ben Dooks he agreed with me, and suggested that it may be worth adding support for the BMA250 to bmc150_accel for now and then later on removing it from bma180. I may have misinterpreted something, but unfortunately he's on holidays now. He may be able to give some insight on this next week. > But, Peter/Jonathan will most likely show you the right path to follow! > :) > > laurentiu > Thank you :-) -- Violeta Menéndez González http://www.codethink.co.uk/ Software Engineer Codethink - Providing Genius -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
