06:16 <raphari> hi all, can anyone point out what the purpose of https://github.com/rockchip-linux/rkwifibt utils??

06:17 <raphari> i understand it is for wifi bt related, but what additional purpose they server? especially those related to wifi?

08:16 <phh> raphari: what do you want to know? basically those drivers don't implement standard linux wireless api, so there are custom tools to configure it (like connect to a wifi, make a hotspot, power it down, stuff like that)

08:17 <phh> (but really you shuoldn't use broadcom drivers, just use bcm43xx mainline driver)

10:23 <raphari> phh: thanks, i would like to bring the wifi as access point and currently using hostapd for the purpose. When i configure the AP to perform autochannel selection by setting the channel to 0, the AP fails to show up. I am trying to find solution and was checking if rkwifi could help?

10:25 <phh> perhaps, but honestly that sounds very cumbersome

10:27 <phh> it would be easier to just make a script on top of iwlist scan for that -_-'

10:46 <raphari> phh: i see, thanks

10:46 <raphari> phh:what is the difference softAP and AP mode which the hostapd bringsup??

10:46 <raphari> are they same?

10:47 <phh> to the best of my knowledge, yes that's the same thing

10:47 <raphari> ok

14:52 <Ke> Esmil: OK for me, can someone confirms it fixes the issue ??

14:53 <Ke> I needed quotes on the previous commit

14:53 <Ke> quote

14:53 <Ke> whatever

14:57 <Ke> Esmil: also, yes I get the reserve memblock warning

15:05 <Ke> sphalerite: also you can confirm

15:34 <sphalerite> Ke: currently quite busy moving to another country, but I might find some time over the weekend or next week

15:44 <Ke> =o)

15:48 <Esmil> Ke: it fixes the problem for me

15:48 <Ke> can you comment on the mailing list

15:49 <Esmil> i just worry that it tapers over some underlying problem. it seems like cros_ec_cmd_xfer really ought to return the number of bytes received

15:49 <Esmil> is it just the -1 that's the problem or is it way off?

15:51 <Ke> should is not documented

15:51 <Esmil> it'd be nice if some chrome dev could look at it

15:53 <Esmil> Ke: also you need a Signed-off-by ;)

15:54 <Ke> I sort of left it out, since I did not know whether version 1 is also truncated as mentioned in the second message

15:55 <Ke> but nevertheless I am fine with masking underlying problem

15:55 <Ke> so long as the behaviour is undocumented, only designer can know what is supposed to be

15:57 <Ke> it is definitely not always bytes transferred

15:58 <Ke> oh wait

16:02 <Ke> like on 99% of time the return value is not used

16:07 <Ke> Esmil: based on cros_ec_cmd_xfer_spi we could just use ret at quick glance

16:09 <Ke> I guess this is inspectable without newer firmware

16:14 <Ke> Esmil: if I inspect the xfers, will you accept ret as memcpy param instead of ret-1?

16:14 <Ke> seems to be only i2c and spi, so 50% done

16:15 <Ke> the code is identical cros_ec_cmd_xfer_i2c in

16:16 <Esmil> Ke: well, i'm not really in any position to accept or reject anything. i'm just suggesting ways to maybe improve your patch

16:16 <Ke> well

16:18 <Ke> ok, I'll port memcpy with ret along with my 5min audit results

16:19 <Ke> post

16:19 <Ke> Esmil: you can confirm it works for you, so I am not just some crazy person

16:20 <Ke> what was funny, I tried perf probe to check cros_ec_cmd_xfer retval and I got 0

16:21 <Ke> probably I used the tool somehow incorrectly

16:22 <Ke> it definitely seems it should return len, also the code would not even work at all with 0

16:22 <Ke> got to learn this perf stuff, systemtap is always broken on new kernels

16:25 <Ke> how do I refer to code in commit message, do people kill me, if I post elixir links?

16:45 <Ke> Esmil: https://users.aalto.fi/~jkarlson/files/0001-mfd-cros-ec-copy-the-whole-event-in-get_next_event_x.patch

16:52 <maz> Ke: add a 'Fixes: 57e94c8b974d ("mfd: cros-ec: Increase maximum mkbp event size")' tag to that patch.

16:53 <Ke> what does tag mean?

16:53 <Ke> just a line in commit message?

16:54 <maz> Yes, just above to your Signed-off-by: line.

16:55 <Ke> can I edit the patch in place?

16:55 <Ke> I guess it's just text

16:55 <maz> "git commit --amend" is the best way. otherwise, just edit the generated patch.

16:57 <Ke> added now

17:11 <Ke> can someone explain, why my in reply to failed

21:39 <Ke> can someone advise, how is that alsa supposed to work on kevit

21:39 <Ke> Kevin

22:46 <sphalerite> Ke: https://archlinuxarm.org/platforms/armv8/rockchip/samsung-chromebook-plus#wiki the UCM thing makes it work at least on bob