[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ZthnjPjPB+6cMf9w@shell.armlinux.org.uk>
Date: Wed, 4 Sep 2024 14:58:36 +0100
From: "Russell King (Oracle)" <linux@...linux.org.uk>
To: Kunwu Chan <kunwu.chan@...ux.dev>
Cc: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>, krzk@...nel.org,
andi.shyti@...nel.org, robh@...nel.org, gregkh@...uxfoundation.org,
suzuki.poulose@....com, linux-kernel@...r.kernel.org,
Kunwu Chan <chentao@...inos.cn>
Subject: Re: [PATCH] amba: make amba_bustype constant
On Thu, Aug 29, 2024 at 09:31:29AM +0800, Kunwu Chan wrote:
> Thanks for the reply.
> On 2024/8/28 21:17, Andy Shevchenko wrote:
> > On Wed, Aug 28, 2024 at 10:51:54AM +0800, Kunwu Chan wrote:
> > > On 2024/8/27 21:37, Andy Shevchenko wrote:
> > > > On Tue, Aug 27, 2024 at 03:45:31PM +0800, Kunwu Chan wrote:
> > > > > On 2024/8/26 18:40, Andy Shevchenko wrote:
> > > > > > On Mon, Aug 26, 2024 at 06:08:11PM +0800, Kunwu Chan wrote:
> > ...
> >
> > > > > > Make it patch series:
> > > > > > 1) patch that introduces exported function called dev_is_amba() (1 patch)
> > > Done.
> > > > > > 2) convert user-by-user (N patches)
> > > I've no idea about how to modify, such as in iommu.c:
> > Oh, crap. Yes, this need more thinking.
> > Anyway, Russell is okay with your initial patch, the rest can be done
> > separately. For now probably we can leave it exported for this only case.
> >
> Anyway, thanks for the suggestion, i'll introduce the dev_is_amba function
>
> and add the Suggested tag for you.
>
> If you have a better idea and need me to do it, you can always contact me.
General policy is not to add stuff that doesn't have any users. From
what I can see from briefly reading this discussion, and looking at
the patches submitted to me, the dev_is_amba() patch adds a helper,
but as yet there are no users - and not even any patches on a mailing
list to make use of this helper. Therefore, I won't be applying that
patch.
Good idea, but it needs users...
Please note that I likely won't be reading further discussion (see
my signature below, and I'm having the same op on the other eye -
which is the only eye suitable for screen work at the moment -
this Friday.)
--
*** please note that I probably will only be occasionally responsive
*** for an unknown period of time due to recent eye surgery making
*** reading quite difficult.
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!
Powered by blists - more mailing lists