summaryrefslogtreecommitdiff
path: root/wiki/Chat_log/20170817-io-chatlog
blob: 67df7253082b9f66c704ed355afb2f82f21fddb1 (plain)
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
10:53 < wsa_> hey guys, welcome to the core meeting
10:53 < uli___> wot?
10:53 < geertu> again?
10:53 < wsa_> ups
10:53 < wsa_> io
10:53 < wsa_> sorry, geert
10:53  * wsa_ notes: copy&pasting meetings doesn't work
10:54 < wsa_> let me summarize the status updates:
10:54 < wsa_> simon got the Gen3 enablement patch merged
10:54 < wsa_> uli created the HW timeout config patch for HSCIF but it doesn't work
10:55 < wsa_> wolfram did various things with SDHI and rewrote the I2C core DMA helpers
10:55 < wsa_> please add something if I missed it
10:55 < wsa_> otherwise we have the issues of the non-working HSCIF register from Uli
10:56 < geertu> uli___: Have you checked out the R-Car E2X errata you received recently?
10:56 < geertu> They contain more info about registers that cannot be accessed at any time.
10:56 < wsa_> I'll try to play around today with that, but I'd think we need Morimoto-san on this?
10:56 < uli___> geertu: i'll check it out
10:56 -!- neg [~neg@unaffiliated/neg] has joined #periperi
10:57 < pinchartl> geertu: doesn't sound like very useful registers if they can't be accessed at any time
10:57 < geertu> pinchartl: From the POV of the SW engineer?
10:57 < wsa_> Gen3 docs say "HSSCR can always be read from and written to by the CPU."
10:57 < pinchartl> from any point of view :-)
10:58 < pinchartl> after the read-only register, the write-only register, we have the don't-access-only register
11:00 < wsa_> so, the other issue is SDR104 on Gen3
11:00 < geertu> on H3 ES2.0? ;-)
11:00 < wsa_> (if there are not more ideas on the HSSCR register)
11:00 < wsa_> enabling ES2.0 is also somewhere on my todo-list
11:01 < wsa_> one of the tasks I'd really like to give away, but in my book, this is a base-task, not an additional task
11:02 < wsa_> this is why I'd like to discuss the status-quo in SanSeb
11:02 < wsa_> so, SDR104 memory cards now work fine on H3 ES1.* and M3-W 1.0
11:03 < wsa_> it is the SDIO card which has problems in some slots
11:03 < wsa_> at high speeds
11:03 < wsa_> I think the fact that the line length is ~23cm instead of 10cm might have an influence
11:04 < wsa_> hard to test
11:04 < wsa_> resoldering in SanSeb was also kind of shot down :)
11:05 < wsa_> I wonder where is the line of not enabling-SDR104 because of some setup
11:05 < wsa_> I don't want to enable it now
11:05 < wsa_> ES2.0 testing should also happen and way more testing in SanSeb
11:05 < wsa_> but still
11:05 < pinchartl> wsa_: desoldering such connectors is pretty difficult. a soldering iron isn't the best tool for the job
11:05 < geertu> Can't the driver/subsystem downgrade the feature set if errors are detected?
11:07 < wsa_> on the mmc core layer, could be
11:07 < wsa_> dunno if a driver can request that if e.g. loading firmware fails
11:08 < wsa_> i'd guess if the core notices a problem it will schedule a retune
11:08 < geertu> And if retune fails?
11:09 < wsa_> i expect it to go down with the speed, but i haven't checked
11:10 < wsa_> so far, our tuning only failed because of stalled HW
11:10 < wsa_> but we fixed that
11:12 < wsa_> well, looks like this is a question for SanSeb, too; when we also have more data
11:13 < wsa_> i think that's it for io
11:13 < wsa_> unless you have something to add?
11:13 < geertu> I have one more comment
11:13 < wsa_> sure
11:13 < geertu> About shifting the UART sampling point: This can increase accuracy for serial
11:13 < geertu> speeds that are too much off, cfr. 92a0574867f3329c ("serial: sh-sci: Add
11:13 < geertu> support for SCIFA/SCIFB variable sampling rates").
11:14 < geertu> If the actual speed differs more than a few % from the requested speed, it fails.
11:14 < geertu> Changing the sampling point can fix that.
11:16 < uli___> i'm looking into quantifying that effect in practice
11:16 < uli___> my current idea is to plop a 57.6 kHz square wave into an uart
11:16 < uli___> and then tweak the frequency until the received pattern is not repetitive any more
11:17 < uli___> and then check if pushing the sampling point can improve the margin
11:17 < uli___> dunno if that works, i'll see once i get my equipment
11:18 < geertu> I had a simpler test method when doing the variable sampling rates
11:18 < geertu> Just configure a speed that cannot be done exactly
11:18 < geertu> and see how it receives garbage.
11:19 < uli___> don't you need two ends for that, with differing imprecisions?
11:19 < geertu> USB serial adapters (e.g. FTDI) can usually do all standard rates.
11:20 < geertu> Renesas SCIF ports always can't, with the supplied clocks
11:20 < geertu> E.g. APE6EVM couldn't do 460800 bps before the aforementioned patch
11:22 < geertu> I think SCIFA on Gen2 still can't do 1500000 bps
11:22 < geertu> Perhaps also not 460800
11:23 < uli___> frankly, i'm atm more concerned if that functionality in the hscif actually works at all...
11:24 < wsa_> i understand that
11:24 < wsa_> same here
11:24 < geertu> It's not that difficult to find out.
11:25 < geertu> Try all standard rates up to 4 Mbps, and see which fails.
11:25 < wsa_> if we get feedback from renesas next week, this is kinda late for setting up an add. task
11:25 < geertu> Then try all possible sampling points, and see if it helps
11:25 < wsa_> first start would be to see if you can actually change bits in the register ;)
11:26 < uli___> i'll look into it
11:27 < wsa_> for completeness, the planned IO tasks for Q3/2:
11:27 < wsa_> Simon - IP csum offloading for EtherAVB
11:27 < wsa_> Uli - (hopefully) this sampling point adaption for SCIF
11:28 < wsa_> Wolfram - DT bindings for SD/MMC drive strength settings (and side-effect: SDHI ES2.0 enabling)
11:28 < wsa_> note: not PFC drive strength
11:29 < wsa_> this is a command sent to the controllers on the other side
11:29 < wsa_> are we done?
11:30 < geertu> 10-4
11:31 < wsa_> i don't know what it means, but I assume "yes" :)
11:31 < wsa_> I only know 7-1
11:31 < uli___> https://en.wikipedia.org/wiki/Ten-code
11:32 < uli___> only used by really old men ;)
11:33 < wsa_> hehe
11:33 < wsa_> okay, then, thank you very much
11:33 < wsa_> meeting closed
;ve submitted additional tasks for Q3 to Morimoto-san [16:28] <pinchartl> we have exchanged a few e-mails about them, hopefully everything is on track now <pinchartl> morimoto: do you still need more information from me ? is there anything I can do to help ? <morimoto> Now, shimoda-san and our boss is checking it. <pinchartl> or is everything fine ? <pinchartl> is that a good sign ? :-) [16:29] <morimoto> pinchartl: please wait. please start meeting first [16:30] <pinchartl> well, I'll assume it's not a bad sign :-) <pinchartl> that's all I wanted to announce on this topic <pinchartl> I've also noticed that Salvator-X M3 boards have started shipping [16:31] <pinchartl> so let's get ready for integration work and testing on M3 <morimoto> pinchartl: starting "paper work", not shipping ;P [16:32] <morimoto> pinchartl: I send question mail to you. please check it <pinchartl> well, paperwork is the first step for shipping :-) <pinchartl> I will right after the meeting <neg> back, sorry about that <pinchartl> neg: no worries <pinchartl> I'll give you a minute to read the log. any question ? [16:33] <neg> thx, I'm good <pinchartl> ok, let's go round the (virtual) table then [16:34] <pinchartl> in alphabetical order as usual <pinchartl> using the real alphabetical order this time, I'll start <pinchartl> since last meeting in Japan three weeks ago <pinchartl> - there was RenesasCon and LinuxCon Japan [16:35] <pinchartl> followed by a week of holidays <pinchartl> leaving a week and a half of work <pinchartl> - I've continued working on the request API. Hans Verkuil is back from holidays, I've discussed contention points with him and Sakari Ailus [16:36] <pinchartl> in the end there was no more disagreement with my proposal, so I've continued working on the implementation [16:37] <pinchartl> - I've also started IPMMU + DU integration, targetting Gen3 <pinchartl> the result can't be tested properly on H3 as the IPMMU is known to be faulty <pinchartl> partial tests will be performed on Gen2 by hooking the VSP1 to the DU the same way as we do on Gen3 [16:38] <pinchartl> - last but not least, I acted as Magnus' deputy for additional tasks negotiation <pinchartl> during the next two weeks, I will [16:39] <pinchartl> - continue working on IPMMU + DU and the request API <pinchartl> - continue Kieran's work on the VSP image partitioning implementation <pinchartl> (Kieran has left for his honeymoon and will be back at the end of the month) [16:40] <pinchartl> issues and blockers: <pinchartl> - the IPMMU is faulty on H3, blocking full testing of the IPMMU + DU integration, but that's a known problem and not a blocker as explained above [16:41] <pinchartl> no other issue for me <pinchartl> ah, I will also during the next two weeks try to get the VSP HGO support upstreamed [16:42] <pinchartl> a new iteration of the patch series might be needed with minor changes, I don't expect any problem <pinchartl> that's it for me <pinchartl> Morimoto-san, you're next in the alphabetical order [16:43] <morimoto> OK, <morimoto> I have posted cleanup patches which is needed for OF graph. [16:44] <morimoto> but still no responce from maintainer. I wonder is Europe under summer vacation now ?? <morimoto> 1 - 1.5 month, no responce <pinchartl> some people must be [16:45] <morimoto> OK <pinchartl> who is the maintainer ? <pinchartl> Mark Brown ? <morimoto> Yes <morimoto> Do you know his situation ?? <pinchartl> he has been quite active on the kernel summit mailing list lately, so he's definitely alive <morimoto> OK alive. but no responce... [16:46] <morimoto> Anyway, because of this, I started to solve other headache <pinchartl> he doesn't seem to be on vacation <pinchartl> you can ping him <morimoto> Sometimes, he post his mail, but he doesn't accept patches, not only me. [16:47] <morimoto> So now, I started to solve other headache which is "unload" issue on ALSA SoC [16:48] <pinchartl> ah <pinchartl> a big one <pinchartl> I've seen your e-mails on that <pinchartl> the topic was also mentioned by Lars in the kernel summit mailing list <morimoto> Yes. ALSA SoC has same issue [16:49] <pinchartl> that's related to hotplug support, right ? <morimoto> Yes. <morimoto> especially, hot-unplug [16:50] <pinchartl> ok <morimoto> I'm not sure how to solve this issue, but I started to investigate ALSA SoC framework, and I noticed that it has many duplicate implementation. <morimoto> It makes hotplug support difficult I think [16:51] <morimoto> So, I would like to cleanup ALSA SoC <pinchartl> I think you can discuss it with Lars if needed, he seems quite interested in this topic as well <morimoto> Now, I'm asking about it ot ML. <morimoto> Nice idea. [16:52] <morimoto> I think I can have a chance to talk him on next ELCE ? <pinchartl> I would expect so, yes <pinchartl> Mark Brown should hopefully be there too <morimoto> I hope so <pinchartl> maybe you should schedule a meeting with them at ELCE <pinchartl> I would be interested in joining <morimoto> sounds nice for me [16:53] <morimoto> with beer ? <morimoto> Anyway, this is current my status. <pinchartl> :-) <morimoto> I think this "hotplug" is requested for HDMI ? [16:54] <pinchartl> yes <pinchartl> well, I don't know how HDMI audio is implemented in ALSA <pinchartl> and whether unplugging the cable will remove the device or not <morimoto> In my understand, "unplugging cable" and "unload driver" are different problem. [16:55] <morimoto> But, some of them can be related ? <morimoto> no sure <pinchartl> they can be, but that depends on how ALSA handles it. I don't know <pinchartl> is that something you can investigate ? <morimoto> I think so. [16:56] <pinchartl> thanks <pinchartl> what is your plan for the next two weeks ? <morimoto> For this purpose, my 1st step is cleanup framework. It is now tasty spaghetti <morimoto> 1) continue OF graph related patch work [16:57] <morimoto> 2) investigate and cleanup ALSA SoC <morimoto> It is related Mark's situation <pinchartl> any issue or blocker, apart from Mark being unresponsive ? [16:58] <morimoto> no blocker. <pinchartl> thank you [16:59] <morimoto> Oops, export paper work sometimes block me :) <pinchartl> :-) <pinchartl> neg: you're next <neg> since last time: Working on addaptions to rcar-vin to keep up with patches posted for adv7180 and help out Sergie with his problem. Added ALTERNATE field support to rcar-vin <neg> plan: Keep pushig for those patches and once they are picked up start to break out patches from my Gen3 VIN series and post them in smaller chunks to try and get some tracktion on them <neg> issues: No reviews on the VIN Gen3 and CSI-2 patches and unclear how to handle ADV7482 tasks. <pinchartl> I suppose the "no review" is a hint for me ? :-) [17:00] <neg> :-) <pinchartl> I'll get to that [17:01] <pinchartl> regarding ADV7482, would you like to discuss it at some point ? <neg> np, if you are super swamped maybe you can hold off abit untill I start sending out break out parts of the Gen3 series and review them <pinchartl> early next week would work best for me if that's fine with you [17:02] <pinchartl> Monday or Tuesday <neg> yes I would like to discuss ADV7482, just let me know when you have some time <pinchartl> (I'll be unavailable next Wednesday) <pinchartl> ok [17:03] <neg> sure, Monday morning and between 15-17 I will have builders starting construction but other than that just ping me when you have time <pinchartl> Monday 13:00 your time ? <neg> works <pinchartl> ok [17:04] <pinchartl> uli___: you're next <uli___> for work done, see the status update <uli___> planned is to start on the additional tasks, plus deal with this: <uli___> issue: <uli___> the hdmi gen3 output needs to read the product register to handle es1.1 [17:05] <pinchartl> ouch <uli___> right now, that is a hardcoded register access <uli___> which people dislike <uli___> in response, dirk behme wrote a driver <uli___> which people consider overkill, i guess :) <uli___> the last alternative is to code that in the DT, which i consider a technical fail, considering it can be autodetected <uli___> any opinions on how to tackle this? [17:06] <pinchartl> why does the hdmi code need the product register value ? <pinchartl> what is handled differently ? <uli___> i must admit i have not looked into what it exactly does :) maybe morimoto knows? <morimoto> about ES1.1 ? [17:07] <uli___> yes <morimoto> Unfortunately, ES1.0 and ES1.1 have different behavior [17:08] <morimoto> on HDMI <pinchartl> is the code that handles the ES1.0/ES1.1 differences public ? <uli___> it's from the bsp [17:09] <pinchartl> where I can get it from ? <uli___> morimoto-san sent an email about that to periperi [17:10] <uli___> as a response to a group meeting summary <uli___> Then, apply this new version of DPLL support patch from BSP. <uli___> But, you will get conflict, but it is not difficult to fix. <uli___> 01b4818ac558f17f7c74505003c5cca2fc149de1 <uli___> (drm: rcar-du: Add DPLL support) <uli___> Above DPLL support needs new header from BSP. <uli___> 9c4c8fb9e6c11fb9594e3edaccc1c73976e0cfe6 <uli___> (soc: renesas: Add product register helpers) <pinchartl> what's the mail's subject ? <uli___> "Renesas Multimedia Group Meeting" :) [17:11] <morimoto> Re: [periperi] Renesas Multimedia Group Meeting <uli___> it's from june 2 <morimoto> Date: Thu, 2 Jun 2016 01:17:13 +0000 <morimoto> <pinchartl> got it <pinchartl> I will have a look <morimoto> It came from BSP <pinchartl> uli___: when do you plan to address that ? [17:12] <pinchartl> (to know what my deadline is to look at the issue) <uli___> in the not too distant future, if possible <uli___> i actually wanted to send it out yesterday, then i remembered that this isn't addressed yet [17:13] <pinchartl> could you be slightly more precise ? :-) <uli___> in a week? <uli___> or two? [17:14] <pinchartl> ok, I'll try to check that on Monday too [17:15] <pinchartl> is that fine ? <uli___> yes, perfectly fine. thanks. <pinchartl> you're welcome <morimoto> I and Magnus had discussed about that, we can use Geert's DT compatible magic ? I believe it will add es version on compatible. <morimoto> "r8a7795-salvator" -> "r8a7795-salvator-es1.1" "r8a7795-salvator" <uli___> that's the option i'd like to avoid if at all possible <morimoto> OK [17:16] <uli___> but i can deal with it, if necessary <pinchartl> I also wonder how much work we should put into that, given that es 1.0 is meant to disappear <pinchartl> whatever solution we come up with, it must not make es 1.1 support too complex or dirty <pinchartl> anyway, I'll have a look [17:18] <uli___> ok, thank you <pinchartl> any last comment from anyone ? <pinchartl> I'll take that as a no [17:19] <neg> thanks for steping up as Magnus deputy :-) <pinchartl> you're welcome <pinchartl> I propose scheduling the next meeting in two weeks, Wednesday the 17th, at the usual time (half an hour later than today) <pinchartl> would that be fine for everybody ? <neg> OK for me [17:20] <uli___> fine for me <morimoto> Not OK for me <pinchartl> :-/ <morimoto> it is under summer vacation <pinchartl> when are your summer vacation ? <morimoto> 13 - 21 <morimoto> 13th Aug to 21th Aug I mean [17:21] <pinchartl> I give you permission to skip the meeting then :-) <morimoto> :) <pinchartl> if you could send a status report either before leaving for vacation, or after coming back, it would be appreciated <neg> I will be in Berlin from tomorrow to sunday, so I might be slow in responding during that time [17:22] <morimoto> pinchartl: OK, I will try <pinchartl> morimoto: thank you <pinchartl> neg: ok, thanks for the information <pinchartl> on the same topic, I will be on vacation from Saturday the 20th to Sunday the 28th [17:23] <pinchartl> we've reached the one hour limit <pinchartl> and we're done with the meeting, so it's a good timing <pinchartl> thank you everybody, and have a nice day in Europe or evening in Japan [17:24] <neg> thanks all <morimoto> Thanks !