All of lore.kernel.org
 help / color / mirror / Atom feed
* Duplicate htu21 drivers
@ 2015-10-28 10:03 ` Michal Marek
  0 siblings, 0 replies; 6+ messages in thread
From: Michal Marek @ 2015-10-28 10:03 UTC (permalink / raw)
  To: William Markezana, Jean Delvare, Guenter Roeck, Ludovic Tancerel,
	Jonathan Cameron
  Cc: lm-sensors, linux-iio

Hi,

commit 2b5c53d2c958 ("Add htu21 meas-spec driver support") introduced a
new implementation of the htu21 driver into drivers/iio/humidity. At
first glance, it provides the same features as drivers/hwmon/htu21 with
a different sysfs interface. What is the plan with these two drivers?
Will the iio driver replace the hwmon one? At a minimum, they should
conflict in kconfig, because module names must be unique.

Thanks,
Michal

^ permalink raw reply	[flat|nested] 6+ messages in thread
* [lm-sensors] Duplicate htu21 drivers
@ 2015-10-28 10:03 ` Michal Marek
  0 siblings, 0 replies; 6+ messages in thread
From: Michal Marek @ 2015-10-28 10:03 UTC (permalink / raw)
  To: William Markezana, Jean Delvare, Guenter Roeck, Ludovic Tancerel,
	Jonathan Cameron
  Cc: lm-sensors, linux-iio

Hi,

commit 2b5c53d2c958 ("Add htu21 meas-spec driver support") introduced a
new implementation of the htu21 driver into drivers/iio/humidity. At
first glance, it provides the same features as drivers/hwmon/htu21 with
a different sysfs interface. What is the plan with these two drivers?
Will the iio driver replace the hwmon one? At a minimum, they should
conflict in kconfig, because module names must be unique.

Thanks,
Michal

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: Duplicate htu21 drivers
       [not found] ` <20151028105128.770BF1E93@relay1.suse.de>
@ 2015-10-28 11:15     ` Michal Marek
  0 siblings, 0 replies; 6+ messages in thread
From: Michal Marek @ 2015-10-28 11:15 UTC (permalink / raw)
  To: ludovic.tancerel@maplehightech.com
  Cc: William Markezana, Jean Delvare, Guenter Roeck, Jonathan Cameron,
	lm-sensors, linux-iio

Dne 28.10.2015 v 11:52 ludovic.tancerel@maplehightech.com napsal(a):
> Hello Michal,
> 
> the plan discussed with Jonathan and Guenter was to remove the driver
> in hwmon once the iio one was introduced.
> 
> Should I  provide a patch in Kconfig to make them conflict as of now
> ?

Then you can either execute your plan :), or make the hwmon driver
depend on HTU21=n.

Michal

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [lm-sensors] Duplicate htu21 drivers
@ 2015-10-28 11:15     ` Michal Marek
  0 siblings, 0 replies; 6+ messages in thread
From: Michal Marek @ 2015-10-28 11:15 UTC (permalink / raw)
  To: ludovic.tancerel@maplehightech.com
  Cc: William Markezana, Jean Delvare, Guenter Roeck, Jonathan Cameron,
	lm-sensors, linux-iio

Dne 28.10.2015 v 11:52 ludovic.tancerel@maplehightech.com napsal(a):
> Hello Michal,
> 
> the plan discussed with Jonathan and Guenter was to remove the driver
> in hwmon once the iio one was introduced.
> 
> Should I  provide a patch in Kconfig to make them conflict as of now
> ?

Then you can either execute your plan :), or make the hwmon driver
depend on HTU21=n.

Michal

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: Duplicate htu21 drivers
  2015-10-28 11:15     ` [lm-sensors] " Michal Marek
@ 2015-10-28 12:15       ` Guenter Roeck
  -1 siblings, 0 replies; 6+ messages in thread
From: Guenter Roeck @ 2015-10-28 12:15 UTC (permalink / raw)
  To: ludovic.tancerel@maplehightech.com
  Cc: Michal Marek, William Markezana, Jean Delvare, Jonathan Cameron,
	lm-sensors, linux-iio

On Wed, Oct 28, 2015 at 12:53:12PM +0100, ludovic.tancerel@maplehightech.com wrote:
> I was not sure about the timing for the plan … :)
> Now that the patch has been approved for IIO, then indeed I can remove the patch for htu21 in hwmon.

Yes, the driver in hwmon needs to be removed. Patch welcome.

Guenter

> Thanks,
> Ludovic
> 
> 
> 
> > Le 28 oct. 2015 à 12:15, Michal Marek <mmarek@suse.cz> a écrit :
> > 
> > Dne 28.10.2015 v 11:52 ludovic.tancerel@maplehightech.com napsal(a):
> >> Hello Michal,
> >> 
> >> the plan discussed with Jonathan and Guenter was to remove the driver
> >> in hwmon once the iio one was introduced.
> >> 
> >> Should I  provide a patch in Kconfig to make them conflict as of now
> >> ?
> > 
> > Then you can either execute your plan :), or make the hwmon driver
> > depend on HTU21=n.
> > 
> > Michal
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [lm-sensors] Duplicate htu21 drivers
@ 2015-10-28 12:15       ` Guenter Roeck
  0 siblings, 0 replies; 6+ messages in thread
