LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [GIT_PULL] SOC: TI Keystone Ring Accelerator driver for v5.6
@ 2020-01-16 20:07 Santosh Shilimkar
  2020-01-17  0:03 ` Olof Johansson
  0 siblings, 1 reply; 6+ messages in thread
From: Santosh Shilimkar @ 2020-01-16 20:07 UTC (permalink / raw)
  To: soc, arm, linux-arm-kernel
  Cc: khilman, arnd, olof, linux-kernel, santosh.shilimkar, vkoul

Its bit late for pull request, but if possible, please pull it to
soc drivers tree.

The following changes since commit e42617b825f8073569da76dc4510bfa019b1c35a:

  Linux 5.5-rc1 (2019-12-08 14:57:55 -0800)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/ssantosh/linux-keystone.git tags/drivers_soc_for_5.6

for you to fetch changes up to 3277e8aa2504d97e022ecb9777d784ac1a439d36:

  soc: ti: k3: add navss ringacc driver (2020-01-15 10:07:27 -0800)

----------------------------------------------------------------
SOC: TI Keystone Ring Accelerator driver

The Ring Accelerator (RINGACC or RA) provides hardware acceleration to
enable straightforward passing of work between a producer and a consumer.
There is one RINGACC module per NAVSS on TI AM65x SoCs.

----------------------------------------------------------------
Grygorii Strashko (2):
      bindings: soc: ti: add documentation for k3 ringacc
      soc: ti: k3: add navss ringacc driver

 .../devicetree/bindings/soc/ti/k3-ringacc.txt      |   59 +
 drivers/soc/ti/Kconfig                             |   11 +
 drivers/soc/ti/Makefile                            |    1 +
 drivers/soc/ti/k3-ringacc.c                        | 1157 ++++++++++++++++++++
 include/linux/soc/ti/k3-ringacc.h                  |  244 +++++
 5 files changed, 1472 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/soc/ti/k3-ringacc.txt
 create mode 100644 drivers/soc/ti/k3-ringacc.c
 create mode 100644 include/linux/soc/ti/k3-ringacc.h

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [GIT_PULL] SOC: TI Keystone Ring Accelerator driver for v5.6
  2020-01-16 20:07 [GIT_PULL] SOC: TI Keystone Ring Accelerator driver for v5.6 Santosh Shilimkar
@ 2020-01-17  0:03 ` Olof Johansson
  2020-01-17  5:05   ` santosh.shilimkar
  0 siblings, 1 reply; 6+ messages in thread
From: Olof Johansson @ 2020-01-17  0:03 UTC (permalink / raw)
  To: Santosh Shilimkar
  Cc: soc, arm, linux-arm-kernel, khilman, arnd, linux-kernel, vkoul

Hi,

On Thu, Jan 16, 2020 at 12:07:39PM -0800, Santosh Shilimkar wrote:
> Its bit late for pull request, but if possible, please pull it to
> soc drivers tree.
> 
> The following changes since commit e42617b825f8073569da76dc4510bfa019b1c35a:
> 
>   Linux 5.5-rc1 (2019-12-08 14:57:55 -0800)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/ssantosh/linux-keystone.git tags/drivers_soc_for_5.6
> 
> for you to fetch changes up to 3277e8aa2504d97e022ecb9777d784ac1a439d36:
> 
>   soc: ti: k3: add navss ringacc driver (2020-01-15 10:07:27 -0800)
> 
> ----------------------------------------------------------------
> SOC: TI Keystone Ring Accelerator driver
> 
> The Ring Accelerator (RINGACC or RA) provides hardware acceleration to
> enable straightforward passing of work between a producer and a consumer.
> There is one RINGACC module per NAVSS on TI AM65x SoCs.

This driver doesn't seem to have exported symbols, and no in-kernel
users. So how will it be used?

Usually we ask to hold off until the consuming side/drivers are also ready.

Also, is there a reason this is under drivers/soc/ instead of somewhere more
suitable in the drivers subsystem? It's not "soc glue code" in the same way as
drivers/soc was intended originally.


-Olof

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [GIT_PULL] SOC: TI Keystone Ring Accelerator driver for v5.6
  2020-01-17  0:03 ` Olof Johansson
@ 2020-01-17  5:05   ` santosh.shilimkar
  2020-01-17  5:29     ` Vinod Koul
  2020-01-17 18:11     ` Olof Johansson
  0 siblings, 2 replies; 6+ messages in thread
From: santosh.shilimkar @ 2020-01-17  5:05 UTC (permalink / raw)
  To: Olof Johansson
  Cc: soc, arm, linux-arm-kernel, khilman, arnd, linux-kernel, vkoul

On 1/16/20 4:03 PM, Olof Johansson wrote:
> Hi,
> 
> On Thu, Jan 16, 2020 at 12:07:39PM -0800, Santosh Shilimkar wrote:
>> Its bit late for pull request, but if possible, please pull it to
>> soc drivers tree.
>>
>> The following changes since commit e42617b825f8073569da76dc4510bfa019b1c35a:
>>
>>    Linux 5.5-rc1 (2019-12-08 14:57:55 -0800)
>>
>> are available in the git repository at:
>>
>>    git://git.kernel.org/pub/scm/linux/kernel/git/ssantosh/linux-keystone.git tags/drivers_soc_for_5.6
>>
>> for you to fetch changes up to 3277e8aa2504d97e022ecb9777d784ac1a439d36:
>>
>>    soc: ti: k3: add navss ringacc driver (2020-01-15 10:07:27 -0800)
>>
>> ----------------------------------------------------------------
>> SOC: TI Keystone Ring Accelerator driver
>>
>> The Ring Accelerator (RINGACC or RA) provides hardware acceleration to
>> enable straightforward passing of work between a producer and a consumer.
>> There is one RINGACC module per NAVSS on TI AM65x SoCs.
> 
> This driver doesn't seem to have exported symbols, and no in-kernel
> users. So how will it be used?
> 
> Usually we ask to hold off until the consuming side/drivers are also ready.
> 
The other patches getting merged via Vinod's tree. The combined series
is split into couple of series. Vinod is going to pull this branch
and apply rest of the patchset. And then couple of additional consumer
drivers will get posted.

> Also, is there a reason this is under drivers/soc/ instead of somewhere more
> suitable in the drivers subsystem? It's not "soc glue code" in the same way as
> drivers/soc was intended originally.
> 
These kind of SOC IP drivers, we put into drivers/soc/ because of lack
of specific subsystem where they fit in. Navigator was also similar example.

Regards,
Santosh

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [GIT_PULL] SOC: TI Keystone Ring Accelerator driver for v5.6
  2020-01-17  5:05   ` santosh.shilimkar
@ 2020-01-17  5:29     ` Vinod Koul
  2020-01-17 18:11     ` Olof Johansson
  1 sibling, 0 replies; 6+ messages in thread
From: Vinod Koul @ 2020-01-17  5:29 UTC (permalink / raw)
  To: santosh.shilimkar, Olof Johansson
  Cc: Peter Ujfalusi, soc, arm, linux-arm-kernel, khilman, arnd, linux-kernel

Hi,

Add Peter

On 16-01-20, 21:05, santosh.shilimkar@oracle.com wrote:
> On 1/16/20 4:03 PM, Olof Johansson wrote:
> > Hi,
> > 
> > On Thu, Jan 16, 2020 at 12:07:39PM -0800, Santosh Shilimkar wrote:
> > > Its bit late for pull request, but if possible, please pull it to
> > > soc drivers tree.
> > > 
> > > The following changes since commit e42617b825f8073569da76dc4510bfa019b1c35a:
> > > 
> > >    Linux 5.5-rc1 (2019-12-08 14:57:55 -0800)
> > > 
> > > are available in the git repository at:
> > > 
> > >    git://git.kernel.org/pub/scm/linux/kernel/git/ssantosh/linux-keystone.git tags/drivers_soc_for_5.6
> > > 
> > > for you to fetch changes up to 3277e8aa2504d97e022ecb9777d784ac1a439d36:
> > > 
> > >    soc: ti: k3: add navss ringacc driver (2020-01-15 10:07:27 -0800)
> > > 
> > > ----------------------------------------------------------------
> > > SOC: TI Keystone Ring Accelerator driver
> > > 
> > > The Ring Accelerator (RINGACC or RA) provides hardware acceleration to
> > > enable straightforward passing of work between a producer and a consumer.
> > > There is one RINGACC module per NAVSS on TI AM65x SoCs.
> > 
> > This driver doesn't seem to have exported symbols, and no in-kernel
> > users. So how will it be used?
> > 
> > Usually we ask to hold off until the consuming side/drivers are also ready.
> > 
> The other patches getting merged via Vinod's tree. The combined series
> is split into couple of series. Vinod is going to pull this branch
> and apply rest of the patchset. And then couple of additional consumer
> drivers will get posted.

Yeah the TI driver series has been reviewed and was 'waiting' for
dependency to be resolved before I could apply them
FWIW here is the series under consideration: https://lore.kernel.org/dmaengine/20191223110458.30766-1-peter.ujfalusi@ti.com/

> > Also, is there a reason this is under drivers/soc/ instead of somewhere more
> > suitable in the drivers subsystem? It's not "soc glue code" in the same way as
> > drivers/soc was intended originally.
> > 
> These kind of SOC IP drivers, we put into drivers/soc/ because of lack
> of specific subsystem where they fit in. Navigator was also similar example.
> 
> Regards,
> Santosh

-- 
~Vinod

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [GIT_PULL] SOC: TI Keystone Ring Accelerator driver for v5.6
  2020-01-17  5:05   ` santosh.shilimkar
  2020-01-17  5:29     ` Vinod Koul
@ 2020-01-17 18:11     ` Olof Johansson
  2020-01-17 18:20       ` santosh.shilimkar
  1 sibling, 1 reply; 6+ messages in thread
From: Olof Johansson @ 2020-01-17 18:11 UTC (permalink / raw)
  To: Santosh Shilimkar
  Cc: SoC Team, ARM-SoC Maintainers, Linux ARM Mailing List,
	Kevin Hilman, Arnd Bergmann, Linux Kernel Mailing List,
	Vinod Koul

On Thu, Jan 16, 2020 at 9:05 PM <santosh.shilimkar@oracle.com> wrote:
>
> On 1/16/20 4:03 PM, Olof Johansson wrote:
> > Hi,
> >
> > On Thu, Jan 16, 2020 at 12:07:39PM -0800, Santosh Shilimkar wrote:
> >> Its bit late for pull request, but if possible, please pull it to
> >> soc drivers tree.
> >>
> >> The following changes since commit e42617b825f8073569da76dc4510bfa019b1c35a:
> >>
> >>    Linux 5.5-rc1 (2019-12-08 14:57:55 -0800)
> >>
> >> are available in the git repository at:
> >>
> >>    git://git.kernel.org/pub/scm/linux/kernel/git/ssantosh/linux-keystone.git tags/drivers_soc_for_5.6
> >>
> >> for you to fetch changes up to 3277e8aa2504d97e022ecb9777d784ac1a439d36:
> >>
> >>    soc: ti: k3: add navss ringacc driver (2020-01-15 10:07:27 -0800)
> >>
> >> ----------------------------------------------------------------
> >> SOC: TI Keystone Ring Accelerator driver
> >>
> >> The Ring Accelerator (RINGACC or RA) provides hardware acceleration to
> >> enable straightforward passing of work between a producer and a consumer.
> >> There is one RINGACC module per NAVSS on TI AM65x SoCs.
> >
> > This driver doesn't seem to have exported symbols, and no in-kernel
> > users. So how will it be used?
> >
> > Usually we ask to hold off until the consuming side/drivers are also ready.
> >
> The other patches getting merged via Vinod's tree. The combined series
> is split into couple of series. Vinod is going to pull this branch
> and apply rest of the patchset. And then couple of additional consumer
> drivers will get posted.

Ok -- might have been useful to get that in the tag description for
context. Something to consider next time.

> > Also, is there a reason this is under drivers/soc/ instead of somewhere more
> > suitable in the drivers subsystem? It's not "soc glue code" in the same way as
> > drivers/soc was intended originally.
> >
> These kind of SOC IP drivers, we put into drivers/soc/ because of lack
> of specific subsystem where they fit in. Navigator was also similar example.

Hmm. At some point we'll have to push the brakes on this, since
drivers/soc can't become a catch-all for random stuff like the old
mach directories were. But it's tricky to tell just when -- sometimes
you have to let the mess show up too.

I'll merge this when I do the next pass (today, likely).

-Olof

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [GIT_PULL] SOC: TI Keystone Ring Accelerator driver for v5.6
  2020-01-17 18:11     ` Olof Johansson
@ 2020-01-17 18:20       ` santosh.shilimkar
  0 siblings, 0 replies; 6+ messages in thread
From: santosh.shilimkar @ 2020-01-17 18:20 UTC (permalink / raw)
  To: Olof Johansson
  Cc: SoC Team, ARM-SoC Maintainers, Linux ARM Mailing List,
	Kevin Hilman, Arnd Bergmann, Linux Kernel Mailing List,
	Vinod Koul

On 1/17/20 10:11 AM, Olof Johansson wrote:
> On Thu, Jan 16, 2020 at 9:05 PM <santosh.shilimkar@oracle.com> wrote:
>>
>> On 1/16/20 4:03 PM, Olof Johansson wrote:
>>> Hi,
>>>
>>> On Thu, Jan 16, 2020 at 12:07:39PM -0800, Santosh Shilimkar wrote:
>>>> Its bit late for pull request, but if possible, please pull it to
>>>> soc drivers tree.
>>>>
>>>> The following changes since commit e42617b825f8073569da76dc4510bfa019b1c35a:
>>>>
>>>>     Linux 5.5-rc1 (2019-12-08 14:57:55 -0800)
>>>>
>>>> are available in the git repository at:
>>>>
>>>>     git://git.kernel.org/pub/scm/linux/kernel/git/ssantosh/linux-keystone.git tags/drivers_soc_for_5.6
>>>>
>>>> for you to fetch changes up to 3277e8aa2504d97e022ecb9777d784ac1a439d36:
>>>>
>>>>     soc: ti: k3: add navss ringacc driver (2020-01-15 10:07:27 -0800)
>>>>
>>>> ----------------------------------------------------------------
>>>> SOC: TI Keystone Ring Accelerator driver
>>>>
>>>> The Ring Accelerator (RINGACC or RA) provides hardware acceleration to
>>>> enable straightforward passing of work between a producer and a consumer.
>>>> There is one RINGACC module per NAVSS on TI AM65x SoCs.
>>>
>>> This driver doesn't seem to have exported symbols, and no in-kernel
>>> users. So how will it be used?
>>>
>>> Usually we ask to hold off until the consuming side/drivers are also ready.
>>>
>> The other patches getting merged via Vinod's tree. The combined series
>> is split into couple of series. Vinod is going to pull this branch
>> and apply rest of the patchset. And then couple of additional consumer
>> drivers will get posted.
> 
> Ok -- might have been useful to get that in the tag description for
> context. Something to consider next time.
> 
Sure. Will keep that in mind.

>>> Also, is there a reason this is under drivers/soc/ instead of somewhere more
>>> suitable in the drivers subsystem? It's not "soc glue code" in the same way as
>>> drivers/soc was intended originally.
>>>
>> These kind of SOC IP drivers, we put into drivers/soc/ because of lack
>> of specific subsystem where they fit in. Navigator was also similar example.
> 
> Hmm. At some point we'll have to push the brakes on this, since
> drivers/soc can't become a catch-all for random stuff like the old
> mach directories were. But it's tricky to tell just when -- sometimes
> you have to let the mess show up too.
> 
> I'll merge this when I do the next pass (today, likely).
> 
Thanks Olof !!

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2020-01-17 18:20 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-16 20:07 [GIT_PULL] SOC: TI Keystone Ring Accelerator driver for v5.6 Santosh Shilimkar
2020-01-17  0:03 ` Olof Johansson
2020-01-17  5:05   ` santosh.shilimkar
2020-01-17  5:29     ` Vinod Koul
2020-01-17 18:11     ` Olof Johansson
2020-01-17 18:20       ` santosh.shilimkar

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).