Thread (3 messages) 3 messages, 2 authors, 2021-06-13

Re: [PATCH v1] gpio: mt7621: Assign base field in gpio_chip

From: Reto Schneider <hidden>
Date: 2021-06-13 15:51:36
Also in: linux-arm-kernel, linux-mediatek, lkml

Possibly related (same subject, not in this thread)

Hi Andy,

On 13.06.21 14:44, Andy Shevchenko wrote:
It smells like a fixing non-existing bug. Yes, I understand that there 
is some custom hardware with custom firmware which is using Linux kernel 
with some never upstreamed patches. On top of this sysfs interface is 
deprecated for already a few years and shouldn’t be considered as 1st 
class citizen.

That’s said, if there is no Fixes tag provided with clear point that _at 
some point_ it was like this in upstream, I would tend to NAK this patch.
I did some digging and indeed, it turned out that I used the 
pre-mainline GPIO driver from OpenWRT even on Kernel 4.19 (for which the 
current driver already got mainlined).

Surely do agree on not breaking the interface in mainline, and the NAK.

Sorry for wasting your time!

Kind regards,
Reto
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help