| mth | larsc: the battery driver uses its parent's (mfd) platform data as its own platform data | 02:11 |
|---|---|---|
| mth | however, that approach becomes problematic if more than one subdriver needs platform data | 02:11 |
| larsc | mth: yes | 07:13 |
| Ayla | morning larsc | 07:13 |
| larsc | hi Ayla | 07:14 |
| Ayla | I'm back from night club, I'm going to sleep now ;) | 07:14 |
| Ayla | ciao | 07:14 |
| larsc | hehe | 07:14 |
| larsc | mth: And to be honest I don't like that approach of using the parents platform_data anymore, because it does not allow you to reuse a mfd cell for different mfd devices | 07:16 |
| qi-bot | The build was successful: http://fidelio.qi-hardware.com/~xiangfu/build-nanonote/openwrt-xburst.full_system-20120916-0654 | 11:48 |
| LunaVorax | hi! | 12:25 |
| mth | larsc: is there a way though to provide platform_data directly to an mfd subdevice? | 13:23 |
| larsc | should be possible | 13:24 |
| mth | I wouldn't know where to attach it | 13:24 |
| larsc | well you'd have to split the platform data up in the mfd driver and pass it to the child devies | 13:24 |
| mth | what would be a clean way of doing that? as in, without having the mfd driver know which child devices exist | 13:25 |
| mth | I've got an analog joystick connected to the touch screen pins | 13:25 |
| mth | so that demonstrates that there can be more than one driver for the same cell | 13:26 |
| mth | (since there could be an ordinary touch screen driver as well) | 13:26 |
| larsc | yea that case it is a bit tricky | 13:26 |
| larsc | how does the analog joystick driver work? | 13:27 |
| mth | I'm not sure if driver selection should be done at configure time (one driver per cell for each board) or even at runtime (loading and unloading modules) | 13:27 |
| mth | the x and y axis both provide a voltage, which is read from the touch screen regs in the SADC | 13:27 |
| mth | the driver doesn't work yet, I'm still writing it ;) | 13:28 |
| mth | I think I'll just have to poll it at a fixed interval to start a read and when the read completes, inject the input events | 13:28 |
| larsc | I wrote a TS driver back than. Take a look at commit d21ad639b3d42a2124432fa1ea7e7ea2b38e0a8d | 13:29 |
| larsc | I think you can program the core to report new values when they change | 13:29 |
| larsc | generate an interrupt, when the values cahnge | 13:29 |
| mth | larsc: thanks; the probe looks very similar to what I have so far, the rest might be useful when adding the actual reading | 13:38 |
| mth | I'll add is_open to my driver as well, since it's useful for doing suspend properly | 13:39 |
| larsc | mth: Have you seen commit 95ceafd46 ("cpufreq: Add a generic cpufreq-cpu0 driver") | 14:51 |
| larsc | basically a cpufreq driver which uses the clk API | 14:52 |
| larsc | could replace the jz4740 cpufreq driver | 14:52 |
| Ayla | awesome | 14:53 |
| larsc | at least I think it could | 14:54 |
| mth | larsc: I haven't seen it yet | 15:46 |
| mth | replacing the driver would be good if possible | 15:46 |
| mth | the most problematic part though is that derived clocks have to be shut down during the PLL reconfiguration | 15:46 |
| Ayla | it's not a problem | 15:47 |
| mth | no? | 15:47 |
| Ayla | just move the code that reinits the PLL to the clock driver | 15:47 |
| mth | it's already there | 15:47 |
| Ayla | no, it's on the cpufreq driver | 15:47 |
| mth | it's not; the cpufreq driver calls the clock driver | 15:48 |
| mth | you can't just disable the other clocks at an arbitrary moment, you have to give the respective drivers a chance to stop their work at a point where that is possible | 15:48 |
| larsc | the current approach is to use the cpufreq notifiers, isn't it? | 16:04 |
| Ayla | readl / writel are deprecated? | 16:08 |
| mth | larsc: yes, but it's only implemented for mmc at the moment | 16:10 |
| larsc | Ayla: sort of, but nobody cares | 16:10 |
| Ayla | I'll just use iowrite32/ioread32 | 16:13 |
| Ayla | I don't know what header to include for readl/writel :) | 16:14 |
| larsc | io.h | 16:14 |
| --- Mon Sep 17 2012 | 00:00 | |
Generated by irclog2html.py 2.9.2 by Marius Gedminas - find it at mg.pov.lt!