On 11/30, Anjana Sasindran wrote: > This patch fixes the checkpatch.pl warnings: Your subject line is somewhat broken as it does not really follow the canonical patch format. You can read up on it here (section 15) https://www.kernel.org/doc/Documentation/SubmittingPatches and this is the short version: - it is not necessary to put the full path to the driver/file in the subject line. The patches that apply to the staging start with the 'staging' mark (so no need for the 'drivers') followed by the driver name ('octeon' in this case). - the file name is not needed in the subject as this information is in the diff anyway. - use the space after the colon ':' because otherwise it is hard to read. Hope this pays offs in the future. Thanks, Konrad > WARNING: Missing blank line after declaration > > Signed-off-by: Anjana Sasindran <anjanasasindran123@xxxxxxxxx> > --- > drivers/staging/octeon/ethernet-rx.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/staging/octeon/ethernet-rx.c b/drivers/staging/octeon/ethernet-rx.c > index 44e372f..61131ff 100644 > --- a/drivers/staging/octeon/ethernet-rx.c > +++ b/drivers/staging/octeon/ethernet-rx.c > @@ -84,6 +84,7 @@ static int cvm_irq_cpu; > static void cvm_oct_enable_napi(void *_) > { > int cpu = smp_processor_id(); > + > napi_schedule(&cvm_oct_napi[cpu].napi); > } > > @@ -169,6 +170,7 @@ static inline int cvm_oct_check_rcv_error(cvmx_wqe_t *work) > int interface = cvmx_helper_get_interface_num(work->ipprt); > int index = cvmx_helper_get_interface_index_num(work->ipprt); > union cvmx_gmxx_rxx_frm_ctl gmxx_rxx_frm_ctl; > + > gmxx_rxx_frm_ctl.u64 = > cvmx_read_csr(CVMX_GMXX_RXX_FRM_CTL(index, interface)); > if (gmxx_rxx_frm_ctl.s.pre_chk == 0) { > @@ -272,6 +274,7 @@ static int cvm_oct_napi_poll(struct napi_struct *napi, int budget) > did_work_request = 0; > if (work == NULL) { > union cvmx_pow_wq_int wq_int; > + > wq_int.u64 = 0; > wq_int.s.iq_dis = 1 << pow_receive_group; > wq_int.s.wq_int = 1 << pow_receive_group; > @@ -296,6 +299,7 @@ static int cvm_oct_napi_poll(struct napi_struct *napi, int budget) > union cvmx_pow_wq_int_cntx counts; > int backlog; > int cores_in_use = core_state.baseline_cores - atomic_read(&core_state.available_cores); > + > counts.u64 = cvmx_read_csr(CVMX_POW_WQ_INT_CNTX(pow_receive_group)); > backlog = counts.s.iq_cnt + counts.s.ds_cnt; > if (backlog > budget * cores_in_use && napi != NULL) > -- > 1.9.1 > > _______________________________________________ > devel mailing list > devel@xxxxxxxxxxxxxxxxxxxxxx > http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
