[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <b13f4181-7818-6780-2aaa-7966d5722780@marcan.st>
Date: Wed, 21 Dec 2022 21:52:16 +0900
From: Hector Martin <marcan@...can.st>
To: Uwe Kleine-König <u.kleine-koenig@...gutronix.de>,
Sasha Finkelstein <fnkl.kernel@...il.com>
Cc: thierry.reding@...il.com, robh+dt@...nel.org,
krzysztof.kozlowski+dt@...aro.org, sven@...npeter.dev,
alyssa@...enzweig.io, asahi@...ts.linux.dev,
linux-arm-kernel@...ts.infradead.org, linux-pwm@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v4 2/4] pwm: Add Apple PWM controller
On 19/12/2022 22.06, Uwe Kleine-König wrote:
> Hello,
>
> over all the driver looks good. Just a few smaller issues below.
>
> I wonder if it's a good idea to call this driver "apple". SoC vendors
> seem to reinvent their peripherals (or buy them somewhere else) for
> their different generations of processors. Maybe call it "apple-s5l"
> already today and not only when the next generation SoC appears?
> (I don't feel strong here, if you want to delay that renaming until
> there is an incompatible SoC that's fine for me.)
Well... considering s5l refers to the s5l8920x, the SoC used in the
iPhone 3GS released in 2009, and here we are with the M1 13 years later
and it's still the same, I think we're doing pretty well here.
(Apple doesn't reinvent their peripherals nearly as often as other vendors).
- Hector
Powered by blists - more mailing lists