Re: [PATCH] ARM: dts: am33xx-l4: Add missing touchscreen clock properties

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

 



Hi Miquel,
some tiny typos...

> Am 07.03.2022 um 12:14 schrieb Miquel Raynal <miquel.raynal@xxxxxxxxxxx>:
> 
> When adding support for TI magadc, the MFD driver (common to the

"magadc"?

> touchscreen and the ADC) got updated to ease the insertion of a new DT
> node for the ADC, with its own compatible, clocks, etc. Commit
> 235a96e92c16 ("mfd: ti_am335x_tscadc: Don't search the tree for our
> clock") removed one compatible specific information which was the clock
> name, because the clock was looked up from scratch in the DT while this
> hardware block was only fed by a single clock, already defined and
> properly filled in the DT.
> 
> Problem is, this change was only validated with an am437x-based board,
> where the clocks are effectively correctly defined and referenced. But
> on am33xx, the ADC clock is also correctly defined but is not referenced
> with a clock phandle as it out to be.

maybe you mean "ought to be"?

> 
> The touchscreen bindings clearly state that the clocks/clock-names
> properties are mandatory, but they have been forgotten in one DTSI. This
> was probably not noticed in the first place because of the clock
> actually existing and the clk_get() call going through all the tree
> anyway.
> 
> Add the missing clock phandles in the am33xx touchscreen description.

Yes, makes touch on BeagleBoard Black with Chipsee 4"3 panel work again!

> 
> Reported-by: H. Nikolaus Schaller <hns@xxxxxxxxxxxxx>
> Fixes: 235a96e92c16 ("mfd: ti_am335x_tscadc: Don't search the tree for our clock")
Tested-by: H. Nikolaus Schaller <hns@xxxxxxxxxxxxx>
> 
> Signed-off-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx>
> ---
> 
> Hello Nikolaus, as I told you I don't have the relevant hardware to
> verify that this actually fixes your situation but I am rather
> confident. Could you please give this a try?
> Thanks! Miquel

> 
> arch/arm/boot/dts/am33xx-l4.dtsi | 2 ++
> 1 file changed, 2 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/am33xx-l4.dtsi b/arch/arm/boot/dts/am33xx-l4.dtsi
> index c9629cb5ccd1..7da42a5b959c 100644
> --- a/arch/arm/boot/dts/am33xx-l4.dtsi
> +++ b/arch/arm/boot/dts/am33xx-l4.dtsi
> @@ -263,6 +263,8 @@ tscadc: tscadc@0 {
> 				compatible = "ti,am3359-tscadc";
> 				reg = <0x0 0x1000>;
> 				interrupts = <16>;
> +				clocks = <&adc_tsc_fck>;
> +				clock-names = "fck";
> 				status = "disabled";
> 				dmas = <&edma 53 0>, <&edma 57 0>;
> 				dma-names = "fifo0", "fifo1";
> -- 
> 2.27.0
> 

BR and thanks,
Nikolaus






[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux