path: root/wiki/Chat_log/20160921-io-chatlog

--- Log opened Wed Sep 21 09:27:21 2016
09:27 -!- wsa_ [~wsa@p5B384463.dip0.t-ipconnect.de] has joined #periperi-io
09:27 -!- Irssi: #periperi-io: Total of 4 nicks [0 ops, 0 halfops, 0 voices, 4 normal]
09:27 -!- Irssi: Join to #periperi-io was synced in 0 secs
09:27 < wsa_> hi guys!
09:29 < khiemnguyen_> hello
09:30 -!- shimoda [~shimoda@relprex3.renesas.com] has joined #periperi-io
09:30 -!- horms [~horms@217.111.208.18] has quit Quit: Leaving
09:30 -!- horms [~horms@217.111.208.18] has joined #periperi-io
09:31 -!- neg [~neg@unaffiliated/neg] has joined #periperi-io
09:32 < morimoto> hi
09:32 < neg> hello everyone
09:32 < shimoda> hi
09:33 < wsa_> morimoto: before we start, can you also keep an eye that either Magnus or you bring me a ZComex AC-180M SDIO card to Berlin?
09:34 < wsa_> I need it for an additional task
09:34 < wsa_> I think Shimoda-san used it for testing
09:35 < wsa_> I already asked Magnus. He recommended to ask you as well, so it will be less likely forgotten ;)
09:35 < morimoto> Magnus recommended to me ?
09:35 < morimoto> OK, will check
09:36 < morimoto> I will bring it to you on Berlin
09:36 < wsa_> well, if two people know about it, they can remind each other ;)
09:36 < wsa_> so, let's start
09:36  * wsa_ runs sort -R
09:37 -!- geertu [~geert@d54c189fd.access.telenet.be] has joined #periperi-io
09:37 < geertu> Hi
09:38 < wsa_> hi geert
09:38 < horms> nice to see you
09:38 < wsa_> shimoda: you won first place this time :)
09:39 < shimoda> yes
09:39 < shimoda> A) What have I done since last time:
09:39 < shimoda>     - IPMMU issue on USB EHCI + Gen3.
09:39 < shimoda> B) What I plan to do till next time:
09:39 < shimoda>     - IPMMU issue on USB EHCI + Gen3.
09:39 < shimoda> C) Problems I have currently:
09:39 < shimoda>     - Nothing except the IPMMU issue.
09:39 < shimoda> i have trouble about the IPMMU :)
09:39 < shimoda> done
09:40 < wsa_> Yeah, that looks like a major one
09:40 < wsa_> any news from HW guys?
09:42 < wsa_> ah, i think you said last time that he didn't find anything
09:42 < shimoda> I got a little. but, they said it seems strorange behavior by usb EHCI hw. so, i would like to investigate EHCI SW or IPMMU HW/SW more
09:43 < wsa_> good luck with that!!
09:43 < wsa_> I am next
09:43 < wsa_> A)
09:43 < wsa_> added 8 bit eMMC bus width support for SDHI
09:43 < wsa_> tested SDR104 & enablement patches
09:43 < wsa_> worked on additional tasks and todo-lists
09:43 < wsa_> B)
09:43 < wsa_> organize meeting room in Berlin & IO meeting
09:43 < wsa_> some more SDR104 testing
09:43 < wsa_> start evaluating the BSP for IO patches
09:43 < wsa_> start playing with UHS SDIO cards
09:43 < wsa_> C)
09:43 < wsa_> (SDR104 is nasty)
09:43 < wsa_> c) is not really my part
09:43 < horms> I have an update on that
09:44 < wsa_> yeah, i read that :)
09:44 < horms> yes, SDR104 is a headache
09:45 < wsa_> but first morimoto
09:45 < horms> lets discuss it a bit later. either here of back on #periperi after this meeting
09:45 < wsa_> horms: sure
09:46 < wsa_> although, morimoto said he has nothing going for IO
09:46 < wsa_> khiemnguyen_: you are next
09:47 < wsa_> how is your load? do you have more time for upstreaming again?
09:47 < khiemnguyen_> wsa_: I have time for upstream, I think.
09:47 < khiemnguyen_> a) Sent updated patches for R-Car Gen3 Thermal
09:48 < khiemnguyen_> Got review from Morimoto-san and Geert.
09:48 < khiemnguyen_> Still, some comments have not been fixed yet. And failed to catch v4.9.
09:48 < khiemnguyen_> b) Continue R-Car Gen3 Thermal.     It will be available before ELCE, I believe.
09:48 < khiemnguyen_> c) Still try to solve issue of my email account.     Trying to use git-send-emails instead of MS Outlook email client.
09:48 < khiemnguyen_> that's all.
09:49 < wsa_> good news. i updated the todo and upstreaming thermal is now scheduled for v4.10
09:49 < khiemnguyen_> wsa_: thanks
09:49 < morimoto> this is out-of-IO-topics. horms: can you update peripelist ?
09:50 < wsa_> good luck and I hope you can get git-send-email to run. will be much more convenient!
09:50 < wsa_> horms: you are next anyhow ;)
09:50 < horms> morimoto: I am confused. What would you like me to update?
09:50 < horms> --- start ---
09:51 < horms> A) What have I done since last time:
09:51 < horms>     - Posted updated SDR104 Driver patches
09:51 < horms>     - Posted updated SDR50/SDR104 enablement patches
09:51 < horms>       - merged non-SDR104 portions
09:51 < horms>     - Support patches for pfc, gpio, clks have been merged
09:51 < horms> B) What I plan to do till next time:
09:51 < horms>     - Continue working on "Samsung SDR104 initialisation problem"
09:51 < horms>       - Locally I seem to have a stack that resolves this problem,
09:51 < horms>         I need to:
09:51 < horms>         + Verify this and if it is so
09:51 < horms>         + clean it up and isolate what needs to go upstream.
09:51 < horms>     - Investigate why SDR50 can't be initialised on Gose SDHI1
09:51 < horms> C) Problems I have currently:
09:51 < horms>     - SDR headaches, see B)
09:51 < horms> --- end ---
09:51 < morimoto> horms: can you check email from me ?
09:51 < morimoto> Subject: Re: [periperi] PeriPelist fixup request 2016-09-13 for	integration/upport
09:51 < morimoto> Date: Tue, 13 Sep 2016 01:54:57 +0000
09:51 < morimoto>  
09:52 < horms> sure, i will look at that
09:52 < wsa_> horms: yeah, let's talk about the technical details after this meeting here?
09:52 < horms> good plan
09:52 < wsa_> and maybe about Gose, this is a new issue, or?
09:53 < wsa_> I can't recall it at least
09:54 < geertu> Perhaps SDHI is wired differently on Gose?
09:54 < horms> i think we can regard it as a minor issue
09:54 < horms> its not so new to me
09:54 < geertu> We don't seem to have schematics...
09:54 < horms> but i may not have raised it to you before
09:54 < horms> I believe I do have them
09:54 < horms> The problem is that SDR%0 is not initialised on SDHI1
09:54 < horms> I had magnus move the card to SDHI2 and it seems fine there
09:55 < horms> I think it is most likely there is an error in the code somewhere
09:55 < morimoto> geertu: I will send Gose schematics soon
09:55 < morimoto> It is under export paper work
09:55 < horms> Also, I wasn't entirely sure on some of the GPIOs, so a second pair of eyes on the schematic would help
09:55 < geertu> horms: it's not due to some silly SDHIx naming issue? Indices are different on H2 and M2-W.
09:55 < geertu> IIRC, we did it slightly different on M2-N?
09:55 < horms> I am aware of that
09:56 < horms> no I don't think that is the problem here
09:56 < geertu> morimoto: Thank you
09:56 < horms> the documentation is a little inconsistent with its usage of sd1/sd2
09:56 < horms> but I think its clear enough given some context
09:57 < wsa_> ok?
09:58 < horms> I'm ok
09:58 < wsa_> good :)
09:58 < wsa_> neg: I know you don't have anyhthing IO related, but can you tell me if you have time for base task assignments in the near future?
09:59 < neg> I think I should have time for that
09:59 < neg> I do not yet have a base contract for Q4 but other than that it should be fine :-)
09:59 < wsa_> same here:D
10:00 < wsa_> ok, so then let's move on to Geert
10:00 < neg> any particular task you think I should get started on?
10:00 < wsa_> neg: I'll ask Geert in a second ;)
10:01 < wsa_> geertu: your turn
10:01 < geertu> A) What have I done since last time: - SPI slave support v2, http://elinux.org/Tests:MSIOF-SPI-Slave
10:01 < wsa_> (let's see if the alerting for irssi I sent him recently works ;))
10:01 < geertu> B) What I plan to do till next time:
10:01 < geertu>     - No I/O tasks scheduled, but core 64-bit Memory and Ethernet without
10:01 < geertu>       Bounce Buffers Prototype,
10:01 < geertu>     - Awaiting SPI slave feedback,
10:01 < geertu>     - Verify MSIOF SPI functionality on M3-W, if time permits.
10:01 < geertu> C) Problems I have currently: - Nothing I/O related.
10:02 < geertu> First SPI slave feedback is from RobH, who seem to want to revert partly to v1, while I made the changes due to his request ;-)
10:02 < wsa_> heh
10:03 < wsa_> this task:
10:03 < wsa_> SCIF,?,noplan,?,difference in behavior between hardware-assisted flow control and GPIO flow control # Cfr. the thread "[PATCH] sh-sci: fix transition from noflow to HW flow control"
10:03 < wsa_> do you think this could be a investigation suited for a base task
10:03 < wsa_> or is it rather an additional task on its own?
10:03 < geertu> Perhaps I can look into that in the context of the (not yet confirmed) base task
10:04 < wsa_> if you have time...
10:04 < wsa_> ... this is the one where I wondered if neg could have a look if he had interest.
10:04 < geertu> I think it's to be investigated first, whether it's a bug in the driver, on in the serial core.
10:05 -!- horms [~horms@217.111.208.18] has quit Read error: Connection reset by peer
10:05 < geertu> s/on/or/
10:05 -!- horms [~horms@217.111.208.18] has joined #periperi-io
10:06 < geertu> I have a hardware setup to test it, so perhaps I should just fine some time to bite the bullet
10:06 < geertu> s/fine/find/
10:06 < wsa_> i see
10:06 < wsa_> makes sense
10:08 < wsa_> i think we are done now?
10:08 < wsa_> anything from you?
10:08 < wsa_> next meeting will be in Berlin
10:09 < wsa_> if you have topics for that please let me know
10:09 < wsa_> additional tasks will be an obvious one
10:10 < horms> Have we fixed the meeting schedule for Berlin?
10:10 < wsa_> i think so
10:11 < horms> Excellent
10:11 < wsa_> now that the core group decided to have their meeting in two parts...
10:11 < horms> I want to be sure to be there at the right time :)
10:11 < horms> Shall we talk SDR?
10:11 < wsa_> yes, I am very curious on that topic
10:12 < wsa_> but before
10:12 < wsa_> Thanks guys for the meeting!
10:12 < wsa_> Have a safe trip to Berlin!
10:12 < neg> thanks all
10:12 < khiemnguyen_> Please take a photo, as well.
10:13 < morimoto> thanks.
10:13 < shimoda> thanks.
10:13 < geertu> Has anyone found the big group photo taken at the closing of LCJ?
10:13 < horms> thanks
10:14 < morimoto> geertu: like this one ?
10:14 < morimoto> https://osdr.renesas.com/projects/linux-kernel-development/wiki/Miniperi-2016-07
10:14 < khiemnguyen_> geertu: https://osdr.renesas.com/projects/linux-kernel-development/wiki/Miniperi-2016-07
10:15 < morimoto> It is shrink version, I have original photo
10:15 < geertu> morimoto: No, the big one with +100 persons
10:15 < morimoto> Ahh, I don't have it.
10:16 < horms> wsa_: I think I am making some progress on the "Samsung problem"
10:16 < horms> as I mentioned in my email
10:16 < horms> but there have been so many false dawns on sdr104 I'd rather not push out v8 in a hurry, even though v4.9 is closing soon
10:17 < horms> what I was hoping was to get something clean toghether in the next day or so for you to test.
10:17 < wsa_> morimoto: haha, haven't seen those pictures before
10:17 < horms> or alternatively find out its not working after all (not unlikely given my experiences so far)
10:18 < wsa_> horms: yes, I agree we should test more internally before sending out a new version
10:18 < wsa_> horms: I'd also test unclean stuff ;) just to see if I can reproduce your findings
10:18 < horms> I at least have physical access to a system where I can reprocuce the current problem (The Samsung one)
10:18 < wsa_> anyway, I'm happy that you at least have a pointer
10:19 < horms> thanks, I'll see if I can get something semi-clean together
10:19 < horms> there is quite a bit of noise in my current stack
10:19 < wsa_> any high-level description of the suspect?
10:19 < horms> but one thing that stood out was the NO_SLEEP handling in the BSP
10:19 < horms> which skips sleeping for 10ms on GEN3
10:20 < horms> Assuming we need such a thing, which at this point seems likely
10:20 < horms> I wonder how it should be handled
10:20 < horms> * Using a NO_SLEEP cap as per the BSP
10:20 < horms> * Using a (new) GEN3 cap (like the GEN2 cap in mainline - which is actually GEN2/3)
10:20 < horms> * Some other way
10:21 < wsa_> NO_SLEEP? gotta check
10:22 < horms> I also noticed that SH_MOBILE_SDHI_SCC_RVSCNTL_RVSEN should be BIT(0) rather than BIT(1). Which clearly should be fixed. But its not clear if it relates to the problem under discussion.
10:22 < horms> I also flagged the following as interesting but have not yet been able to confirm if they related to the problem in question:
10:23 < horms> * mmc: tmio: Add SDCLK enable after reset
10:24 < horms> * clk: renesas: rcar-gen3: Replace SD divider setting
10:24 < horms> From the BSP
10:24 < horms> The first patch seems like a mainline candidate
10:24 < horms> the second I am rather unsure about
10:25 < horms> Other than that my strategy has been to try to reduce differences between the BSP and mainline+sdr topic branch.
10:25 < horms> That seems to be working, I need to work out which of those changes, which are rather verbose, are of consequence
10:25 < wsa_> that's what I did to fix 8 bit eMMC support as well
10:25 < horms> thats the bit I'D like to clean up a bit
10:25 < horms> really its too noisy to give you right now
10:26 < wsa_> I think we have the same NO_SLEEP behaviour upstream as in the BSP?
10:26 < wsa_> that was at least my intention, let me double check
10:26 < horms> It didn't seem to be in the branch I was working against
10:26 < horms> I did see some differentiation between GEN2 and non-GEN2 for one of the timeouts
10:26 < horms> but nothing for others
10:27 < wsa_> bsp does:
10:27 < wsa_> 172                 if (!(host->pdata->flags & TMIO_MMC_CLK_NO_SLEEP))
10:27 < wsa_>  173                         msleep(10);
10:28 < wsa_>  upstream does
10:28 < wsa_>  160         msleep(host->pdata->flags & TMIO_MMC_MIN_RCAR2 ? 1 : 10); 
10:28 < horms> ok, but the BSP does it in several places
10:28 < wsa_> so, the timeout is reduced from 10 to 1 ms
10:29 < wsa_> and bsp for gen3 does 0
10:29 < horms> in any case, i can do some more testing of this
10:30 < horms> btw, which samsung card do you have. I have a "Pro+" 32Gb. I can look up the product id if you like.
10:31 < wsa_> let me check
10:33 < wsa_> it is labelled "32 pro"
10:33 < horms> http://www.samsung.com/de/consumer/memory-storage/memory-storage/memory-cards/MB-SD32D/EU
10:33 < horms> Is it grey and white?
10:34 < horms> A bit like this? https://www.amazon.co.jp/Samsung-SDXC%E3%82%AB%E3%83%BC%E3%83%89-SAMSUNG-Class10-%E6%9C%80%E5%A4%A7%E8%AA%AD%E5%87%BA%E9%80%9F%E5%BA%A690MB/dp/B019FKR6QQ
10:35 < wsa_> so, no "+" for me
10:35 < wsa_> http://www.samsung.com/de/consumer/memory-storage/memory-storage/memory-cards/MB-MG32E/EU
10:35 < wsa_> it is grey and white
10:35 < horms> Ok
10:35 < horms> I have some of them in Japan
10:36 < horms> I thought I sent one to my new home too, but I can't locate it
10:36 < horms> I noticed these things seem to be much more expsnsive here than in Japan
10:36 < horms> I might just post some to myself :)
10:37 < horms> But I digress
10:37 < horms> its good to know you have a different card
10:37 < horms> but I think for now we are seeing the same problem
10:37 < wsa_> so, the gen2 bsp does not use msleep as well
10:38 < wsa_> i decided to use 1 ms because the datasheet has
10:38 < wsa_> "After waiting for at least 1 ms since the SD clock supply was started , check that the DAT0 bit in
10:38 < wsa_> SD_INFO2 is 1."
10:38 < wsa_> in the Gen2 docs
10:38 < wsa_> checking gen3 docs
10:39 < horms> I just noticed that the NO_SLEEP change was added to the Gen3 BSP specifically for r8a7795 support (presumably r8a7796 was not on the todo list at the time)
10:39 < horms> Possibly 1ms is ok, possibly it is not
10:39 < horms> Lets try to get some sort of semi clean patch stack together that wrosk
10:39 < horms> works
10:39 < wsa_> yeah
10:39 < horms> and then we can invistigate this in more depth
10:40 < wsa_> but it is nice to know that this is a difference to the BSP
10:40 < wsa_> I wasn't aware of that
10:40 < horms> at this point I think we may have several issues to discuss
10:40 < wsa_> unlike this patch I am aware of:
10:41 < wsa_> mmc: tmio: Add SDCLK enable after reset
10:41 < horms> What is your thoguths on that one?
10:41 < horms> What are your thoughts on that one?
10:41 < wsa_> but I haven't seen making any difference on any issue I have seen so far
10:41 < wsa_> I don't know if this a angst-patch or a real one
10:41 < morimoto> bye
10:41 -!- morimoto [~user@relprex2.renesas.com] has left #periperi-io ["ERC Version 5.3 (IRC client for Emacs)"]
10:42 < wsa_> it is a good candidate to tell about the importance of commit messages
10:42 < wsa_> "not so much what but especially why"
10:43 < horms> ok
10:43 < horms> we/I can raise this with the bsp team
10:43 < horms> I'll check if it helps or not
10:43 < horms> it is in my stack at the moment
10:44 < horms> possibly we would get better commit messages if there was a policy to allow them to be partially in Japanese
10:44 < horms> but that could also backfire
10:45 < wsa_> stuff like
10:45 < wsa_> "fixes customer issue" vs "is more correct" already helps IMO
10:45 < horms> Ok, so this actually relates to the upporting tasks
10:45 < wsa_> or "do as described in docs"
10:45 < horms> which Magnus was discussing with us
10:46 < wsa_> yes
10:46 < horms> We could produce some guidelines to help get better information
10:46 < horms> probably part of the problem is who is the audience for the changelog
10:46 < horms> * The BSP team?
10:46 < horms> * The upstream team?
10:46 < horms> * Mainline iself?
10:46 < horms> * Customers?
10:47 < horms> * Other?
10:47 < horms> I would be interesting to see what the intended audience is: if any
10:47 < horms> but i think regardless it would be good to provide some guidance
10:47 < shimoda> sorry, time to go. bye!
10:47 -!- shimoda [~shimoda@relprex3.renesas.com] has quit Quit: WeeChat 0.4.2
10:47 < wsa_> I dunno about customers, but BSP team and upporting team for sure
10:48 < wsa_> mainline not so much
10:48 < horms> I think they are the most important
10:48 < wsa_> I'd say
10:48 < wsa_> but for all of the groups, I'd think the "why" is pretty important
10:49 < horms> true
10:49 < wsa_> and the patch we just mentioned is a good example for that
10:49 < horms> ok
10:49 < horms> I think we can query specific patches by contactng their author
10:50 < horms> but in terms of a general request I'd shy away from specific examples: we don't want anyone in the BSP team to give less information for fear or being made an example of
10:50 < wsa_> i agree
10:50 < geertu> The "why" is very important. For several commits in the BSP, I'm completely puzzled by the rationale behind them.
10:50 < horms> i agree
10:51 < wsa_> luckily, it is pretty easy to cook an artifical example
10:51 -!- uli___ [~uli@gateway/vpn/privateinternetaccess/uli/x-17929268] has joined #periperi-io
10:51 < horms> So how about I try to bring this up with the BSP team at the meeting they invite me to sometimes.
10:51 < wsa_> hi uli___ 
10:51 < uli___> hello
10:51 < wsa_> got internet again? :)
10:51 < uli___> figured out why my desktop keeps locking up
10:51 < uli___> turns out you can't turn off displayport devices in linux...
10:52 < uli___> switched to dvi -> works
10:52 < horms> good to know
10:52 < wsa_> horms: gotta run now
10:53 < horms> likewise
10:53 < wsa_> looking forward to your semi-cleaned up stack
10:53 < horms> thanks for your time
10:53 < wsa_> to test
10:53 < wsa_> you're welcome
10:53 < horms> i'll get that too you in the coming days: or some disapointed email explaining that its not working
10:53 < wsa_> oh, and I'll bring all my cards to Berlin, of course
10:53 < horms> oh. great
10:53 < wsa_> Magnus shall do the same
10:54 < wsa_> so, cya all!
10:54 < horms> bye
10:55 < uli___> bye
10:57 < wsa_> uli___: BTW can you respin the MSIOF chipselect patch?
10:58 < uli___> yes, that's on the todo
10:58 -!- horms [~horms@217.111.208.18] has quit Ping timeout: 244 seconds
11:05 < wsa_> cool
--- Log closed Wed Sep 21 11:05:52 2016