Hi, Guenter Roeck <linux@xxxxxxxxxxxx> writes: > On Sun, Jun 23, 2013 at 05:39:32PM +0200, Simon Guinot wrote: >> On Thu, Jun 20, 2013 at 10:21:04PM +0200, Arnaud Ebalard wrote: >> > >> > GMT G762/763 fan speed PWM controller is connected directly to a fan >> > and performs closed-loop or open-loop control of the fan speed. Two >> > modes - PWM or DC - are supported by the chip. Introduced driver >> > provides various knobs to control the operations of the chip (via >> > sysfs interface). Specific characteristics of the system can be passed >> > either using board init code or via DT. Documentation for both the >> > driver and DT bindings are also provided. >> > >> > Signed-off-by: Arnaud Ebalard <arno@xxxxxxxxxxxx> >> > --- >> > Hi Guenter, >> > >> > I guess we can wait for Simon's tests against its 5Big Network to check >> > everything is ok on another platform. Note that I also tested the patch >> > on my (Armada 370 based) ReadyNAS 102 and it works as expected. >> > >> > To be very accurate, I had to revert eda6bee6c7 to get both an out of >> > tree driver for ISL 12057 chip and the g762 work *on the 102*, but this >> > is an unrelated story (Debian bug #622325 [1] has more on the topic) for >> > which I will create a separate thread. Simon, the symptom you reported >> > for your read failures are different from those I get so I don't think >> > the revert will fix your problem but it may be worth trying it. >> > >> > [1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=622325 >> >> Hi Arnaud and Guenter, >> >> Actually, reverting the commit eda6bee6c7 seems to solve the -ENODEV >> issue... >> > Did you let the i2c maintainers know ? As I wrote, I will create a separate thread on the topic. It is good to have another unrelated example for which the problem is solved by reverting this commit. I will keep both of you in the CC: list if you want to follow the discussion. >> Then, I have been able to test successfully the g762 driver against the >> following boards: >> >> - 2Big NAS (open-loop mode) >> - 2Big Network v2 (open-loop mode) >> - 5Big Network v2 (closed-loop mode) >> >> For the last, I have also been able to configure the clock frequency >> through the platform_data structure. >> >> Tested-by: Simon Guinot <simon.guinot@xxxxxxxxxxxx> Thanks for your time and efforts, Simon. Cheers, a+ -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
