Re: [PATCH v2 2/4] dt-bindings: maxbotix,i2cxl: Add MaxBotix i2c ultrasonic rangers

[Rob Herring]

On Sat, Mar 02, 2019 at 06:26:39PM +0000, Jonathan Cameron wrote:
> On Fri, 1 Mar 2019 14:42:27 +0100
> Andreas Klinger <ak@xxxxxxxxxxxxx> wrote:
> 
> > Add doc for dt binding maxbotix,i2cxl. This binding is for MaxBotix
> > I2CXL-MaxSonar ultrasonic rangers which share a common i2c interface.
> 
> I'm a bit confused on the naming. Certainly the binding should have
> separate entries for each supported part, rather than a group one
> for their i2cxl naming.  Having said that they do have a wide
> range of parts with only the one datasheet.
> 
> Rob, what do you think?  Go with this i2cxl naming
> or list the individual supported parts?

Looks like the difference is mainly just the range. Is that something 
s/w needs to know about?

> 
> mb1202, mb1212, mb1222, mb1232, mb1242, mb7040, 
> mb7137?

OTOH, that's not really too many.

Rob