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 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


>
> 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

Attachment: signature.asc
Description: PGP signature

_______________________________________________
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