1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
|
Core-chat-meeting-2016-03-15
2016-03-15 18:00:36 --> shimoda (~shimoda@relprex1.renesas.com) has joined #periperi
2016-03-15 18:00:36 -- Topic for #periperi is "marzen lock holder = none / managed in-channel rather than in-topic"
2016-03-15 18:00:36 -- Topic set by geertu on Thu, 01 Oct 2015 23:19:22
2016-03-15 18:00:36 -- Nicks #periperi: [dammsan geertu morimoto mturquette narmstrong_ neg pinchart1 shimoda uli___]
2016-03-15 18:00:36 -- Channel #periperi: 9 nicks (0 ops, 0 voices, 9 normals)
2016-03-15 18:00:39 -- Mode #periperi [+cnt]
2016-03-15 18:00:39 -- Channel created on Wed, 08 Jul 2015 19:41:24
2016-03-15 18:02:11 geertu Welcome to the Core Group Chat
2016-03-15 18:03:06 geertu Let's answer Magnus' question first
2016-03-15 18:03:09 geertu Magnus: What is current state and next step of the IPMMU + SYS-DMAC framework bits?
2016-03-15 18:03:31 geertu Vinod merged the patch to use phys_addr_t instead of dma_addr_t
2016-03-15 18:03:41 geertu So everything's on track again
2016-03-15 18:03:47 geertu No need to escalate
2016-03-15 18:04:04 neg sort of
2016-03-15 18:04:06 geertu dammsan: Does that answer your question?
2016-03-15 18:04:32 dammsan geertu: sort of, but i got the impression that more framework changes are needed...
2016-03-15 18:04:51 dammsan thanks for helping out to get that piece of the puzzle worked out though
2016-03-15 18:05:46 dammsan i think a first nice goal could be to get IPMMU + SYS-DMAC working on R-Car Gen2
2016-03-15 18:06:16 dammsan i'd love to kill off my dirty workaround patch =)
2016-03-15 18:06:41 neg Now the isse have hit its next roadblock where the idea seems to be something new/else is needed then the dmamapping api since the assumtion there is that it's backed by a physical page
2016-03-15 18:09:25 neg I'm talking with Christoph Hellwig but have yet to figure out a good way forward
2016-03-15 18:11:00 dammsan thanks
2016-03-15 18:12:31 --> horms (~horms@penelope-musen.kanocho.kobe.vergenet.net) has joined #periperi
2016-03-15 18:12:39 <-- horms (~horms@penelope-musen.kanocho.kobe.vergenet.net) has left #periperi
2016-03-15 18:12:39 neg I was hoping pinchart1 would review the series to put some weight behind it. He prommised he look at it last core meeting
2016-03-15 18:13:05 --> horms (~horms@penelope-musen.kanocho.kobe.vergenet.net) has joined #periperi
2016-03-15 18:13:10 geertu Welcome Simon
2016-03-15 18:13:17 horms sorry to be late (again!)
2016-03-15 18:13:57 horms The Gen2 documentation is too exciting for me to take my eyes off it!
2016-03-15 18:14:17 dammsan neg: so the case w/o IPMMU, we currently assume we do not need to map anything?
2016-03-15 18:14:58 dammsan i'm trying to understand what is so special about our situation =)
2016-03-15 18:15:34 geertu Yeah, the x86 and PPC guys have used IOMMUs for ages
2016-03-15 18:15:40 neg yes w/o a IPMMU the dma_addr_t is the same as phys_addr_t
2016-03-15 18:15:51 geertu x86 IO is always special, but PPC should be similar
2016-03-15 18:17:08 dammsan so we need to use the DMA API for this i guess
2016-03-15 18:17:23 dammsan and no one else is doing that in a similar way i wonder?
2016-03-15 18:17:47 dammsan stuck between DMA Engine and DMA API =)
2016-03-15 18:19:53 dammsan so pinchartl is not attending?
2016-03-15 18:20:03 geertu No, he's without network access today
2016-03-15 18:20:13 horms i believe he is at linaro connect
2016-03-15 18:20:24 geertu LC was last week, IIRC
2016-03-15 18:20:28 horms ok
2016-03-15 18:20:34 dammsan meh
2016-03-15 18:20:42 horms in that case hopefully is is not at lc (any more!)
2016-03-15 18:21:27 dammsan ok i guess we are blocked on him
2016-03-15 18:21:49 dammsan shall we move to next?
2016-03-15 18:21:51 neg not that I know of and there have been patches from Robin Murphy where he talk about toying with the same solution see http://article.gmane.org/gmane.linux.kernel.renesas-soc/404
2016-03-15 18:22:44 geertu Topic 1. Upcoming Gen3 development for the Core group
2016-03-15 18:24:02 geertu I posted a v3 of the PM Domain patch series last week, switching from a DT hierarchy to a C hierarchy again
2016-03-15 18:24:08 geertu s/again//
2016-03-15 18:24:11 geertu ;-)
2016-03-15 18:24:16 dammsan hehe
2016-03-15 18:24:35 horms shimoda-san: I'm wondering about PWM. Is it on your-list, done, or should I be looking at it?
2016-03-15 18:24:38 geertu There are some details to resolve w.r.t. SH4 power areas
2016-03-15 18:25:02 geertu After that I'll repost a v4.
2016-03-15 18:25:13 horms :)
2016-03-15 18:25:38 horms geertu: are you waiting on me for anything (e.g. to queue up any of your patches that are ready)?
2016-03-15 18:25:42 geertu I have a few more cleanups in the pipeline for SoC-specific PM code under arch/arm/mach-shmobile/, those depend on DT PM Domains
2016-03-15 18:25:47 geertu horms: Not yet
2016-03-15 18:26:05 horms ok
2016-03-15 18:26:23 geertu The code is in today's renesas-drivers, so it can be used by VSP users
2016-03-15 18:26:48 geertu (and it was in an integration topic branch last week)
2016-03-15 18:27:16 dammsan geertu: with the power domains, how do you work together with PSCI?
2016-03-15 18:27:40 dammsan just assume the firmware are handling the cpu power domains?
2016-03-15 18:27:52 geertu dammsan: The CPU power areas cannot be controlled from SYC for Gen2 and Gen3
2016-03-15 18:28:02 geertu That's done through APMU.
2016-03-15 18:28:08 geertu On Gen2 the C code does that
2016-03-15 18:28:15 geertu On Gen3 PSCI does that
2016-03-15 18:28:43 geertu However, the SYSC status bits don't indicate that the CPU power areas are powered down after CPU hot-unplug on Gen3
2016-03-15 18:29:03 horms Is that a bug or a feature?
2016-03-15 18:29:06 shimoda simon-san: sorry for the delayed response. about PWM integration for r8a7795, it seems ulrich-san sent upporting patches to the ML
2016-03-15 18:29:11 geertu Replugging the CPU doesn't work on Salvator-X, perhaps related to the above.
2016-03-15 18:29:42 horms shimoda-san: thanks, got it. I suppose we need to prod him a bit.
2016-03-15 18:30:09 dammsan geertu: how about the snoop controller, i guess on gen2 it needs manual setup
2016-03-15 18:30:17 geertu The link with controlling the Linux-view on the CPU PM Domains is not there. Lina Iyer is working on generic CPU PM Domain stuff
2016-03-15 18:30:18 horms geert: ok, so this is related to a thread with morimoto-san on the periperi ML?
2016-03-15 18:30:50 geertu dammsan: You mean the SCU power area?
2016-03-15 18:30:55 dammsan yeah
2016-03-15 18:31:07 dammsan psci is assumed to be in control i guess
2016-03-15 18:31:15 geertu dammsan: It's already running when booting Linux
2016-03-15 18:31:36 geertu and we never power down the first CPU core.
2016-03-15 18:31:51 dammsan good. on gen2 it depended on boot loader version and if it the cores were booted in big or little mode
2016-03-15 18:31:52 geertu H2 also doesn't have big.LITTLE enabled.
2016-03-15 18:32:13 geertu There are no changes to CPU bring-up introduced by my series.
2016-03-15 18:32:30 dammsan you csure, just curious about the assumptions
2016-03-15 18:32:31 geertu The later cleanups have, and I have it running locally with your old APMU DT series.
2016-03-15 18:32:40 dammsan ouch =)
2016-03-15 18:33:07 geertu horms: Yes, discussing SH4 on various SoCs with Morimoto-san
2016-03-15 18:33:48 dammsan so who is handling the non-working SMP CPU Hotplug bits?
2016-03-15 18:34:07 geertu I sent an email to periperi about it last week.
2016-03-15 18:34:08 dammsan hate to ask =)
2016-03-15 18:34:23 geertu Secondary core bringup works once
2016-03-15 18:34:37 geertu So after boot you have 4 x CA57
2016-03-15 18:34:51 geertu If you power down a Ca57, you loose it forever (until next reboot)
2016-03-15 18:35:17 geertu It seems India has a newer firmware?
2016-03-15 18:35:28 dammsan of course it is up to you guys how you want to roll
2016-03-15 18:35:29 geertu No idea if that helps. Firmware is black box to us ;-)
2016-03-15 18:35:40 dammsan i would not merge any code unless it is working myself
2016-03-15 18:36:07 horms is the code in question merged or not?
2016-03-15 18:36:10 geertu There is no new non-working SMP code added.
2016-03-15 18:36:38 dammsan is SMP including or excluding CPU Hotplug in your view? =)
2016-03-15 18:36:51 dammsan it my mind CPU Hotplug is the only sane test bed for SMP =)
2016-03-15 18:37:00 geertu Then it is broken.
2016-03-15 18:37:03 dammsan otherwise how would you test power down?
2016-03-15 18:37:13 dammsan ok
2016-03-15 18:37:17 geertu It broke when adding the secondary CPU cores to DT, with enable-method = "psci"
2016-03-15 18:37:19 dammsan so how can we fix this?
2016-03-15 18:37:29 geertu Fix PSCI?
2016-03-15 18:37:30 dammsan yes i assumed so
2016-03-15 18:37:42 dammsan well, we need to fix it somehow
2016-03-15 18:37:54 dammsan the rest of the organization is busy flippin burgers
2016-03-15 18:37:55 geertu Who's developing PSCI?
2016-03-15 18:38:18 dammsan i don't know to be sure
2016-03-15 18:38:21 horms we can remove the dt nodes pending a working psci implementation
2016-03-15 18:38:28 dammsan but i would check before i merged it if i had the chance
2016-03-15 18:38:39 dammsan that would be my preferred way forward
2016-03-15 18:38:44 dammsan not sure what you guys are thinking
2016-03-15 18:39:02 geertu Well, adding the CPU nodes to DT did allow to use them after booting.
2016-03-15 18:39:03 shimoda geertu: bootloader team develops the psci (arm-trasted-firmware) for gen3
2016-03-15 18:39:35 dammsan geertu: then we have same support level as emev2 =)
2016-03-15 18:39:36 shimoda it is a problem that i also don't know the detail of the psci firmware versioning
2016-03-15 18:40:00 dammsan i propose that we don't enable upstream until we have something working
2016-03-15 18:40:12 dammsan and put pressure on local people based on that
2016-03-15 18:40:26 geertu In this case, the "something" was 4 running CPU cores.
2016-03-15 18:40:27 dammsan i'm open to other suggestions though
2016-03-15 18:40:37 shimoda i heard end of this month, a new version of the firmware released on the git hub
2016-03-15 18:40:43 shimoda public git hub
2016-03-15 18:40:44 geertu Actually we had 8 running, but it broke by upgrading firmware.
2016-03-15 18:40:47 horms can i clarify if we are talking about 1) code that is already merged 2) code that posted but not merged or 3) code that is neither posted nor merged?
2016-03-15 18:41:00 geertu 1 = 4 x CA57
2016-03-15 18:41:05 geertu 2 = 8 x CA57
2016-03-15 18:41:10 geertu 3 = PSCI ;-)
2016-03-15 18:41:25 geertu s/8x CA57/4 x CA57 + 4 x CA53/
2016-03-15 18:41:54 dammsan you can see that the release procedure of binaries is working well
2016-03-15 18:42:04 dammsan at least we can generate some entropy
2016-03-15 18:42:14 dammsan =)
2016-03-15 18:42:21 dammsan so what shall we do?
2016-03-15 18:43:36 geertu In the kernel, we revert something if it causes regressions.
2016-03-15 18:43:54 shimoda I heard the current psci blocks CA53, however if we changes the boot mode we can use 8 core as the cpu0 as CA53 :)
2016-03-15 18:43:58 dammsan this did not cause a regression really, but CPU hotplug did not work
2016-03-15 18:43:59 geertu With firmware, it's more complicated
2016-03-15 18:44:39 dammsan i think on 32-bit arm it became possible to block cpu hotplug at some point i think
2016-03-15 18:44:47 horms does it cause a regression in the sense that reset no longer works?
2016-03-15 18:44:51 geertu dammsan: Actually I do not know if CPU hotplug worked with the previous firmware. I never tried unplug/replug
2016-03-15 18:45:06 dammsan same here
2016-03-15 18:45:19 dammsan but previous firmware version does not matter really
2016-03-15 18:45:24 dammsan it is too early to care
2016-03-15 18:45:27 dammsan imo
2016-03-15 18:45:29 geertu Reset doesn't work. We don't have reboot support.
2016-03-15 18:45:56 dammsan but ordering wise, getting SMP and CPU hotplug working before system reset is one option
2016-03-15 18:46:05 dammsan we can also change things around
2016-03-15 18:46:29 dammsan i like that we test stuff before we merge though
2016-03-15 18:47:00 horms ok, so perhaps its not a regression. but with the current firmware - which is all we support - SMP does not function as required because hotplug is broken. Correct?
2016-03-15 18:47:23 geertu Correct.
2016-03-15 18:47:33 horms If so it was an error (by me) to add the extra CPU nodes to DT. And I think we should revert the change until we have a good solution.
2016-03-15 18:47:43 geertu dammsan: It was tested by checking if we have 4 CPU cores running after bootup.
2016-03-15 18:48:03 dammsan right. next time please check with me that have been doing this for a while =)
2016-03-15 18:48:16 horms The main down side is that Dirk @ Bosch will not be happy. But the code is still around if he wants to use it locally.
2016-03-15 18:49:04 dammsan but he cant be the main motivator for us moving forward, can he?
2016-03-15 18:49:29 dammsan must be more important to feed back issues to renesas and get them to fix the PSCI firmware
2016-03-15 18:49:49 dammsan ideally satisfy both
2016-03-15 18:49:52 geertu What has the biggest use case: habing 4 CPU cores running? Or being able to unplug and replug CPU cores after boot up?
2016-03-15 18:50:00 horms He posted the patch. It was reviewed and accepted. Subsequently an issue came up. I think we should move on.
2016-03-15 18:50:20 geertu Indeed.
2016-03-15 18:51:27 dammsan i guess it depends on what our role is
2016-03-15 18:51:40 dammsan just being integrator and push issues different ways
2016-03-15 18:51:56 dammsan or try to tackle technical issues a bit more
2016-03-15 18:52:04 dammsan i prefer to focus on the technical bits
2016-03-15 18:52:10 dammsan but sure, lets move on
2016-03-15 18:52:54 horms sure, we could have done a better job on this one.
2016-03-15 18:53:41 dammsan no biggie, lets just fix it
2016-03-15 18:53:49 horms agreed
2016-03-15 18:55:08 dammsan shimoda: morimoto: can we report to renesas about PSCI failure somehow?
2016-03-15 18:55:26 dammsan and then we need to push hard
2016-03-15 18:55:38 shimoda dammsan: sure, i will ask bsp team or/and bootloader team about it
2016-03-15 18:55:38 dammsan we hinted a couple of times already
2016-03-15 18:55:44 dammsan thanks!
2016-03-15 18:56:21 dammsan imagine hypervisors and virtualization on top of this swiss cheese =)
2016-03-15 18:56:32 dammsan so it is good to push a bit more
2016-03-15 18:57:10 dammsan geertu: can you include boot loader version in the commit log for the power domain bits?
2016-03-15 18:57:27 geertu dammsan: Sure
2016-03-15 18:57:29 dammsan (you may have already)
2016-03-15 18:57:30 geertu v230
2016-03-15 18:57:30 dammsan thanks
2016-03-15 18:57:49 dammsan excellent
2016-03-15 18:59:02 dammsan horms: is it possible to get information from the bosch guy about his environment and if cpu hotplug is working for him?
2016-03-15 18:59:30 dammsan or maybe that is too detailed
2016-03-15 19:00:07 dammsan horms: sorry for being slow, but what was the plan forward with the SMP and CPU Hotplug? Revert or wait for new firmware version?
2016-03-15 19:00:29 dammsan im ok with anything
2016-03-15 19:00:36 dammsan we just need to make sure it becomes better
2016-03-15 19:00:57 horms dammsan: sure, i will try and get that information from Dirk
2016-03-15 19:01:09 geertu Making it better implies not reverting basic SMP support...
2016-03-15 19:01:12 horms my plan, which I am happy to change is as follows:
2016-03-15 19:01:22 dammsan geertu: that is fine with me
2016-03-15 19:01:26 dammsan as long as we can fix it
2016-03-15 19:01:32 dammsan _somehow_
2016-03-15 19:01:34 horms 1. post patch to revert the patch that added the extra cpu nodes to dt.
2016-03-15 19:01:51 horms 2. queue it up for v4.6 and possibly v4.5-stable after suitable discussion on the ML
2016-03-15 19:02:17 horms 3. discuss internally how a working psci implementation can be achieved
2016-03-15 19:02:34 dammsan seems like a lot of work to me
2016-03-15 19:02:53 horms how so?
2016-03-15 19:03:06 geertu What about skipping steps 1 and 2?
2016-03-15 19:03:13 dammsan like geert said, it does not really help anything for people just booting
2016-03-15 19:03:39 horms personally i am comfortable with skipping 1 and 2
2016-03-15 19:04:13 dammsan i dont think 1 and 2 helps us much at this point
2016-03-15 19:04:32 dammsan but i think despite that we need to agree about a plan how to improve things
2016-03-15 19:05:06 dammsan can we get some jinso members to test for us?
2016-03-15 19:05:58 horms fwiw i just tested it on my board
2016-03-15 19:06:09 horms which i suppose has the same fw revision as geert
2016-03-15 19:06:20 horms we can ask dirk if he has seen hotplug working
2016-03-15 19:06:30 horms and if we are lucky he may have a firmware version that works
2016-03-15 19:06:47 dammsan we can perhaps use dirk to put customer pressure at renesas =)
2016-03-15 19:06:53 horms which is information that might help the firmware developers to make it work in a future version
2016-03-15 19:06:58 horms right
2016-03-15 19:07:02 horms its clearly important to dirk
2016-03-15 19:07:17 horms and dirk's employer is important to renesas (I imagine)
2016-03-15 19:07:18 shimoda dammsan: of cause we can get some jinso members to test
2016-03-15 19:07:19 geertu Given he added the CA53 nodes too, he's probably still using the older firmware
2016-03-15 19:07:36 dammsan shimoda: it may be good to make a test case that can be reused on new SoCs
2016-03-15 19:08:20 horms fwiw, this is my test case so far:
2016-03-15 19:08:36 horms # echo 0 > /sys/bus/cpu/devices/cpu1/online
2016-03-15 19:08:37 horms [ 42.358400] CPU1: shutdown
2016-03-15 19:08:37 horms [ 42.359758] psci: CPU1 killed.
2016-03-15 19:08:37 horms # cat /sys/bus/cpu/devices/cpu1/online
2016-03-15 19:08:37 horms 0
2016-03-15 19:08:37 horms # echo 1 > /sys/bus/cpu/devices/cpu1/online
2016-03-15 19:08:39 horms [ 47.870239] CPU1: failed to come online
2016-03-15 19:08:41 horms -bash: echo: write error: Input/output error
2016-03-15 19:08:48 geertu Yep, that's it
2016-03-15 19:09:09 dammsan i used to run these in a tight loop to test some older platforms
2016-03-15 19:09:16 geertu Note that renesas-bsp/v4.2/rcar-3.0.x:arch/arm64/boot/dts/renesas/r8a7795.dtsi also has 8 CPU cores, with enable-method = "psci"
2016-03-15 19:09:44 dammsan "tick-box: SMP working"
2016-03-15 19:09:45 shimoda horms: thank you for the detail, i will ask bsp team or someone
2016-03-15 19:10:21 horms ok, so far we seem to have:
2016-03-15 19:10:26 horms * collect information from Dirk
2016-03-15 19:10:27 geertu FWIW, the same sequence was in my email "CPU hotplug on Salvator-X"
2016-03-15 19:10:33 horms * Get Jinso to test (their board?)
2016-03-15 19:10:45 horms * Talk with Renesas about how important this is
2016-03-15 19:11:49 shimoda geertu: oops, i missed your email...
2016-03-15 19:12:42 dammsan geertu: thanks for pushing
2016-03-15 19:12:53 dammsan horms: sounds good to me
2016-03-15 19:13:49 horms Ok, i can talk to Dirk unless someone else wishes to. Perhaps Shimoda-san could take the other two items for now?
2016-03-15 19:14:44 shimoda horms: yes, i will ask jinso and bsp team about 2 items
2016-03-15 19:15:01 horms excellent
2016-03-15 19:15:28 horms feel free to pass the Jinso guys on to me if you feel the need
2016-03-15 19:17:43 shimoda horms: thanks
2016-03-15 19:19:04 geertu Any other core things people are working on?
2016-03-15 19:19:23 dammsan Trying to get the CMT DT bits over the edge
2016-03-15 19:19:39 dammsan but thats about it
2016-03-15 19:19:59 horms I have made a little progress on the dmac/bus mastering/32-bit+ support analysis as requested by Magnus in Lueven
2016-03-15 19:20:30 horms Hopefully it will be complete enough to circulate something soon
2016-03-15 19:20:57 dammsan thanks
2016-03-15 19:21:00 neg going to post v2 of dmas pointing to all SYS-DMAC engines after this meeting
2016-03-15 19:21:19 horms in a nutshell so far things seem very similar in gen3 to that of gen2
2016-03-15 19:21:19 neg geertu: is it OK for me to add your Ack to the DT patch for i2c[6-8] on r8a7793 in this series? This patch was needed after rebasing on top of renesas-devel-20160315-v4.5 as you stated it would.
2016-03-15 19:21:58 geertu neg: if the patch is correct, yes ;-)
2016-03-15 19:22:26 geertu neg: Just email me the new hunk, and I'll have a look
2016-03-15 19:22:51 neg ok
2016-03-15 19:24:28 dammsan geertu: i'm also poking around with the gen3 ipmmu
2016-03-15 19:24:30 geertu dammsan: BTW, I wanted to include your v2 of ipmmu-multi-arch in today;s renesas-drivers, but r8a7795-ipmmu-rfc no longer applies
2016-03-15 19:24:45 dammsan geertu: thanks, yeah i know
2016-03-15 19:24:56 dammsan i will make a new version of the r8a7795 patches
2016-03-15 19:25:22 geertu dammsan: the CMT series didn't include the driver changes?
2016-03-15 19:25:24 dammsan you gave me some feedback about bitfields and features, anything you feel strongly about?
2016-03-15 19:25:32 dammsan geertu: that is correct
2016-03-15 19:25:39 geertu dammsan: I think that was Laurent.
2016-03-15 19:25:42 dammsan thought we would agree on DT bindigns first
2016-03-15 19:25:54 geertu dammsan: you can also use a plain int or long. and ffs() and ffz()
2016-03-15 19:26:39 geertu dammsan: OK, but you remove properties. Shouldn't that be done afer the driver supports the new stuff?
2016-03-15 19:26:41 dammsan geertu: unless laurent is using your email address it was you =)
2016-03-15 19:27:02 dammsan geertu: let me check
2016-03-15 19:28:02 dammsan geertu: i believe i only remove stuff that is unused
2016-03-15 19:28:21 geertu ok
2016-03-15 19:28:26 dammsan regarding the compat strings at least
2016-03-15 19:28:53 geertu Anyway, I'm still running with your previous version of the CMT series, so consider it tested
2016-03-15 19:30:07 geertu Topic 2. Status check for tasks from last meeting - what is remaining?
2016-03-15 19:30:27 dammsan geertu: thanks
2016-03-15 19:31:23 dammsan eventually i need to get to r8a7795 CMT
2016-03-15 19:31:46 dammsan not sure about other people
2016-03-15 19:32:42 geertu Any plan about SMP,v4.6,public,magnus,Discuss SMP DT bindings with ARM SoC maintainers
2016-03-15 19:33:04 dammsan not so much i'm afraid
2016-03-15 19:33:48 dammsan perhaps that is not a battle i can win
2016-03-15 19:34:35 shimoda no progress about "Investigate SYS-DMAC2" of my task, i'm afraid
2016-03-15 19:35:04 horms dammsan: in that case perhaps we should consider a fall back position?
2016-03-15 19:36:26 dammsan horms: sure
2016-03-15 19:37:01 geertu enable-method = "renesas,apmu" is working fine here (and on remote Lager)
2016-03-15 19:37:42 dammsan it is not so much a techincal problem, more that DT is used to describe software (PSCI) and overly verbose when not needed
2016-03-15 19:38:23 geertu That happens when you put critical functionality in firmware
2016-03-15 19:38:28 dammsan and we need to keep our existing support for some time instead of breaking DT ABI with non-enable method
2016-03-15 19:38:53 geertu Actually I have an idea to handle backwards-compatbility there.
2016-03-15 19:39:16 dammsan good, why dont you implement it?
2016-03-15 19:39:24 geertu All we have to do is match on r8a779x, add apmu device nodes, and enable-method properties, right?
2016-03-15 19:39:34 geertu dammsan: Time time time...
2016-03-15 19:39:46 dammsan yeah i know
2016-03-15 19:39:48 geertu it's on my list...
2016-03-15 19:40:03 dammsan in my mind we dont need the enable-method at all
2016-03-15 19:40:13 dammsan since we can use the apmu nodes by themselves
2016-03-15 19:40:29 geertu True.
2016-03-15 19:40:29 dammsan but that seems to the the "standard way" to do things
2016-03-15 19:40:35 geertu But the PSCI fans don't have PSCI nodes
2016-03-15 19:40:38 dammsan and thats what i wanted to discuss
2016-03-15 19:40:42 dammsan right
2016-03-15 19:40:50 geertu I'm afraid it's already too late for that
2016-03-15 19:41:02 dammsan regarding PSCI for sure
2016-03-15 19:41:15 dammsan but i wonder how it helps us to change the DT ABI?
2016-03-15 19:41:25 dammsan seems like no big win with the enable method
2016-03-15 19:41:31 dammsan i can see the point of the APMU bits
2016-03-15 19:42:12 dammsan anyway, it is possible to do something
2016-03-15 19:42:18 dammsan but feels like rather low priority IMO
2016-03-15 19:42:33 dammsan it is along there with the JTAG SMP support =)
2016-03-15 19:42:56 geertu That has a real use case ;-)
2016-03-15 19:43:02 dammsan also question is how to handle APMU on Gen3
2016-03-15 19:43:05 dammsan the hardware is there
2016-03-15 19:43:14 dammsan just not accessible perhaps
2016-03-15 19:44:10 geertu Hidden behind the PSCI curtain
2016-03-15 19:44:28 dammsan it is supposed to be hidden at least =)
2016-03-15 19:45:00 dammsan ParaSiteComputerInterface
2016-03-15 19:45:20 geertu Anything else to discuss (without a beer/wine/cocktail)?
2016-03-15 19:45:32 dammsan not from me
2016-03-15 19:46:56 shimoda not from me
2016-03-15 19:47:13 geertu ok
2016-03-15 19:47:17 geertu thx for joining!
2016-03-15 19:47:19 geertu bye!
2016-03-15 19:47:24 neg thanks bye
2016-03-15 19:47:34 shimoda thanks! bye!
2016-03-15 19:47:45 dammsan thanks
|