Re: [PATCH] vc04_services: Fixing coding guideline error
On Wed, Mar 15, 2017 at 12:01:08PM +0530, Pushkar Jambhlekar wrote: > Fixing 'if' block coding style. '{' should follow 'if' for multiline block > > Signed-off-by: Pushkar Jambhlekar> --- > drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) Patch does not apply to my tree at all, please rebase and refresh and resend. thanks, greg k-h
Re: [PATCH] vc04_services: Fixing coding guideline error
On Wed, Mar 15, 2017 at 12:01:08PM +0530, Pushkar Jambhlekar wrote: > Fixing 'if' block coding style. '{' should follow 'if' for multiline block > > Signed-off-by: Pushkar Jambhlekar > --- > drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) Patch does not apply to my tree at all, please rebase and refresh and resend. thanks, greg k-h
Re: [PATCH] vc04_services: Fixing coding guideline error
On Thu, Mar 16, 2017 at 11:01:38AM +0530, Pushkar Jambhlekar wrote: > Any comment? > You're too impatient. Wait for 2 weeks before asking for responses. regards, dan carpenter
Re: [PATCH] vc04_services: Fixing coding guideline error
On Thu, Mar 16, 2017 at 11:01:38AM +0530, Pushkar Jambhlekar wrote: > Any comment? > You're too impatient. Wait for 2 weeks before asking for responses. regards, dan carpenter
Re: [PATCH] vc04_services: Fixing coding guideline error
On Thu, Mar 16, 2017 at 11:01:38AM +0530, Pushkar Jambhlekar wrote: > Any comment? > > On Wed, Mar 15, 2017 at 12:01 PM, Pushkar Jambhlekar One day's response? Hah, give me a few weeks at best for trivial stuff like this please. Actually, I totally missed this in my last sweep of staging patches, because you forgot to include "Staging" in the subject line, so it will have to wait a few more days at the earliest. thanks, greg k-h
Re: [PATCH] vc04_services: Fixing coding guideline error
On Thu, Mar 16, 2017 at 11:01:38AM +0530, Pushkar Jambhlekar wrote: > Any comment? > > On Wed, Mar 15, 2017 at 12:01 PM, Pushkar Jambhlekar One day's response? Hah, give me a few weeks at best for trivial stuff like this please. Actually, I totally missed this in my last sweep of staging patches, because you forgot to include "Staging" in the subject line, so it will have to wait a few more days at the earliest. thanks, greg k-h
Re: [PATCH] vc04_services: Fixing coding guideline error
Any comment? On Wed, Mar 15, 2017 at 12:01 PM, Pushkar Jambhlekarwrote: > Fixing 'if' block coding style. '{' should follow 'if' for multiline block > > Signed-off-by: Pushkar Jambhlekar > --- > drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c > b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c > index 48984ab..ca896a5 100644 > --- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c > +++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c > @@ -797,8 +797,7 @@ int32_t vchi_get_peer_version(const VCHI_SERVICE_HANDLE_T > handle, short *peer_ve > { > int32_t ret = -1; > SHIM_SERVICE_T *service = (SHIM_SERVICE_T *)handle; > - if (service) > - { > + if (service) { > VCHIQ_STATUS_T status; > > status = vchiq_get_peer_version(service->handle, > peer_version); > -- > 2.7.4 > -- Jambhlekar Pushkar Arun M.Tech IIT Roorkee
Re: [PATCH] vc04_services: Fixing coding guideline error
Any comment? On Wed, Mar 15, 2017 at 12:01 PM, Pushkar Jambhlekar wrote: > Fixing 'if' block coding style. '{' should follow 'if' for multiline block > > Signed-off-by: Pushkar Jambhlekar > --- > drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c > b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c > index 48984ab..ca896a5 100644 > --- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c > +++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c > @@ -797,8 +797,7 @@ int32_t vchi_get_peer_version(const VCHI_SERVICE_HANDLE_T > handle, short *peer_ve > { > int32_t ret = -1; > SHIM_SERVICE_T *service = (SHIM_SERVICE_T *)handle; > - if (service) > - { > + if (service) { > VCHIQ_STATUS_T status; > > status = vchiq_get_peer_version(service->handle, > peer_version); > -- > 2.7.4 > -- Jambhlekar Pushkar Arun M.Tech IIT Roorkee
[PATCH] vc04_services: Fixing coding guideline error
Fixing 'if' block coding style. '{' should follow 'if' for multiline block Signed-off-by: Pushkar Jambhlekar--- drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c index 48984ab..ca896a5 100644 --- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c +++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c @@ -797,8 +797,7 @@ int32_t vchi_get_peer_version(const VCHI_SERVICE_HANDLE_T handle, short *peer_ve { int32_t ret = -1; SHIM_SERVICE_T *service = (SHIM_SERVICE_T *)handle; - if (service) - { + if (service) { VCHIQ_STATUS_T status; status = vchiq_get_peer_version(service->handle, peer_version); -- 2.7.4
[PATCH] vc04_services: Fixing coding guideline error
Fixing 'if' block coding style. '{' should follow 'if' for multiline block Signed-off-by: Pushkar Jambhlekar --- drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c index 48984ab..ca896a5 100644 --- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c +++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c @@ -797,8 +797,7 @@ int32_t vchi_get_peer_version(const VCHI_SERVICE_HANDLE_T handle, short *peer_ve { int32_t ret = -1; SHIM_SERVICE_T *service = (SHIM_SERVICE_T *)handle; - if (service) - { + if (service) { VCHIQ_STATUS_T status; status = vchiq_get_peer_version(service->handle, peer_version); -- 2.7.4