summaryrefslogtreecommitdiff
path: root/wiki/Chat_log/20171005-core-chatlog
diff options
context:
space:
mode:
authorKuninori Morimoto <kuninori.morimoto.gx@renesas.com>2019-12-09 15:29:52 +0900
committerKuninori Morimoto <kuninori.morimoto.gx@renesas.com>2019-12-09 16:23:07 +0900
commit55e3b2f45880faaf06f3c660ca9e8a6d9aa14bce (patch)
tree6392fd201a51ff0f6dc0e474803e6f3b20919504 /wiki/Chat_log/20171005-core-chatlog
parent5d9e1b983faf7645ddc3d45d28e612d2ac4179c0 (diff)
wiki: Porting wiki: Porting Chat Log
Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Diffstat (limited to 'wiki/Chat_log/20171005-core-chatlog')
-rw-r--r--wiki/Chat_log/20171005-core-chatlog219
1 files changed, 219 insertions, 0 deletions
diff --git a/wiki/Chat_log/20171005-core-chatlog b/wiki/Chat_log/20171005-core-chatlog
new file mode 100644
index 0000000..1c43b17
--- /dev/null
+++ b/wiki/Chat_log/20171005-core-chatlog
@@ -0,0 +1,219 @@
+Core-chat-meeting-2017-10-05
+
+09:48 < geertu> Welcome to today's core group meeting.
+09:48 < kbingham> and various people leave the room :D
+09:48 * pinchartl has to go for one hour
+09:48 * kbingham hides
+09:48 < geertu> Given the small amount of time left, OK if I just summarize the copy-and-paste received status reports?
+09:49 < geertu> A) What have I done since last time:
+09:49 < geertu>
+09:49 < geertu> Geert:
+09:49 < geertu> - Sent patch to fix new unwind warnings with CONFIG_DEBUG_LOCK_ALLOC=y,
+09:49 < geertu> - Sent v1 of sh-pfc suspend/resume,
+09:49 < geertu> - Sent first pull requests for clk and pfc for v4.15,
+09:49 < geertu> Jacopo:
+09:49 < geertu> - Some small patches for Gr-Peach
+09:49 < geertu> Marek (U-Boot):
+09:49 < geertu> - Got more patches applied to u-boot-sh/rmobile, PR pending
+09:49 < geertu> http://git.denx.de/?p=u-boot/u-boot-sh.git;a=shortlog;h=refs/heads/rmobile
+09:49 < geertu> - Worked on the SDIF SDR104/HS200 support based on out-of-tree
+09:49 < geertu> patchset (for now):
+09:49 < geertu> - added support for the SDR104/HS200 calibration to the uniphier-sd
+09:49 < geertu> driver
+09:49 < geertu> - added support for pin voltage switching to PFC driver (POCCTRL)
+09:49 < geertu> - added proper regulator handling to uniphier-sd
+09:49 < geertu> - Worked on HS400, but that's still not ready
+09:49 < geertu> Morimoto:
+09:49 < geertu> - V3M/D3 shipping paper work
+09:49 < geertu> Shimoda:
+09:49 < geertu> - Submitted the following patches:
+09:49 < geertu> * Add PWM entries of PFC for r8a77995.
+09:49 < geertu> * Add USB3.0 entries of PFC for r8a7795{-es1} and merged.
+09:49 < geertu> * Fix avb_pins for salvator-common, ulcb and draak.
+09:49 < geertu> Remarks: "*" means merged into the subsystem repo.
+09:49 < geertu> - I obtained a R-Car M3-N SiP, but I don't got bootloader yet.
+09:49 < geertu> Simon:
+09:49 < geertu> - Continued addressing review of CPUFreq patches
+09:50 < geertu> B) What I plan to do till next time
+09:50 < geertu> Geert:
+09:50 < geertu> - v3 of cpg-mssr suspend/resume,
+09:50 < geertu> - Fix genpd for wake-up devices.
+09:50 < geertu> Marek (U-Boot):
+09:50 < geertu> - Finish HS400 support
+09:50 < geertu> - Finish sorting up remaining PCIe patches from the BSP (mostly
+09:50 < geertu> suspend/resume support)
+09:50 < geertu> Morimoto:
+09:50 < geertu> - continue V3M/D3 paper work
+09:50 < geertu> Niklas:
+09:50 < geertu> - Send v2 of 'rcar_gen3_thermal: fix initialization sequence for H3 ES2.0'
+09:50 < geertu> Shimoda:
+09:50 < geertu> - Add usb3.0 phy device node for r8a779[56].dtsi after the usb3 peri
+09:50 < geertu> supported a generic phy.
+09:50 < geertu> - Prepare M3-N Salvator-X board for remote access if I obtain the
+09:50 < geertu> bootloader.
+09:50 < geertu> Simon:
+09:50 < geertu> - Finalise addressing review of CPUFreq patches and repost
+09:50 < geertu> C) Problems I have currently
+09:50 < geertu> [Here I'll add breaks, in case people want to discuss]
+09:51 < geertu> Jacopo:
+09:51 < geertu> - Not really an issue to discuss with the whole group, but I keep seeing
+09:51 < geertu> the GR-Peach hanging on sleeps longer than 1ms. I suspect this is also
+09:51 < geertu> why ETHER is not working, and I had to substitute all sleep functions in
+09:51 < geertu> the image sensor driver I am currently using to develop CEU with for
+09:51 < geertu> loops of udelay(10). I blame XIP for this, but I'm currently not sure.
+09:51 < geertu> Suggestions on how to debug and prove this are welcome :)
+09:51 < geertu> jacopo: ?
+09:51 < dammsan> use migo-r =)
+09:51 < geertu> Does it work on Genmai?
+09:51 < dammsan> (i don't dislike gr-peach support though)
+09:52 < jmondi> geertu: here I am
+09:52 < jmondi> never seen anything like this on genmai
+09:52 < jmondi> but seems like sleeps longer than 1ms hang the board
+09:53 < dammsan> which timer do you use?
+09:53 < geertu> Peach doesn't have rtc_x1_clk in the DTS?
+09:53 < jmondi> dammsan: as system clock?
+09:53 < jmondi> geertu: no, no RTC
+09:54 < geertu> The dmesg should tell you (compare genmai and peach)
+09:54 < jmondi> not just in dts, not RTC at all on the board
+09:54 < dammsan> rtc != clockevent != clocksource
+09:55 < wsa_> marex-cloud: the PCIE patches you mentioned, are these for U-Boot or Linux?
+09:55 < jmondi> let me look through dmesg
+09:55 < dammsan> a single clockevent used to be enough to run the kernel
+09:55 < Marex> wsa_: Linux, mostly suspend support
+09:55 < dammsan> but the twd on CA9 used to exist only in case of multi-core
+09:55 < dammsan> so instead of local timer i'm not sure what is used
+09:56 < geertu> rskrza1 and genmai enable mtu2, peach doesn't
+09:56 < dammsan> there you go
+09:56 < wsa_> Marex: nice! did you send out patches already? I don't see any on the renesas-soc list.
+09:56 < Marex> wsa_: I have them rebased to -next since a few days ago, I need to understand what some of them are doing
+09:56 < geertu> rskrza1 enables ostm[01], peach and genmai don't
+09:56 < wsa_> Marex: I can imagine :)
+09:57 < geertu> Ok, issue assumed fixed until proven otherwise soon :-)
+09:57 < Marex> wsa_: there's one which digs in the PCIe core , but not much details in the commit message
+09:57 < geertu> Marek:
+09:57 < geertu> - HS400 calibration doesn't pass on Salvator-XS for me in U-Boot,
+09:57 < geertu> but that's because I'm doing something wrong, it works in Linux
+09:57 < jmondi> geertu: dammsan: I lost you two
+09:57 < jmondi> I'm so ignorant on this
+09:57 < geertu> Marex: if you want to discuss this, I think I/O is more appropriate
+09:58 < dammsan> jmondi: your timer configuration on gr-peach is incomplete
+09:58 < geertu> jmondi: seacrh for mtu and ostm in the varipus r7s72100 DTS
+09:58 < Marex> geertu: ACK, I have feedback from wsa_ , need to check it on real board
+09:58 < geertu> Marex: OK
+09:58 < geertu> Shimoda:
+09:58 < geertu> - I got a request from BSP team about upstreaming of MFIS hwspinlock
+09:58 < geertu> feature.
+09:58 < geertu> - But, the current implementation is not good for upstreaming, I think.
+09:58 < geertu> I have 3 concerns:
+09:58 < geertu> 1) The device nodes ("mfis" and "mfis-lock") are duplicated like
+09:58 < geertu> below...
+09:58 < geertu> https://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas-bsp.git/tree/arch/arm64/boot/dts/renesas/r8a7795.dtsi?h=v4.9/rcar-3.5.8#n875
+09:58 < geertu> 2) The BSP team would like to support "MFIS lock register 8" in
+09:58 < geertu> the future, but the register area is separated from "MFIS lock
+09:58 < geertu> register [0-7]".
+09:58 < geertu> 3) The "mfis" driver for rpmsg (Remote Processor Messaging) will
+09:58 < geertu> be not upstreaming for now.
+09:58 < geertu> - This module has two (or more) features and BSP supports the following
+09:58 < geertu> features:
+09:58 < geertu> 1) hwspinlock
+09:58 < geertu> 2) rpmsg (Remote Processor Messaging)
+09:58 < geertu> - My concern point is how to make the device node for it.
+09:58 < geertu> - In my opinion, it should be a mfd driver like below:
+09:58 < geertu> mfis {
+09:58 < geertu> reg = <0 0xe6260000 0 0x1000>;
+09:58 < geertu> hwspinlock {
+09:58 < geertu> ...
+09:58 < geertu> };
+09:58 < geertu> rpmsg {
+09:58 < geertu> ...
+09:58 < geertu> };
+09:58 < geertu> };
+09:58 < geertu> - But, what do you think?
+09:58 < geertu> shimoda: I had a quick look at the DTS patch
+09:59 < geertu> mfis-lock covers lock registers 0-7
+09:59 < geertu> The mfis node covers all registers in the first 0x200 byts, but the only registers there are the same lock registers?
+09:59 < jmondi> dammsan: geert: will do.. in the meantime fyi: https://paste.debian.net/989118/
+10:00 < wsa_> Marex: and don't hesitate to forward questions to the BSP team via Shimoda-san or Morimoto-san
+10:00 < shimoda> geertu: the mfis node is for rpmsg
+10:00 < geertu> shimoda: Using the same lock registers? Or are there undocumented registers in that block?
+10:01 < geertu> You can have multiple register blocks in a single reg property, cfr. the gic nodes
+10:01 < Marex> wsa_: will do if anything pops up
+10:01 < shimoda> the out of tree driver of mfis driver for rpmsg will not use the same lock registers. Yes, there is undocumented registers for now
+10:01 < geertu> You can just have a single one, too (cfr. your "reg = <0 0xe6260000 0 0x1000>")
+10:02 < Marex> wsa_: the HS400 was just a quick test bolted on top of HS200
+10:02 < geertu> A single driver can be both a hwspinlock and and rpcmsg provider, without using MFD, I think.
+10:03 < geertu> Cfr. the CPG/MSSR driver, which is clk provider (for the clocks), genpd provider (for the clock domain), and reset controller (for module resets)
+10:05 < shimoda> geertu: but, they make two drivers (hwspinlock and rpmsg) and two device nodes now
+10:05 < geertu> shimoda: Two overlapping device nodes is a definite no-go
+10:06 < shimoda> geertu: yes, I know. but bsp team doesn't know...
+10:06 < geertu> shimoda: So you should tell them, so they know ;-)
+10:07 < shimoda> geertu: yes, I should do so :)
+10:07 < geertu> jmondi: timer_probe: no matching timers found
+10:07 < geertu> jmondi: while my last boot on genmai of v4.9 said:
+10:07 < geertu> sh_mtu2 fcff0000.timer: ch0: used for clock events
+10:07 < geertu> sh_mtu2 fcff0000.timer: ch0: used for periodic clock events
+10:08 < jmondi> geertu: yes... If I got this right, without MTU2 all clock events used for wake ups are not generated
+10:08 < jmondi> so the only working methof for sleeping is busy waiting, as udelay does
+10:09 < shimoda> about this mfis topic, bsp team don't want to make a single driver to support a hwspinlock and rpmsg, especially rpmsg. But, I'll forward this infor and discuss how do we do. Thanks!
+10:09 < geertu> shimoda: Do you need any special properties in the hwspinlock and rpmsg nodes from your proposal?
+10:09 < jmondi> I'm now testing it, just need to remove my workarounds in sensor driver code
+10:10 < dammsan> shimoda: isn't hwspinlock vs rpmsg assignment software policy?
+10:11 < shimoda> geertu: I don't need any special properties, I think. But, BSP team concern, M3-W and H3 ES1.x has only 8 locks, but other SoCs has 64 locks
+10:11 < shimoda> s/BSP team concern/BSP team has concern/
+10:11 < geertu> That can be handled by the driver based on the renesas,mfis-<soctype> compatible value, right?
+10:12 < geertu> (no, "renesas,<soctype>-mfis")
+10:12 < shimoda> dammsan: I think so, BSP team wants hwspinlock to go to upstream, but rpmsg is no-go because this is related to undocumented
+10:13 < jmondi> geertu: damsan: no more hangs... trying yout ETHER now... thank you both so far :)
+10:13 < dammsan> i see. i think mfis has existed even on mobile platforms
+10:13 < geertu> dammsan: Indeed, on all SoCs we still support, I think.
+10:13 < dammsan> register layout in such case seemed similar to msiof but it was used for mailbox communication between cou coes
+10:14 < dammsan> i mean cpu cores
+10:14 < shimoda> geertu: i think so. but also need soc_device_match for H3 ES2.0
+10:14 < dammsan> but anyway, good with hwspinlock
+10:14 < geertu> We're running out of time. Anything else related to this topic?
+10:14 < geertu> Simon:
+10:14 < geertu> - Correct handling of PLL dividor for CPUFreq
+10:15 < geertu> horms: Do you want to discuss this here?
+10:15 < horms> I think I should do some more investigation. Perhaps we can chat about it on this channel some time?
+10:16 < geertu> OK.
+10:16 < dammsan> FYI, i'm adding a silk board for remote access now
+10:16 < geertu> Anything I missed in the status reports?
+10:16 * Marex steals a bit of time, OK?
+10:17 < horms> dammsan: great, can you give me access if its not already oversubscribed?
+10:17 < dammsan> sure
+10:17 < dammsan> i'll announce the reshuffled boards via email, lets take it from there
+10:17 < geertu> Marex: Core related? I should hand over the mic to Wolfram...
+10:17 < Marex> shimoda: I sent out the xhci patch I plan to submit to the U-Boot list to the periperi list first, the license grant should be OK, can you double-check please ?
+10:18 < Marex> geertu: well it's inbetween, U-Boot is core and xhci is IO
+10:18 < shimoda> Marex: sure, I will check it
+10:18 < Marex> shimoda: thank you !
+10:18 < Marex> shimoda: it'd be convenient if this worked out, since now I can just type "usb reset" and it works :)
+10:19 < Marex> shimoda: it's hassle-free
+10:19 < shimoda> Marex: nice! :)
+10:20 < wsa_> geertu: don't worry, I am fine
+10:20 < Marex> shimoda: btw re HS200/400 and SDR104, I am a bit blocked by the SD/MMC maintainer in U-Boot, sorry about that
+10:20 < Marex> shimoda: he's not picking patches as fast as I'd like him to
+10:20 < Marex> shimoda: I'm looking for ways to ... uh ... expedite that process
+10:21 < shimoda> Marex: i got it. thanks!
+10:21 < Marex> shimoda: you're welcome, I'll keep you updated
+10:22 < wsa_> Marex: is yamada-san working on this, too?
+10:22 < Marex> wsa_: Masahiro Yamada-san ?
+10:22 < wsa_> hai
+10:22 < Marex> wsa_: he works for socionext, although he did implement the uniphier-sd which I now use on RCar Gen3
+10:22 * neg brb 2 min
+10:22 < Marex> wsa_: their block doesn't support the HS200/400/SDR104 modes though, that's renesas specific
+10:23 < Marex> wsa_: so he just reviews my uniphier-sd patches (which is nice) and makes sure they work on both
+10:23 < wsa_> I see
+10:24 < Marex> wsa_: the blocker really is the SD/MMC subsystem maintainer, he just doesn't respond much
+10:24 < Marex> wsa_: some not long time ago I was really tempted to just replace him and start handling the SDMMC stuff myself
+10:25 < Marex> wsa_: but then he came back, at like 5 mins to 12 and rolled out a PR ...
+10:25 < wsa_> how about co-maintaining?
+10:25 < Marex> wsa_: I feel like I'm hoarding functions ;-)
+10:26 < wsa_> I see
+10:26 < wsa_> that happens easily, yes
+10:26 < Marex> wsa_: right
+10:26 * neg back
+10:26 < Marex> wsa_: and since U-Boot isn't as prestigeous as linux, maintainers don't really queue up to do it
+10:27 < wsa_> okay
+10:27 < Marex> wsa_: anyway ... maybe we should continue rather than complain about lack of manpower all over the place ? :)
+10:27 < wsa_> ack