Message ID | 1409865234-9932-1-git-send-email-santosh.shilimkar@ti.com |
---|---|
State | New |
Headers | show |
On Thu, Sep 04, 2014 at 05:13:54PM -0400, Santosh Shilimkar wrote: > Hi Arm-soc folks, > > Please pull below Keystone SOC drivers. They have been on the list for review > for last few merge windows. I got handful of comments on DTS bindings which > has been already addressed. This should ungate me to get network support for > Keystone which is missing in mainline. > > The following changes since commit 7d1311b93e58ed55f3a31cc8f94c4b8fe988a2b9: > > Linux 3.17-rc1 (2014-08-16 10:40:26 -0600) > > are available in the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/ssantosh/linux-keystone.git tags/drivers-soc-ti > > for you to fetch changes up to e9e7494bdcc7926237823253707691d0f3787be2: > > MAINTAINERS: Add Keystone Multicore Navigator drivers entry (2014-09-04 16:40:16 -0400) > > ---------------------------------------------------------------- > Keystone SOC Navigator drivers for 3.18 > > The Keystone Multi-core Navigator contains QMSS and packet DMA > subsystems which interwork together to form the Navigator cloud > used by various subsystems like NetCP, SRIO, SideBand Crypto > engines etc. > > ---------------------------------------------------------------- > Sandeep Nair (3): > firmware: add Keystone QMSS PDSP accumulator firmware blob > Documentation: dt: soc: add Keystone Navigator QMSS bindings > soc: ti: add Keystone Navigator QMSS driver > > Santosh Shilimkar (3): > Documentation: dt: soc: add Keystone Navigator DMA bindings > soc: ti: add Keystone Navigator DMA support > MAINTAINERS: Add Keystone Multicore Navigator drivers entry Hi Santosh, I was going through the queue of merges to do and it seems like Arnd missed this one. Before I merge it though -- it looks like the firmware should be added to the linux-firmware repository instead? We try to avoid adding new firmware files in the kernel tree these days and get them from there instead... Can you respin with that fixed, or should I cherry-pick the contents from this branch instead (all but that patch)? -Olof
On Wednesday 24 September 2014 12:43 AM, Olof Johansson wrote: > On Thu, Sep 04, 2014 at 05:13:54PM -0400, Santosh Shilimkar wrote: >> Hi Arm-soc folks, >> >> Please pull below Keystone SOC drivers. They have been on the list for review >> for last few merge windows. I got handful of comments on DTS bindings which >> has been already addressed. This should ungate me to get network support for >> Keystone which is missing in mainline. >> >> The following changes since commit 7d1311b93e58ed55f3a31cc8f94c4b8fe988a2b9: >> >> Linux 3.17-rc1 (2014-08-16 10:40:26 -0600) >> >> are available in the git repository at: >> >> git://git.kernel.org/pub/scm/linux/kernel/git/ssantosh/linux-keystone.git tags/drivers-soc-ti >> >> for you to fetch changes up to e9e7494bdcc7926237823253707691d0f3787be2: >> >> MAINTAINERS: Add Keystone Multicore Navigator drivers entry (2014-09-04 16:40:16 -0400) >> >> ---------------------------------------------------------------- >> Keystone SOC Navigator drivers for 3.18 >> >> The Keystone Multi-core Navigator contains QMSS and packet DMA >> subsystems which interwork together to form the Navigator cloud >> used by various subsystems like NetCP, SRIO, SideBand Crypto >> engines etc. >> >> ---------------------------------------------------------------- >> Sandeep Nair (3): >> firmware: add Keystone QMSS PDSP accumulator firmware blob >> Documentation: dt: soc: add Keystone Navigator QMSS bindings >> soc: ti: add Keystone Navigator QMSS driver >> >> Santosh Shilimkar (3): >> Documentation: dt: soc: add Keystone Navigator DMA bindings >> soc: ti: add Keystone Navigator DMA support >> MAINTAINERS: Add Keystone Multicore Navigator drivers entry > > Hi Santosh, > > I was going through the queue of merges to do and it seems like Arnd > missed this one. > > Before I merge it though -- it looks like the firmware should be added to > the linux-firmware repository instead? We try to avoid adding new firmware > files in the kernel tree these days and get them from there instead... > Right. Typically firmware files in kernel are not encoraged. This is infrastructure driver so the firmware is needed for it to work. I will try to get that patch via linux-firmware tree. > Can you respin with that fixed, or should I cherry-pick the contents from > this branch instead (all but that patch)? > Just refreshed the branch and updated the new tag. git://git.kernel.org/pub/scm/linux/kernel/git/ssantosh/linux-keystone.git tags/drivers-soc-ti-v2 Can you please pull that one ? Thanks !! regards, Santosh
On Wed, Sep 24, 2014 at 09:57:05AM -0400, Santosh Shilimkar wrote: > On Wednesday 24 September 2014 12:43 AM, Olof Johansson wrote: > > On Thu, Sep 04, 2014 at 05:13:54PM -0400, Santosh Shilimkar wrote: > >> Hi Arm-soc folks, > >> > >> Please pull below Keystone SOC drivers. They have been on the list for review > >> for last few merge windows. I got handful of comments on DTS bindings which > >> has been already addressed. This should ungate me to get network support for > >> Keystone which is missing in mainline. > >> > >> The following changes since commit 7d1311b93e58ed55f3a31cc8f94c4b8fe988a2b9: > >> > >> Linux 3.17-rc1 (2014-08-16 10:40:26 -0600) > >> > >> are available in the git repository at: > >> > >> git://git.kernel.org/pub/scm/linux/kernel/git/ssantosh/linux-keystone.git tags/drivers-soc-ti > >> > >> for you to fetch changes up to e9e7494bdcc7926237823253707691d0f3787be2: > >> > >> MAINTAINERS: Add Keystone Multicore Navigator drivers entry (2014-09-04 16:40:16 -0400) > >> > >> ---------------------------------------------------------------- > >> Keystone SOC Navigator drivers for 3.18 > >> > >> The Keystone Multi-core Navigator contains QMSS and packet DMA > >> subsystems which interwork together to form the Navigator cloud > >> used by various subsystems like NetCP, SRIO, SideBand Crypto > >> engines etc. > >> > >> ---------------------------------------------------------------- > >> Sandeep Nair (3): > >> firmware: add Keystone QMSS PDSP accumulator firmware blob > >> Documentation: dt: soc: add Keystone Navigator QMSS bindings > >> soc: ti: add Keystone Navigator QMSS driver > >> > >> Santosh Shilimkar (3): > >> Documentation: dt: soc: add Keystone Navigator DMA bindings > >> soc: ti: add Keystone Navigator DMA support > >> MAINTAINERS: Add Keystone Multicore Navigator drivers entry > > > > Hi Santosh, > > > > I was going through the queue of merges to do and it seems like Arnd > > missed this one. > > > > Before I merge it though -- it looks like the firmware should be added to > > the linux-firmware repository instead? We try to avoid adding new firmware > > files in the kernel tree these days and get them from there instead... > > > Right. Typically firmware files in kernel are not encoraged. This is infrastructure > driver so the firmware is needed for it to work. I will try to get that > patch via linux-firmware tree. > > > Can you respin with that fixed, or should I cherry-pick the contents from > > this branch instead (all but that patch)? > > > Just refreshed the branch and updated the new tag. > > git://git.kernel.org/pub/scm/linux/kernel/git/ssantosh/linux-keystone.git > tags/drivers-soc-ti-v2 > > Can you please pull that one ? Done, merged. -Olof
On Wednesday 24 September 2014 03:06 PM, Olof Johansson wrote: > Fixes below build break by not switching to stubs when the driver is a module: > > drivers/soc/ti/knav_dma.c:418:7: error: redefinition of 'knav_dma_open_channel' > void *knav_dma_open_channel(struct device *dev, const char *name, > ^ > In file included from drivers/soc/ti/knav_dma.c:26:0: > include/linux/soc/ti/knav_dma.h:165:21: note: previous definition of 'knav_dma_open_channel' was here > static inline void *knav_dma_open_channel(struct device *dev, const char *name, > ^ > > Cc: Santosh Shilimkar <santosh.shilimkar@ti.com> > Signed-off-by: Olof Johansson <olof@lixom.net> > --- > include/linux/soc/ti/knav_dma.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/linux/soc/ti/knav_dma.h b/include/linux/soc/ti/knav_dma.h > index e864a3e..dad035c 100644 > --- a/include/linux/soc/ti/knav_dma.h > +++ b/include/linux/soc/ti/knav_dma.h > @@ -157,7 +157,7 @@ struct knav_dma_desc { > u32 pad[4]; > } ____cacheline_aligned; > > -#ifdef CONFIG_KEYSTONE_NAVIGATOR_DMA > +#if IS_ENABLED(CONFIG_KEYSTONE_NAVIGATOR_DMA) > void *knav_dma_open_channel(struct device *dev, const char *name, > struct knav_dma_cfg *config); > void knav_dma_close_channel(void *channel); > Thanks Olof for fixing it up. Regards, Santosh