diff options
Diffstat (limited to 'wiki/Chat_log/20171109-core-chatlog')
| -rw-r--r-- | wiki/Chat_log/20171109-core-chatlog | 199 |
1 files changed, 199 insertions, 0 deletions
diff --git a/wiki/Chat_log/20171109-core-chatlog b/wiki/Chat_log/20171109-core-chatlog new file mode 100644 index 0000000..4029f0a --- /dev/null +++ b/wiki/Chat_log/20171109-core-chatlog @@ -0,0 +1,199 @@ +Core-chat-meeting-2017-11-09 + +09:38 < geertu> Welcome to today's core group meeting! +09:38 < geertu> Agenda: +09:38 < geertu> 1. Status updates +09:38 < geertu> 2. Discussion Topics +09:38 < geertu> Topic 1. Status updates +09:39 < geertu> I'll just replay (a sometimes shortened version of) the update sent by email +09:39 < geertu> A) What have I done since last time +09:39 < geertu> Geert: +09:39 < geertu> - Sent 2nd pull requests for clk-renesas and sh-pfc, +09:39 < geertu> - Attendend ELCE and DT Workshop (Thursday), +09:39 < geertu> - Sent patches to enable SMP on r8a77970/eagle, +09:39 < geertu> - Sent patches for initial support for Salvator-XS with R-Car M3-W. +09:39 < geertu> - Looked a bit into initial support for Salvator-X with R-Car M3-N. +09:39 < geertu> - Fixing genpd for wake-up devices: +09:39 < geertu> - Sent active_wakeup cleanups, +09:39 < geertu> - Working on part two (interrupt routing), +09:39 < geertu> Jacopo: +09:39 < geertu> - Attended ELCE and multimedia meeting on 26/27 November +09:39 < geertu> Marek: +09:39 < geertu> - UBoot: Fixed eMMC data line voltage setting bug in U-Boot DT +09:39 < geertu> - UBoot: Final testing in preparation for the 2017.11 release +09:39 < geertu> Morimoto: +09:39 < geertu> - Posted DMAC TCR/TCRB patch +09:39 < geertu> Niklas: +09:39 < geertu> - [PATCH] net: ethtool: remove error check for legacy setting transceiver +09:39 < geertu> type +09:39 < geertu> - Received a V3M. +09:40 < geertu> Shimoda: +09:40 < geertu> - Prepared test environment for a new feature of M3-N's IPMMU (stage-2 +09:40 < geertu> format). +09:40 < geertu> - Handle our 3rd party (EPAM) requests for Xen. +09:40 < geertu> - Handle our kernel test team for testing v4.14. +09:40 < geertu> - Merged the following patch(es): +09:40 < geertu> - arm64: dts: renesas: salvator-common: add dr_mode property for USB2.0 +09:40 < geertu> channel 0 +09:40 < geertu> Simon: +09:40 < geertu> - Posted, changes requested: +09:40 < geertu> - Gen3 SoC IPMMU DTS patches +09:40 < geertu> - Posted, now under review: +09:40 < geertu> - [PATCH 0/2] iommu/ipmmu-vmsa: r8a779(70|95) support +09:40 < geertu> - Posted, now accepted: +09:40 < geertu> [PATCH] ARM: dts: koelsch: Move cec_clock to root node +09:40 < geertu> [pause to let people catch up] +09:41 < geertu> Any persone I missed? +09:43 < geertu> B) What I plan to do till next time +09:43 < geertu> Geert: +09:43 < geertu> - Finalize genpd for wake-up devices, +09:43 < geertu> - Initial support for Salvator-X with R-Car M3-N? +09:43 < geertu> Marek (U-Boot): +09:43 < geertu> - UBoot: Start converting Gen2 to DM/DT +09:43 < geertu> Morimoto: +09:43 < geertu> - Solve DMAC TCR/TCRB issue. +09:44 < geertu> Niklas: +09:44 < geertu> - Hook up V3M and make sure I can boot it. +09:44 < geertu> - Start to look at 'Add support for more than 32-bits IOVA space' +09:44 < geertu> Shimoda: +09:44 < geertu> - Add usb3.0 phy device node for r8a779[56].dtsi. +09:44 < geertu> Simon: +09:44 < geertu> - Address review of CPUFreq/Z,Z2 Clock patches +09:44 < geertu> - Address review of Gen3 SoC IPMMU DTS patches +09:44 < geertu> - Upport lots of BSP pinctrl patches +09:45 < geertu> My comment here is that several of the patches listed are alrwady upstream ;-) +09:45 < geertu> s/alrwady/already/ +09:45 < horms> geertu: ok, I thought I had pruned those. I'll prune again. +09:45 < horms> Unless kaneko-san posts them before I get that far +09:48 < geertu> C) Problems I have currently +09:48 < geertu> Morimoto: +09:48 < geertu> - I still don't understand DMAC TCR/TCRB issue on serial. +09:48 < geertu> I think problem is in rx_timer_fn(). +09:48 < geertu> According to Geert, "With your patch, the serial driver doesn't know +09:48 < geertu> about them (residue == 0), and the console doesn't see them." What does +09:48 < geertu> this mean? +09:48 < geertu> BTW, about commit e7327c09def48ccf ("serial: sh-sci: Pause DMA engine and +09:48 < geertu> get DMA status again"), I guess the reason why "transaction completes +09:48 < geertu> _after_ DMA stopped" is that it used TCR instead of TCRB? +09:49 < geertu> Perhaps I wasn't very clear in my initial bug report, but serial console input just doesn't work when SCIF DMA is enabled. +09:50 < morimoto> geertu: thank you for your report, we will check it again. +09:51 < wsa_> gotta go, cya guys! +09:51 -!- wsa_ [~wsa@p54B33A46.dip0.t-ipconnect.de] has quit [Quit: ...] +09:51 < horms> ciao +09:51 < geertu> horms: Goodbye! +09:51 < geertu> Oops, wsa_: Goodbye! +09:52 < geertu> morimoto: Checking the direction again may help. +09:52 < morimoto> Yeah, it seems we can guarantee data if we can check DE bit +09:53 < morimoto> geertu: can you agree ? +09:53 < geertu> morimoto: But DE is not the direction bit? +09:54 < morimoto> Yes. using TCR vs TCRB is checking direction +09:54 < morimoto> and guarantee is checking DE bit +09:54 < geertu> With "guarantee", you mean to be sure the DMA has completed? +09:54 < morimoto> yes +09:55 <@pinchartl> geertu: as far as I understand, the DE bit will be cleared the the hardware when the DMA engine finishes draining its buffers +09:55 < geertu> ok +09:55 < morimoto> pinchartl: yes +09:55 <@pinchartl> I'd like to have a clear explanation as to why SCIF doesn't work with the current patch though +09:56 <@pinchartl> I don't think we should merge anything based on guesswork +09:56 < morimoto> Current SCIF is maybe, DMAC read data from device +09:56 < morimoto> then, TCR was updated +09:56 < morimoto> but, DMAC will buffer it untile transferable +09:57 < morimoto> thus, this timing, TCRB is not updated +09:57 < morimoto> Because of this time gap, TCRB indicate 0x20 +09:57 < morimoto> I guess +09:58 < geertu> OK, it's worth a try. +09:58 <@pinchartl> that's the thing, I would feel better with a "I'm sure" than a "I guess" :-) +09:58 < morimoto> :) +09:58 < morimoto> So, geertu: pinchartl: can you agree using direction and DE bit check ? +09:59 < morimoto> using direction = TCR vs TCRB +09:59 < geertu> Yes, it's worth a try. It needs to be tested, though. +10:00 < morimoto> Thanks. +10:00 < morimoto> How about pinchartl ? +10:01 < morimoto> ... +10:01 < morimoto> BTW, on commit e7327c09def48ccfd204025726f11b57a19a9c24, it seems rcar dmac driver doesn't support dmaengine_pause() ? +10:02 < geertu> That's correct, AFAIK. +10:02 <@pinchartl> morimoto: I'm not sure yet, I need to analyze the problem in details. knowing how it works when checking the DE bit would help me understanding +10:03 < morimoto> pinchartl: OK, so I will create patch, and add [RFC]. can you check it ? +10:04 <@pinchartl> morimoto: can you run the patch and include the results ? :-) +10:04 < geertu> morimoto: Please check it doesn't break the serial console before posting ;-) +10:04 < morimoto> geertu: Hehe OK :) +10:04 < morimoto> pinchartl: OK, will try +10:04 < horms> geertu: is "[PATCH 2/3] soc: renesas: Identify R-Car M3-W ES1.1" good to merge? I think so after our discussion with shimida-san on periperi ML. +10:05 < geertu> Case closed 9for now)? +10:05 < morimoto> Sound and SCIF are only user of residue ? +10:05 < geertu> horms: I think it is. +10:05 < horms> thanks, I'll add it the next time I go through my queue +10:05 < geertu> Do you want me to update the patch description with future ES info? Or is that considered too confidential for now? +10:05 < horms> Lets keep it to ourselves +10:06 < geertu> OK, fine for me. +10:06 < geertu> s/9now/(now/ +10:06 < horms> I'll just mention that we had some discussion and it seems likely to work for current and future SoCs +10:06 < geertu> If the hardware people change their mind, we need to change the tests anyway +10:06 < horms> About "[PATCH v4 0/3] iommu/ipmmu-vmsa: r8a7796 support V4". I suppose this is a question for dammsan. But should I rebase/repost? +10:06 < horms> geertu: yes, exactly +10:07 < geertu> horms: dammsan said repost in a recent email +10:07 < horms> The important bit is that its unlikely there will be a clash +10:07 < geertu> Indeed. +10:07 < geertu> Any other topics for +10:07 < geertu> Topic 2. Discussion Topics +10:07 < geertu> ? +10:07 < horms> ok. sorry for missing that. I'll do so +10:07 < morimoto> dammsan is here, please wait +10:07 < horms> I will leave very soon, btw +10:08 < dammsan> horms: i think it is fine to wait with the repost +10:08 < dammsan> but feel free to post if it helps your integration work +10:09 < horms> dammsan: what is the motivation for wating. It effects my integration work in the sense that it adds new bindings. and is a dependency for bindings for other SoCs. +10:09 < dammsan> my gut feeling is that it is good to hold off addting code to avoid potential issueeeees +10:10 < horms> ok, perhaps just repost the bindings? +10:10 < dammsan> good idea +10:10 < horms> ok, will do +10:10 < dammsan> together with other ones too if you like +10:10 < dammsan> we have to revisit non-H3 support in the driver anyway +10:10 < horms> I'll repost all the outstanding bindings but not the driver code. Is that what you have in mind? +10:10 < dammsan> sure +10:10 < dammsan> thanks +10:10 < dammsan> sounds ogod +10:11 < horms> One last question, I take it that you intentionaly did not post a fallback binding +10:11 < dammsan> yeah, the IP varies with soc +10:11 < horms> thanks, got it +10:11 < dammsan> thanks +10:12 < horms> I need to go now, sorry for any inconvenience that causes +10:12 < geertu> I think we're finished with core anyway +10:12 < geertu> horms: Bye! +10:12 < horms> perfect +10:12 < horms> bye +10:12 -!- horms [~horms@61.40.109.130] has quit [Quit: Leaving] +10:13 <@pinchartl> one question for core +10:13 <@pinchartl> (and actually for I/O too, I should have asked earlier) +10:13 <@pinchartl> what's the status of V3M support ? +10:15 < geertu> pinchartl: Boots using serial console, NFS root should work +10:15 <@pinchartl> and while waiting for an answer to that question, let's see who's ready for the multimedia meeting +10:15 <@pinchartl> hi jmondi2 +10:15 <@pinchartl> hi kbingham[m] +10:15 <@pinchartl> hi dammsan +10:15 [Users #periperi] +10:15 [@pinchartl] [ jmondi2 ] [ Marex ] [ mturquette] +10:15 [ dammsan ] [ kbingham ] [ marex-cloud] [ neg ] +10:15 [ geertu ] [ kbingham[m]] [ morimoto ] [ shimoda ] +10:15 -!- Irssi: #periperi: Total of 12 nicks [1 ops, 0 halfops, 0 voices, 11 normal] +10:15 <@pinchartl> hi morimoto +10:15 < kbingham[m]> I'm here +10:15 < kbingham[m]> Morning +10:15 <@pinchartl> hi ul[tab] +10:15 <@pinchartl> no Ulrich today +10:15 <@pinchartl> hi neg +10:15 < jmondi2> I'm here too! +10:15 < neg> pinchartl: I'm not sure about the status, got a V3M yesterday and plan to try and take it for a first spin +10:16 < kbingham[m]> geertu: . Does i2c work? +10:16 <@pinchartl> geertu: any idea whether I2C is working ? how about PFC, is that still unmerged ? +10:16 < geertu> pinchartl: Sergey is working on upstreaming PFC +10:17 < geertu> Unfortunately he discovered recently he doesn't have the xls files describing pin mappings +10:17 <@pinchartl> -_-' +10:17 <@pinchartl> that's a bit of a blocker +10:18 < geertu> I told him to post the initial version anyway. We typically don't bother reviewing the register definitions, and we can review pin groups +10:19 <@pinchartl> thanks +10:19 <@pinchartl> so let's start with the multimedia meeting +10:19 <@pinchartl> welcome everybody +10:19 <@pinchartl> (I expect Magnus and Morimoto-san to come back soon) +10:19 <@pinchartl> first topic, status update +10:20 < geertu> Let's finish the core meeting: Thanks for joining, and have a nice continued day! |