On Thu, Apr 16, 2020 at 09:51:37AM +0200, Linus Walleij wrote: > On Sun, Mar 29, 2020 at 12:46 PM Russell King - ARM Linux admin > <linux@xxxxxxxxxxxxxxx> wrote: > > > This series adds support for the fan PWM output on the Clearfog GT8K > > platform, and can potentially be extended to the Macchiatobin. > > The gpio changes all look fine to me +/- fixes for review comments. I think Uwe is incorrect for his GPIO comments; the clock is only optional on A8040. We know this because A8040 has worked fine without PWM support without the clock, whereas for Armada 370, the driver has hard-failed if the clock is not present. So, on Armada 370, I preserve this behaviour. I also preserve the behaviour that on Armada 8040, we don't fail the driver if the clock is not present so that booting a newer kernel with older DT still works (which is a requirement.) In that case, the driver today still tries to get the clock but never checks the result of getting the clock (which doesn't exist in current DT files.) So no, I'm not going to fix Uwe's comments and potentially introduce regressions into this GPIO driver; I gave up trying to argue the point with Uwe, and I'm at the point of not giving a damn about this patch set if I'm to intentionally introduce regressions based on review comments. About the only change I would make is to move the check introduced in patch 2 into patch 3 instead, inside the MVEBU_PWM_SOC_VARIANT_A8K case, so that deferring for the clock works (which is necessary for the PWM driver to be useful.) -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line in suburbia: sync at 10.2Mbps down 587kbps up
