12:59 <gbisson> ndufresne: FYI I tried your v4l2videodec series about dynamic resolution change and it didn't fix the VFIFO issue but it might be because of my test env (debian bullseye with gst1.18 whereas your series was made on 1.19) so I'll double check on a more recent distro

13:10 <gbisson> narmstrong: hi, regarding the mmc clk phase, is the patch a wip because aml changes the phase depending on the speed?

14:12 <gbisson> narmstrong: any guidlines for adding combinations to the pinctrl driver? By that I mean how do you decide the order? For instance for pwm_f why is the order "GPIOX GPIOH", is it to match the doc order?

14:16 <ndufresne> gbisson: ack

14:30 <narmstrong> gbisson: it’s wip because the actual solution would need more work to implement yes

14:31 <narmstrong> gbisson: i don’t understand your question about pinctrl combinations

14:36 <gbisson> narmstrong: I'm trying to understand the order of the different options, for pwm_f for instance it is not alphabetical order nor pin number order, my guess is it is in doc order

14:36 <narmstrong> gbisson: do you mean https://github.com/torvalds/linux/blob/master/drivers/pinctrl/meson/pinctrl-meson-g12a.c ?

14:36 <narmstrong> line 1071 ?

14:37 <gbisson> yes: https://github.com/torvalds/linux/blob/master/drivers/pinctrl/meson/pinctrl-meson-g12a.c#L285

14:37 <narmstrong> gbisson: order isn't important at all

14:37 <gbisson> why is it x first, then h? because I want to add z and a options

14:38 <gbisson> ok, just making sure if there was any logic behind it

14:38 <narmstrong> the only thing important is the meson_g12a_periphs_groups mapping

14:39 <gbisson> ok, my patch is working right now, I can use the pwm_f on another pin so I know it is working, it's just that I wanted to check the order before submission, I'll try to do that today

17:21 <gbisson> narmstrong: for AOSP, GKI isn't used because uboot isn't capable to merging the ramdisk from vendor_boot yet correct?

17:22 <narmstrong> gbisson: I haven't followed latest GKI requirements, this seems new to me

17:23 <gbisson> narmstrong: ok no problem I'll look into it later, current way to build should be ok to get started anyway

17:25 <narmstrong> gbisson: for kernel, GKI would need an effort to either: add meas support to yukawa to use panfrost OR build kbase arm driver and list all required API for this driver

18:04 <gbisson> narmstrong: oh ok, I'm surprised mali isn't supported in the standard kmi, anyway it's fairly easy to update that list, I thought the limitation was coming from U-Boot and vendor_boot support