Re: [PATCH 0/2] staging: mt7621-pinctrl: use pinconf-generic in some driver ops

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jan 3, 2019 at 3:02 AM NeilBrown <neil@xxxxxxxxxx> wrote:
>
> On Mon, Dec 31 2018, Sergio Paracuellos wrote:
>
> > dt_node_to_map and dt_free_map operations can use pinconf-generic API's
> > instead of redefine operations in the driver. Make use of them cleaning
> > a bit driver's code.
> >
> > Update DT accordly to make sure used bindings property in code match
> > with the board's DT bindings.
> >
> > Changes are only compile-tested.
>
> Thanks.  This appears to work for me.
> It is awkward to test pinctrl changes because the firmware preconfigures
> all the pins, so the hardware will work correctly if pinctrl does
> nothing.
> So I change the dts file to mis-configure some pins for driving LEDs,
> and checked that the LEDs broken.  The fixed the dts, and the LEDs
> started working again.
> I think that will have to do.
>
> Tested-by: NeilBrown <neil@xxxxxxxxxx>
>
> Thanks,
> NeilBrown

Cool! Thanks for testing and let me know.

So, I sent this patch which solves a problem:

http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/2018-December/130260.html

And this new patch is generated with that applied. Also this patch
solves the original problem
also and seems the way to go... Should be better to re-do this one
with all the fixed and reported by
tags added as a solution?

What do you prefer, Greg?

Thanks in advance.

Best regards,
    Sergio Paracuellos
>
>
> >
> > Hope this helps.
> >
> > Best regards,
> >     Sergio Paracuellos
> >
> > Sergio Paracuellos (2):
> >   staging: mt7621-pinctrl: use pinconf-generic for 'dt_node_to_map' and
> >     'dt_free_map'
> >   staging: mt7621-dts: use standard 'groups' instead of 'group' for
> >     pinctrl bindings
> >
> >  drivers/staging/mt7621-dts/gbpc1.dts          |  4 +-
> >  drivers/staging/mt7621-dts/mt7621.dtsi        | 24 +++++------
> >  drivers/staging/mt7621-pinctrl/Kconfig        |  1 +
> >  .../staging/mt7621-pinctrl/pinctrl-rt2880.c   | 42 ++-----------------
> >  4 files changed, 18 insertions(+), 53 deletions(-)
> >
> > --
> > 2.19.1
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux