Re: [PATCH 1/2] amba: Add dev_is_amba() function and export it for modules

[Kunwu Chan]

Thanks for the reply.

On 2024/9/25 06:43, Russell King (Oracle) wrote:
> On Tue, Sep 24, 2024 at 05:28:57PM -0500, Rob Herring wrote:
> > On Mon, Sep 23, 2024 at 05:42:47PM +0800, Kunwu Chan wrote:
> > > Add dev_is_amba() function to determine
> > > whether the device is a AMBA device.
> > >
> > > Suggested-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> > > Signed-off-by: Kunwu Chan <chentao@xxxxxxxxxx>
> > > ---
> > >   drivers/amba/bus.c       | 6 ++++++
> > >   include/linux/amba/bus.h | 5 +++++
> > >   2 files changed, 11 insertions(+)
> > Russell, Can I get an ack for this to take it with patch #2.
> Would be nice to discuss "how shall we merge this cross-subsystem
> patch series" first, hmm?
>
> The reason I didn't take patch 1 originally is because it was submitted
> to me without any users, and the general principle is not to accept
> patches without users. Too many times, I've merged code where there's
> been a "promise" that it will be used, only to have the author go
> silent and users never come along. So now, my rule is... any code that
> adds something must also come with its user.
Just one user in drivers/of/platform.c right now.

Actually, I don't know if dev_is_amba will use it more in the future.

So we keep the usage of  "dev->bus == &platform_bus_type " too?
Although here come a wrapper function "dev_is_platform".

--
Thanks,
  Kunwu.Chan