Re: [PATCH v4 3/7] platform: cznic: turris-omnia-mcu: Add support for MCU connected GPIOs

[Marek Behún <kabel@xxxxxxxxxx>]

On Fri, 08 Dec 2023 15:26:02 +0100
Gregory CLEMENT <gregory.clement@xxxxxxxxxxx> wrote:

> Hello Marek,
> 
> > On Thu, 26 Oct 2023 18:17:59 +0200
> > Marek Behún <kabel@xxxxxxxxxx> wrote:
> >  
> >> +	mcu->gc.request = omnia_gpio_request;
> >> +	mcu->gc.get_direction = omnia_gpio_get_direction;
> >> +	mcu->gc.direction_input = omnia_gpio_direction_input;
> >> +	mcu->gc.direction_output = omnia_gpio_direction_output;
> >> +	mcu->gc.get = omnia_gpio_get;
> >> +	mcu->gc.get_multiple = omnia_gpio_get_multiple;
> >> +	mcu->gc.set = omnia_gpio_set;
> >> +	mcu->gc.set_multiple = omnia_gpio_set_multiple;
> >> +	mcu->gc.init_valid_mask = omnia_gpio_init_valid_mask;
> >> +	mcu->gc.can_sleep = true;
> >> +	mcu->gc.names = omnia_mcu_gpio_templates;
> >> +	mcu->gc.base = -1;
> >> +	mcu->gc.ngpio = ARRAY_SIZE(omnia_gpios);
> >> +	mcu->gc.label = "Turris Omnia MCU GPIOs";
> >> +	mcu->gc.parent = dev;
> >> +	mcu->gc.owner = THIS_MODULE;  
> >
> > Sigh. I will need to add a custom of_xlate here that accepts 3 cells
> > instead of 2. The reason is that Pali prepared U-Boot code last year to
> > patch reset-gpios into pcie nodes if MCU node is present in the
> > devicetree, and back then we decided that the binding should be
> > 3-celled:
> >   reset-gpios = <&mcu BANK GPIO FLAGS>;
> >
> > So now the pcie controllers do not work because OF cannot give proper
> > GPIOs.
> >
> > I will fix this in v5.  
> 
> I was reviewing the seris in order to apply it, but I didn't find your
> v5 in the mainling list. Did you send it ?

Hi Gregory, no I haven't yet. I was on a vacation and before I had to
work on some other issues. Hopefully I will be able to finish it within
the month. I will ping you then.

Thank you.

Marek