[no subject]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



That should allow building of channel maps that provide the correct
device name to allow the provider to be found.

Jonathan

> 
> Thank you,
> Chris
> 
> > 
> > ChenYu
> >   
> > > Thank you,
> > > Chris
> > >  
> > > >
> > > > That's a lot lighter change than moving over to DT based phandles for all this.
> > > > (which is the modern way to handle it).
> > > >
> > > > As a cheeky check, just edit those maps to whatever IDs you have and see
> > > > if it works.  Probably not an upstreamable solution but will confirm we have
> > > > it correct.
> > > >
> > > > Your patch works because we allow for some fuzzy matching (I can't remember
> > > > why) that doesn't use the consumer device name.
> > > > That works as long as there is only one instance.  I'm guessing all this
> > > > mess came about because someone has a board with two of these devices. On such
> > > > a board we need the precise matching including the device name.
> > > >
> > > > Jonathan
> > > >  
> > > > >
> > > > > To fix this, I added a new devm_ helper of devm_iio_channel_get_sys()
> > > > > that directly calls iio_channel_get_sys(), and I updated all the
> > > > > affected drivers with the new routine. I then no longer experienced
> > > > > any issues with the drivers on my devices.
> > > > >
> > > > > Chris Morgan (2):
> > > > >   iio: core: Add devm_ API for iio_channel_get_sys
> > > > >   power: supply: axp20x: Use devm_iio_channel_get_sys() for iio chans
> > > > >
> > > > >  drivers/iio/inkern.c                    | 18 ++++++++++++++++++
> > > > >  drivers/power/supply/axp20x_ac_power.c  |  4 ++--
> > > > >  drivers/power/supply/axp20x_battery.c   | 16 ++++++++--------
> > > > >  drivers/power/supply/axp20x_usb_power.c |  6 +++---
> > > > >  include/linux/iio/consumer.h            | 20 ++++++++++++++++++++
> > > > >  5 files changed, 51 insertions(+), 13 deletions(-)
> > > > >  
> > > >  
> 






[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux