[linux-yocto] [PATCH 0/2] iio: Set correct iio name

Saul Wold sgw at linux.intel.com
Sat May 28 18:50:05 PDT 2016


On Sat, 2016-05-28 at 12:34 +0800, Yong Li wrote:
> Hi Bruce Saul,
> 
> I had submitted the patches into Kernel mail list, the concern is the
> legacy application compatibility:
> http://www.spinics.net/lists/linux-iio/index.html#24331
> 
> For the Ostro OS, Using Soletta framrwork, we have tested/verified
> more more than 30 different I2C
> devices(https://ostroproject.org/documentation/hardware/sensors.html)
> .
> But only the two devices have incorrect device names(the IIO name
> does
> not match the I2C device name). QA team think it is a bug
> 
So is it possible to fix the test in this case to correctly handle the
legacy naming rather than make it fail?  I guess the concern here is if
we "fix" the name, we will really break the legacy applications.  It's
possible for those application to run in Ostro also and since Ostro is
newer, it would break those rather than the other way around.

Sau!

> Thanks,
> Yong
> 
> 2016-05-27 23:51 GMT+08:00 Saul Wold <sgw at linux.intel.com>:
> > 
> > On Fri, 2016-05-27 at 10:24 -0400, Bruce Ashfield wrote:
> > > 
> > > On 2016-05-27 12:58 AM, Yong Li wrote:
> > > > 
> > > > 
> > > > Dear Maintainers,
> > > > 
> > > > This patch fixes the “incorrect IIO device name” issue.
> > > > 
> > > > Please merge it into standard/base branch for linux-yocto-4.4
> > > > if
> > > > this looks okay.
> > > The change looks technically correct, just a question about if
> > > these
> > > are also going upstream to the mainline kernel.
> > > 
> > Bruce,
> > 
> > These are possibly candidates for the standard/intel branch,
> > they where proposed upstream and deemed correct, but not merge-able 
> > due
> > to creating incompatible names.
> > 
> > Yong Li,
> > What is requiring the name change in Ostro, why can't Ostro
> > use the existing incorrect, but compatible name?
> > 
> > Sau!
> > > 
> > > Bruce
> > > 
> > > > 
> > > > 
> > > > 
> > > > Thanks
> > > > Yong Li
> > > > 
> > > > Yong Li (2):
> > > >    iio: tmp006: Set correct iio name
> > > >    iio: si7020: Set correct iio name
> > > > 
> > > >   drivers/iio/humidity/si7020.c    | 2 +-
> > > >   drivers/iio/temperature/tmp006.c | 2 +-
> > > >   2 files changed, 2 insertions(+), 2 deletions(-)
> > > > 


More information about the linux-yocto mailing list