Re: protected pins and debugfs

[Sodagudi Prasad]

On 2018-10-07 23:04, Stephen Boyd wrote:
> Quoting Sodagudi Prasad (2018-10-03 05:38:24)
> >          for (i = 0; i < chip->ngpio; i++, gpio++) {
> > +               label = gpiochip_is_requested(chip, i);
> > +               if (!label)
> > +                       continue;
> >                  msm_gpio_dbg_show_one(s, NULL, chip, i, gpio);
> > -               seq_puts(s, "\n");
> >          }
> >   }
>
> Does something not work with the following code in
> msm_gpio_dbg_show_one()?
>
>
>         if (!gpiochip_line_is_valid(chip, offset))
> 		return;

Hi Stephen,
I didnt realize that these changes are merged on tip. I was testing on 
4.14 kernel.

https://lore.kernel.org/patchwork/patch/878107/
https://lore.kernel.org/patchwork/patch/878106/
https://lore.kernel.org/patchwork/patch/878109/

I will add "gpio-reserved-ranges" to internal platforms and this issue 
should not be observed.

-thanks, Prasad

--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora 
Forum,
Linux Foundation Collaborative Project