From: Guenter Roeck @ 2015-10-28 12:15 UTC (permalink / raw)
  To: ludovic.tancerel@maplehightech.com
  Cc: Michal Marek, William Markezana, Jean Delvare, Jonathan Cameron,
	lm-sensors, linux-iio

T24gV2VkLCBPY3QgMjgsIDIwMTUgYXQgMTI6NTM6MTJQTSArMDEwMCwgbHVkb3ZpYy50YW5jZXJl
bEBtYXBsZWhpZ2h0ZWNoLmNvbSB3cm90ZToKPiBJIHdhcyBub3Qgc3VyZSBhYm91dCB0aGUgdGlt
aW5nIGZvciB0aGUgcGxhbiDigKYgOikKPiBOb3cgdGhhdCB0aGUgcGF0Y2ggaGFzIGJlZW4gYXBw
cm92ZWQgZm9yIElJTywgdGhlbiBpbmRlZWQgSSBjYW4gcmVtb3ZlIHRoZSBwYXRjaCBmb3IgaHR1
MjEgaW4gaHdtb24uCgpZZXMsIHRoZSBkcml2ZXIgaW4gaHdtb24gbmVlZHMgdG8gYmUgcmVtb3Zl
ZC4gUGF0Y2ggd2VsY29tZS4KCkd1ZW50ZXIKCj4gVGhhbmtzLAo+IEx1ZG92aWMKPiAKPiAKPiAK
PiA+IExlIDI4IG9jdC4gMjAxNSDDoCAxMjoxNSwgTWljaGFsIE1hcmVrIDxtbWFyZWtAc3VzZS5j
ej4gYSDDqWNyaXQgOgo+ID4gCj4gPiBEbmUgMjguMTAuMjAxNSB2IDExOjUyIGx1ZG92aWMudGFu
Y2VyZWxAbWFwbGVoaWdodGVjaC5jb20gbmFwc2FsKGEpOgo+ID4+IEhlbGxvIE1pY2hhbCwKPiA+
PiAKPiA+PiB0aGUgcGxhbiBkaXNjdXNzZWQgd2l0aCBKb25hdGhhbiBhbmQgR3VlbnRlciB3YXMg
dG8gcmVtb3ZlIHRoZSBkcml2ZXIKPiA+PiBpbiBod21vbiBvbmNlIHRoZSBpaW8gb25lIHdhcyBp
bnRyb2R1Y2VkLgo+ID4+IAo+ID4+IFNob3VsZCBJICBwcm92aWRlIGEgcGF0Y2ggaW4gS2NvbmZp
ZyB0byBtYWtlIHRoZW0gY29uZmxpY3QgYXMgb2Ygbm93Cj4gPj4gPwo+ID4gCj4gPiBUaGVuIHlv
dSBjYW4gZWl0aGVyIGV4ZWN1dGUgeW91ciBwbGFuIDopLCBvciBtYWtlIHRoZSBod21vbiBkcml2
ZXIKPiA+IGRlcGVuZCBvbiBIVFUyMT1uLgo+ID4gCj4gPiBNaWNoYWwKPiA+IC0tCj4gPiBUbyB1
bnN1YnNjcmliZSBmcm9tIHRoaXMgbGlzdDogc2VuZCB0aGUgbGluZSAidW5zdWJzY3JpYmUgbGlu
dXgtaWlvIiBpbgo+ID4gdGhlIGJvZHkgb2YgYSBtZXNzYWdlIHRvIG1ham9yZG9tb0B2Z2VyLmtl
cm5lbC5vcmcKPiA+IE1vcmUgbWFqb3Jkb21vIGluZm8gYXQgIGh0dHA6Ly92Z2VyLmtlcm5lbC5v
cmcvbWFqb3Jkb21vLWluZm8uaHRtbAo+IAoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f
X19fX19fX19fX19fX19fX18KbG0tc2Vuc29ycyBtYWlsaW5nIGxpc3QKbG0tc2Vuc29yc0BsbS1z
ZW5zb3JzLm9yZwpodHRwOi8vbGlzdHMubG0tc2Vuc29ycy5vcmcvbWFpbG1hbi9saXN0aW5mby9s
bS1zZW5zb3Jz

^ permalink raw reply	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-10-28 12:15 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-28 10:03 Duplicate htu21 drivers Michal Marek
2015-10-28 10:03 ` [lm-sensors] " Michal Marek
     [not found] ` <20151028105128.770BF1E93@relay1.suse.de>
2015-10-28 11:15   ` Michal Marek
2015-10-28 11:15     ` [lm-sensors] " Michal Marek
2015-10-28 12:15     ` Guenter Roeck
2015-10-28 12:15       ` [lm-sensors] " Guenter Roeck

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.