Re: [PATCH v3 5/5] clk: samsung: add exynos7870 CLKOUT support

[David Virag <virag.david003@gmail.com>]

On Mon, 2025-02-24 at 20:13 +0100, Krzysztof Kozlowski wrote:
> On 24/02/2025 18:47, Kaustabh Chakraborty wrote:
> > > 
> > > I wonder why do we need to keep growing this list? All devices
> > > are
> > > compatible, aren't they?
> > 
> > Well, there are two variants of compatibility having different mask
> > values.
> > 
> > > Do you use clkout, BTW?
> > 
> > Using the clocks defined by clkout? No. I added it as downstream
> > had it too. And the devices work fine without it. If you want me
> > to remove this patch and send the PMU patch to its respective
> > series I'll do that then (unless you object or suggest something
> > else).
> 
> clkout is a testing tool and I doubt you can use it on a phone -
> finding
> the actual clkout pins to connect the oscilloscope is tricky.

Actually, jackpotlte uses one of the clkout pins (7885 has two) to feed
it's NFC chip with a clock. 

Not sure if any 7870 (or any other Exynos) devices use it for something
similar but here we have at least one example of a non-debug use for
it. I will avoid it on 7885 though until it's actually needed and I can
test the functionality (and when I have the time to work on 7885
again).

Best Regards,
David