diff options
author | Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> | 2019-12-23 14:27:52 +0900 |
---|---|---|
committer | Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> | 2019-12-23 14:27:52 +0900 |
commit | dc71f3518c95f8d9d306e8a4e53bc9bd2e9928e3 (patch) | |
tree | 54552f6ba6cec40e16cef5c22043d9f510087e00 /wiki/Chat_log/20160216-core-chatlog | |
parent | bb506a3f4c5441ecb212874077ad8b1bf335c936 (diff) | |
parent | 05040a728026b28ce7c6183d2adfa80218b306cb (diff) |
Merge remote-tracking branch 'gitlab/wiki' into HEAD
Diffstat (limited to 'wiki/Chat_log/20160216-core-chatlog')
-rw-r--r-- | wiki/Chat_log/20160216-core-chatlog | 195 |
1 files changed, 195 insertions, 0 deletions
diff --git a/wiki/Chat_log/20160216-core-chatlog b/wiki/Chat_log/20160216-core-chatlog new file mode 100644 index 0000000..2634bf2 --- /dev/null +++ b/wiki/Chat_log/20160216-core-chatlog @@ -0,0 +1,195 @@ +Core-chat-meeting-2016-02-16 + +10:02 < geertu> I think we are complete (Magnus is excuted by paperwork^H^H^H^H^H^H^H^H^H^H^H^H^H^H^H^Hsed) +10:03 < horms> can we add a topic about ravb to the agenda if there is time? +10:03 < geertu> Sure +10:03 < horms> thanks +10:03 < geertu> ravb or gpio? ;-) +10:04 < geertu> Topic 1. Upcoming Gen3 development for the Core group +10:04 < horms> either way :) +10:04 < geertu> Magnus posted his INTC-EX series, and reposted CMT +10:05 < geertu> I got sucked into SYSC PM Domains agai +10:05 < geertu> s/agai/again/ +10:05 < geertu> as this became a blocker for media +10:06 < pinchartl> sorry about that :-) +10:06 < pinchartl> your patch series seems to work fine +10:06 < pinchartl> I can read the VSP version registers +10:06 < geertu> Great! +10:06 < morimoto> Great ! +10:06 < pinchartl> haven't been able to test video processing yet though (and thus haven't tested the FCP) due to an unrelated problem in the media controller core +10:07 < pinchartl> (I'll refrain from blaming Mauro) +10:07 < geertu> So the PM domains are working. +10:07 < pinchartl> (ah, wait, I did, damn :-)) +10:07 < pinchartl> at least for the VSP, yes +10:07 < geertu> Before it was tested by (not) crashing when turning off CPU cores ;) +10:07 < pinchartl> how do they interact with runtime PM btw ? the vsp1 driver doesn't support runtime PM, I assume that will make the power domain always on ? +10:08 < geertu> Interesting, then the PM domain shold be turned off +10:08 < geertu> What does /sys/kernel/debug/pm_genpd/pm_genpd_summary say? +10:08 < pinchartl> I'll tell you in a minute +10:10 < shimoda> i have a question about core/todo. it has the following todo, but I forget it :) +10:10 < pinchartl> let's go on with the rest of the topics in the meantime +10:10 < shimoda> SMP,v4.6,public,shimoda,Discuss SMP DT bindings with ARM SoC maintainers +10:12 < geertu> On which mailing list was this discussed? +10:13 < geertu> Ah, this is for Magnus' "[PATCH v3 00/09] ARM: shmobile: APMU DT support via SMP Enable method V3"? +10:14 < pinchartl> (recompiling with CONFIG_PM_DEBUG enabled) +10:15 < geertu> CONFIG_PM_ADVANCED_DEBUG=y +10:16 < shimoda> geertu: sorry I don't know.. This topic was discussed at 2015-12-01, but I don't have the weechat log... +10:17 < pinchartl> (recompiling with CONFIG_PM_ADVANCED_DEBUG enabled) +10:18 < geertu> Can't find it in my 2015-12-01.log +10:19 < horms> I have a log +10:20 < pinchartl> https://paste.kde.org/pwe40pq96 +10:20 < pinchartl> the power domain is on +10:20 < geertu> Cool, so it's indeed on if Runtime PM is not enabled in a driver, great +10:21 < pinchartl> I'll enable runtime PM anyway +10:22 < geertu> OK, thx +10:25 < horms> I don't see any discussion of SMP in my log of the 2015-12-01 meeting. But there is some discussion in the 2015-12-15 meeting. +10:27 < geertu> That involves PSCI for arm64. +10:27 < geertu> The task was for Gen2 +10:28 < horms> ok +10:28 < shimoda> thanks for the confirm +10:28 < geertu> Originally it was targeted for v4.3. +10:31 < geertu> I guess it's not super-urgent +10:31 < geertu> shimoda: will you handle this? +10:33 < shimoda> geertu: sorry i don't know about the detail. so, if possible, someone handles it. +10:34 < geertu> It's about the bindings in "[PATCH v3 01/09] devicetree: bindings: Renesas APMU and SMP Enable method" +10:35 < geertu> Personally, I don't think there's much to discuss. +10:35 < geertu> Magnus just doesn't like the enable-method ;-) +10:37 < geertu> Any other Upcoming Gen3 development for the Core group +10:37 < geertu> ? +10:37 < shimoda> geertu: thanks for the information, I will ask Magnus about it later F2F :) +10:37 < horms> Was any progress made on the dmaengine issues we discussed in Holsbeek? +10:38 < horms> Or is that a media topic? +10:38 < geertu> OK, he'll be in the office tomorrow, IIRC +10:38 < geertu> Vinod said he will accept the phys_addr_t patch +10:38 < neg> Vinod move forward with dma slave config so I will push forward there +10:38 < geertu> Not yet in his -next, though +10:38 < pinchartl> it's still good news +10:39 < pinchartl> neg: are you going to reply to his e-mail ? +10:39 < geertu> Definitely +10:39 < neg> pinchartl: yes +10:40 < pinchartl> thanks +10:40 < horms> ok so the status is: maybe good soon ? +10:41 < horms> were we waiting on Vinod for other changes too? +10:41 < neg> I'd say it looks good now that Vinod have accepted the approch +10:41 < horms> ok +10:41 < horms> Lets try and keep things going now the ball seems to be rolling in the right direction :) +10:42 < neg> will post a new seris today and hope he picks up on it +10:42 < horms> great +10:43 < neg> horms: question, should I keep including the DT updates or have you queued them up? +10:43 < horms> I have not queued them up; please keep including them +10:43 < neg> ok +10:44 < horms> I'd like to hold off on accepting them until the bindings are accepted. I take it that is not yet the case. +10:45 < neg> ofc, I was unsure you told me you tentative queued them up :) +10:45 < horms> oh, ok. +10:45 < horms> what were the patch subjects? +10:46 < neg> [PATCH v3 7/8] ARM: dts: r8a7790: add iommus to dmac0 and dmac1 +10:46 < neg> and I'm sorry you wrote deferred not tentative my bad +10:46 < horms> ok, at least we agree :) +10:47 < neg> other than that I hade a first go at the DMAC errta, got Laurents dmatest to work on gen2 but it fails for all channels on gen3 need to do more work there +10:47 < pinchartl> all channels ? nice :-) +10:48 < neg> I guess it's someting else that mess stuff up ;) +10:51 < shimoda> for gen3 no SYS-DMAC + IPMMU, I think we have to use magnus' patches +10:51 < shimoda> it is already into topic branch? +10:52 < pinchartl> in which power domains are the iommus ? +10:52 < geertu> topic/ipmmu-multi-arch-v1 +10:52 < geertu> topic/r8a7795-ipmmu-rfc +10:54 < neg> IIRC I did my initial test based on renesas-drivers-2016-02-09-v4.5-rc9 +10:54 < neg> s/rc9/rc3 +10:55 < shimoda> geertu: thanks for the info. +10:55 < neg> but I did not try as much as I wanted so I would like another go +10:57 < pinchartl> next topic ? or has everybody fallen asleep ? +10:58 < geertu> Topic 2. Status check for tasks from last meeting - what is remaining? +10:58 < horms> lets move on +10:58 < geertu> earlycon is in +10:58 < geertu> irqc ra7795 is half-completed, according to Magnus +10:59 < horms> is the rfc half ready or is the whole thing half way to being merged? +10:59 < geertu> pfc improve documentation is queued for pull request +10:59 * geertu checks chat with Magnus +11:00 < geertu> Remaining INTC-EX issue is the unknown parent clock, which Morimoto-san and Shimoda-san will check? +11:01 < horms> ok, sounds simple enough +11:01 < geertu> r8a7795 SYS-DMAC integration is queued by Simon +11:02 < morimoto> geertu: I got same request from Magnus +11:02 < geertu> r8a7795 Cache topology is halfway +11:02 < morimoto> I will ask it to HW guy tomorrow +11:02 < geertu> morimoto: I know ;-) +11:02 < geertu> morimoto: Thx! +11:03 < geertu> Any other task status updates? +11:03 < horms> l2c patches? +11:04 < horms> fwiw, they look good to me and i plan to merge them +11:04 < geertu> == r8a7795 (+ other :-) Cache topology +11:04 < geertu> thx +11:05 < horms> yes. not stricktly on topic +11:05 < shimoda> how about PFC,v4.6,plan,?,PFC 1.8v switching +11:05 < shimoda> ? +11:06 < geertu> That's not assigned yet. +11:07 < geertu> But if I'm not mistaken, Wolfram was going to look into it for his SDHI work? +11:08 < shimoda> i see, so I will ask Wolfram-san on next I/O meeting +11:10 < geertu> Simon: do you plan to queue the fixes for writing to .text for v4.5? +11:10 < horms> Yes +11:11 < geertu> OK, thx +11:11 < horms> I thought I did so this morning +11:11 < geertu> I think it's queued for v4.6 +11:11 < horms> oh ok +11:11 < horms> yes, that would be right +11:11 < horms> are they fixes for v4.5? +11:12 < geertu> Not having them will cause a crash on suspend as soon as Linus will merge rmk's branch for v4.6. +11:12 < geertu> Which may happen before arm-soc is merged. +11:12 < geertu> So I think it's safer to fast track them to v4.5. +11:13 < horms> ok, it seems that it wouldn't hurt to ask the ARM SoC maintainers +11:13 < horms> it would help me to make my case if i knew which patch(es) in RMK's branch you are referring to. +11:13 < geertu> BTW, you can already trigger the crash if you enable CONFIG_DEBUG_RODATA=y now +11:14 < geertu> That was in the cover letter +11:14 < horms> of course :) +11:14 < horms> i'll see about sending them to v4.5 +11:14 < geertu> 25362dc496edaf17f714c0fecd8b3eb79670207b +11:14 < geertu> ARM: 8501/1: mm: flip priority of CONFIG_DEBUG_RODATA +11:15 < horms> I sent myself an email about it to read tomorrow morning +11:16 < geertu> OK, thanks +11:16 < geertu> Topic 3. RAVB / GPIO +11:17 < geertu> Waiting up to 110 more seconds for network. +11:17 < geertu> Seen in v4.5-rc3 and later +11:17 < pinchartl> could it be related to the nfsroot issues I'm seeing ? +11:18 < pinchartl> [ 65.247123] nfs: server 192.168.1.47 not responding, still trying +11:18 < pinchartl> [ 65.261947] nfs: server 192.168.1.47 OK +11:18 < horms> probably not +11:18 < horms> or at least that is a different manifestation +11:18 < geertu> Last week's renesas-drivers already had the RFC fix for rcar-gpio +11:18 < horms> i don't even get an address via DHCP +11:18 < geertu> The PHY interrupt doesn't come through because the rcar-gpio instance is suspended +11:19 < horms> right. so your patches address this. but I wonder if they have any chance of making it into v4.5. +11:19 < horms> If not I supose we have to live with broken ethernet +11:19 < geertu> Before v4.5-rc3, the PHY driver accidentally kept on polling. When that was fixed, it broke ravb on salvator-x +11:20 < geertu> Probably not a chance. +11:20 < horms> which i guess is better than living with no cpg as we did in v4.4 +11:20 < geertu> ;-) +11:20 < geertu> Does it work if you comment out the phy interrupt in the dts? +11:20 < horms> Ok, at least I'll have something to talk about in Tokyo next week +11:20 < geertu> (haven't tried myself, busy fixing bugs in -next) +11:20 < horms> I can try +11:23 < geertu> Lemme try myself +11:23 < horms> fwiw i'm trying +11:24 < geertu> works! +11:24 < horms> excellent +11:24 < horms> i guess we have found our work-around +11:24 < geertu> Yeah, DT describes the hardware ;-) +11:25 < geertu> Shall we add a quirk that removes the interrupt if present? +11:25 < geertu> Would be a good exercise to prepare for ESx.y DT fixups +11:25 < pinchartl> geertu: shall we fix the root cause ? :-) +11:26 < horms> i think we know the root cause +11:26 < geertu> In the Linux kernel, we never fix root causes ;-) +11:26 < horms> and i agree any work around should go into .c not dt +11:28 < geertu> We can get a workaround in v4.5-rc7 if needed, can't we? +11:28 < horms> we can try +11:30 < geertu> Should we stick a pm_runtime_get_sync() in rcar_gpio_probe() instead? +11:30 < geertu> That would help HDMI, too +11:31 < horms> its a posibility +11:31 < horms> do we expect HDMI to be effected: i.e. for it go go into mainline before the root cause is fixed? +11:32 < geertu> HDMI on Koelsch has been broken due to this for a while. +11:33 < horms> ok, then it does seem like a good idea from my armchair point of view +11:34 < geertu> OK, will go that way for today's renesas-drivers +11:34 < horms> great +11:34 < horms> Hopefully Florian reads your email before he spends any time on the problem +11:35 < geertu> ;-) +11:37 < pinchartl> anything else for today or can I have lunch ? +11:37 < geertu> I think we're finished +11:37 < geertu> Thanks for joining! |