Ping! > -----Original Message----- > From: Sai Krishna Potthuri <lakshmis@xxxxxxxxxx> > Sent: Tuesday, May 11, 2021 6:08 PM > To: Andy Shevchenko <andy.shevchenko@xxxxxxxxx> > Cc: Linus Walleij <linus.walleij@xxxxxxxxxx>; Rob Herring > <robh+dt@xxxxxxxxxx>; Michal Simek <michals@xxxxxxxxxx>; Greg Kroah- > Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>; linux-arm Mailing List <linux-arm- > kernel@xxxxxxxxxxxxxxxxxxx>; Linux Kernel Mailing List <linux- > kernel@xxxxxxxxxxxxxxx>; devicetree <devicetree@xxxxxxxxxxxxxxx>; open > list:GPIO SUBSYSTEM <linux-gpio@xxxxxxxxxxxxxxx>; git <git@xxxxxxxxxx>; > saikrishna12468@xxxxxxxxx > Subject: RE: [PATCH v6 3/3] pinctrl: Add Xilinx ZynqMP pinctrl driver support > > Hi Andy Shevchenko, > > > -----Original Message----- > > From: Sai Krishna Potthuri > > Sent: Wednesday, April 28, 2021 11:04 AM > > To: Andy Shevchenko <andy.shevchenko@xxxxxxxxx> > > Cc: Linus Walleij <linus.walleij@xxxxxxxxxx>; Rob Herring > > <robh+dt@xxxxxxxxxx>; Michal Simek <michals@xxxxxxxxxx>; Greg Kroah- > > Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>; linux-arm Mailing List > > <linux-arm- kernel@xxxxxxxxxxxxxxxxxxx>; Linux Kernel Mailing List > > <linux- kernel@xxxxxxxxxxxxxxx>; devicetree > > <devicetree@xxxxxxxxxxxxxxx>; open list:GPIO SUBSYSTEM > > <linux-gpio@xxxxxxxxxxxxxxx>; git <git@xxxxxxxxxx>; > > saikrishna12468@xxxxxxxxx > > Subject: RE: [PATCH v6 3/3] pinctrl: Add Xilinx ZynqMP pinctrl driver > > support > > > > Hi Andy Shevchenko, > > > > > -----Original Message----- > > > From: Andy Shevchenko <andy.shevchenko@xxxxxxxxx> > > > Sent: Monday, April 26, 2021 7:35 PM > > > To: Sai Krishna Potthuri <lakshmis@xxxxxxxxxx> > > > Cc: Linus Walleij <linus.walleij@xxxxxxxxxx>; Rob Herring > > > <robh+dt@xxxxxxxxxx>; Michal Simek <michals@xxxxxxxxxx>; Greg Kroah- > > > Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>; linux-arm Mailing List > > > <linux-arm- kernel@xxxxxxxxxxxxxxxxxxx>; Linux Kernel Mailing List > > > <linux- kernel@xxxxxxxxxxxxxxx>; devicetree > > > <devicetree@xxxxxxxxxxxxxxx>; open list:GPIO SUBSYSTEM > > > <linux-gpio@xxxxxxxxxxxxxxx>; git <git@xxxxxxxxxx>; > > > saikrishna12468@xxxxxxxxx > > > Subject: Re: [PATCH v6 3/3] pinctrl: Add Xilinx ZynqMP pinctrl > > > driver > > support > ... > > > > > > > > > + ret = zynqmp_pm_pinctrl_get_config(pin, param, &arg); > > > > > > + if (arg != PM_PINCTRL_BIAS_PULL_UP) > > > > > > + return -EINVAL; > > > > > > > > > > Error code being shadowed. Instead check it here properly. > > > > > > > Are you mentioning the case where ret is also a non-zero? > > > > If yes, then I will update this check to if (!ret && arg != > > > > PM_PINCTRL_BIAS_PULL_UP) > > > > return -EINVAL; > > > > > > No, this is wrong in the same way. > > > > > > > ret non-zero case, we are handling at the end of switch case. > > > > > > I meant that you need to pass the real return code to the (upper) caller. > > Here we are checking for valid argument and not the return value of the > API. > > If the read value(argument) is not valid and return value of the API > > is zero (SUCCESS) then framework expects driver to be returned with > > '-EINVAL' and it is a legal error code in this case. > Do you agree on this? > I am ready with the other changes, will send out the patch to address your > comments. > > Regards > Sai Krishna
