Re: [PATCH v2 1/2] Staging: fsl-mc: include: mc-bus: Kernel type 's16' preferred over 'int16_t'
On Thu, Nov 10, 2016 at 01:33:10PM +0100, Shiva Kerdel wrote: > Sorry for this misunderstanding, > > Since received a response from Stuart on my previous patch telling me I was > probably working on a out-of-date codebase, > I started to look where I did go wrong. > > After some research I found out that I was always one step behind because I > was developing on the actual Linux kernel branch instead > of using the Staging-next as my codebase. > > He wrote me to sent the patches that still apply over again with a new > version tag so that's why this patch only contains 2 of them. > > I am still new at contributing to the Linux kernel and some things were not > really clear to me at the beginning. > I hope that you could still use these patches and I am trying to avoid > misunderstandings like these. > > Do I still need to resend them and is there anything left for me to do with > these? Please resend, with the acks added, as these are gone from my patch queue. thanks, greg k-h ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH v2 1/2] Staging: fsl-mc: include: mc-bus: Kernel type 's16' preferred over 'int16_t'
Sorry for this misunderstanding, Since received a response from Stuart on my previous patch telling me I was probably working on a out-of-date codebase, I started to look where I did go wrong. After some research I found out that I was always one step behind because I was developing on the actual Linux kernel branch instead of using the Staging-next as my codebase. He wrote me to sent the patches that still apply over again with a new version tag so that's why this patch only contains 2 of them. I am still new at contributing to the Linux kernel and some things were not really clear to me at the beginning. I hope that you could still use these patches and I am trying to avoid misunderstandings like these. Do I still need to resend them and is there anything left for me to do with these? Best regards, Shiva Kerdel On 11/10/16 13:24, Greg KH wrote: On Tue, Nov 08, 2016 at 04:42:13PM +0100, Shiva Kerdel wrote: Follow the kernel type preferrences of using 's16' over 'int16_t'. Signed-off-by: Shiva Kerdel--- drivers/staging/fsl-mc/include/mc-bus.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) What is different from your v1 patch? I'm confused now, I have two different series from you, one 6 patches long, and one 2 patches long, claiming to do the same thing. I've dropped them both from my patch review queue and please, resend what you really want me to apply :) thanks, greg k-h ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH v2 1/2] Staging: fsl-mc: include: mc-bus: Kernel type 's16' preferred over 'int16_t'
On Tue, Nov 08, 2016 at 04:42:13PM +0100, Shiva Kerdel wrote: > Follow the kernel type preferrences of using 's16' over 'int16_t'. > > Signed-off-by: Shiva Kerdel> --- > drivers/staging/fsl-mc/include/mc-bus.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) What is different from your v1 patch? I'm confused now, I have two different series from you, one 6 patches long, and one 2 patches long, claiming to do the same thing. I've dropped them both from my patch review queue and please, resend what you really want me to apply :) thanks, greg k-h ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
RE: [PATCH v2 1/2] Staging: fsl-mc: include: mc-bus: Kernel type 's16' preferred over 'int16_t'
> -Original Message- > From: Shiva Kerdel [mailto:sh...@exdev.nl] > Sent: Tuesday, November 08, 2016 9:42 AM > To: Stuart Yoder> Cc: gre...@linuxfoundation.org; german.riv...@freescale.com; German Rivera > ; > tred...@nvidia.com; itai.k...@nxp.com; Nipun Gupta ; > linux-ker...@vger.kernel.org; > de...@driverdev.osuosl.org; Shiva Kerdel > Subject: [PATCH v2 1/2] Staging: fsl-mc: include: mc-bus: Kernel type 's16' > preferred over 'int16_t' > > Follow the kernel type preferrences of using 's16' over 'int16_t'. > > Signed-off-by: Shiva Kerdel > --- > drivers/staging/fsl-mc/include/mc-bus.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/staging/fsl-mc/include/mc-bus.h > b/drivers/staging/fsl-mc/include/mc-bus.h > index e915574..c7cad87 100644 > --- a/drivers/staging/fsl-mc/include/mc-bus.h > +++ b/drivers/staging/fsl-mc/include/mc-bus.h > @@ -42,8 +42,8 @@ struct msi_domain_info; > */ > struct fsl_mc_resource_pool { > enum fsl_mc_pool_type type; > - int16_t max_count; > - int16_t free_count; > + s16 max_count; > + s16 free_count; > struct mutex mutex; /* serializes access to free_list */ > struct list_head free_list; > struct fsl_mc_bus *mc_bus; Acked-by: Stuart Yoder ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel