| Message ID | 9fd1e7dd9e8f944b520f2a789e66e458aeb7b555.1602869338.git.mh12gx2825@gmail.com |
|---|---|
| State | Accepted |
| Commit | c7b2c0ccd064a9866b08935e48aa132fdcf2d48f |
| Headers | show |
| Series | [v5,1/2] staging: media: imx: remove commented code | expand |
Hi Deepak, This is fine with me as long as the comment block just above the #if 0 code is left intact, which it is. Acked-by: Steve Longerbeam <slongerbeam@gmail.com> On 10/16/20 10:40 AM, Deepak R Varma wrote: > The virtual channel is always treated as 0 once the stream enters > CSI's. Commented code in the "#if 0" can be safely removed as it will > not serve any purpose in future. Issue reported by checkpatch script. > > Signed-off-by: Deepak R Varma <mh12gx2825@gmail.com> > --- > Changes since v4: > - Rephrase patch description to make it exact and accurate as > suggested by Julia. > - Add linux-media to the recipient list. Suggested by Hans. > Changes since v3: > - Rephrase Patch description > - Implement feedback from Vaishali & Helen to mention checkpatch in > the patch description. > Changes since v2: > - None > Changes since v1: > - Implement feedback from Julia to be exact with the patch description > > drivers/staging/media/imx/imx-media-csi.c | 7 ------- > 1 file changed, 7 deletions(-) > > diff --git a/drivers/staging/media/imx/imx-media-csi.c b/drivers/staging/media/imx/imx-media-csi.c > index 21ebf7769696..9320e3974d86 100644 > --- a/drivers/staging/media/imx/imx-media-csi.c > +++ b/drivers/staging/media/imx/imx-media-csi.c > @@ -1131,13 +1131,6 @@ static int csi_link_validate(struct v4l2_subdev *sd, > * enters the CSI's however, they are treated internally > * in the IPU as virtual channel 0. > */ > -#if 0 > - mutex_unlock(&priv->lock); > - vc_num = imx_media_find_mipi_csi2_channel(&priv->sd.entity); > - if (vc_num < 0) > - return vc_num; > - mutex_lock(&priv->lock); > -#endif > ipu_csi_set_mipi_datatype(priv->csi, vc_num, > &priv->format_mbus[CSI_SINK_PAD]); > }
On Fri, Oct 16, 2020 at 11:10:33PM +0530, Deepak R Varma wrote: Hello All, Request for review / ack of this patch. Thank you. Deepak. > The virtual channel is always treated as 0 once the stream enters > CSI's. Commented code in the "#if 0" can be safely removed as it will > not serve any purpose in future. Issue reported by checkpatch script. > > Signed-off-by: Deepak R Varma <mh12gx2825@gmail.com> > --- > Changes since v4: > - Rephrase patch description to make it exact and accurate as > suggested by Julia. > - Add linux-media to the recipient list. Suggested by Hans. > Changes since v3: > - Rephrase Patch description > - Implement feedback from Vaishali & Helen to mention checkpatch in > the patch description. > Changes since v2: > - None > Changes since v1: > - Implement feedback from Julia to be exact with the patch description > > drivers/staging/media/imx/imx-media-csi.c | 7 ------- > 1 file changed, 7 deletions(-) > > diff --git a/drivers/staging/media/imx/imx-media-csi.c b/drivers/staging/media/imx/imx-media-csi.c > index 21ebf7769696..9320e3974d86 100644 > --- a/drivers/staging/media/imx/imx-media-csi.c > +++ b/drivers/staging/media/imx/imx-media-csi.c > @@ -1131,13 +1131,6 @@ static int csi_link_validate(struct v4l2_subdev *sd, > * enters the CSI's however, they are treated internally > * in the IPU as virtual channel 0. > */ > -#if 0 > - mutex_unlock(&priv->lock); > - vc_num = imx_media_find_mipi_csi2_channel(&priv->sd.entity); > - if (vc_num < 0) > - return vc_num; > - mutex_lock(&priv->lock); > -#endif > ipu_csi_set_mipi_datatype(priv->csi, vc_num, > &priv->format_mbus[CSI_SINK_PAD]); > } > -- > 2.25.1 >
On Mon, Oct 26, 2020 at 09:30:17AM +0530, Deepak R Varma wrote: > On Fri, Oct 16, 2020 at 11:10:33PM +0530, Deepak R Varma wrote: > > Hello All, > Request for review / ack of this patch. Please never top-post :( The merge window only opened up a few hours ago, give maintainers time to get to patches (like 2 weeks now). Also remember, for the outreachy project, drivers/staging/media/ patches are not considered and take much longer to get reviewed and accepted, as the outreachy documentation states. good luck! greg k-h
On Mon, Oct 26, 2020 at 06:56:31AM +0100, Greg Kroah-Hartman wrote: > On Mon, Oct 26, 2020 at 09:30:17AM +0530, Deepak R Varma wrote: > > On Fri, Oct 16, 2020 at 11:10:33PM +0530, Deepak R Varma wrote: > > > > Hello All, > > Request for review / ack of this patch. > > Please never top-post :( Apologies. Will keep this in mind going forward. > > The merge window only opened up a few hours ago, give maintainers time > to get to patches (like 2 weeks now). Also remember, for the outreachy > project, drivers/staging/media/ patches are not considered and take much > longer to get reviewed and accepted, as the outreachy documentation > states. We had both the patches review acked by Steve L. However, they did not make it to your list of accepted patches. Hence was checking if there is anything more needed from me to have the patches get onto your acceptance. No problem. I will wait for the normal course be followed. Thank you as always :) Deepak. > > good luck! > > greg k-h
On 26/10/2020 06:56, Greg Kroah-Hartman wrote: > On Mon, Oct 26, 2020 at 09:30:17AM +0530, Deepak R Varma wrote: >> On Fri, Oct 16, 2020 at 11:10:33PM +0530, Deepak R Varma wrote: >> >> Hello All, >> Request for review / ack of this patch. > > Please never top-post :( > > The merge window only opened up a few hours ago, give maintainers time > to get to patches (like 2 weeks now). Also remember, for the outreachy > project, drivers/staging/media/ patches are not considered and take much > longer to get reviewed and accepted, as the outreachy documentation > states. As media reviewer I've been very, very busy with other things for the past 3 weeks. I plan to use most (all?) of next week to catch up on things. Regards, Hans
diff --git a/drivers/staging/media/imx/imx-media-csi.c b/drivers/staging/media/imx/imx-media-csi.c index 21ebf7769696..9320e3974d86 100644 --- a/drivers/staging/media/imx/imx-media-csi.c +++ b/drivers/staging/media/imx/imx-media-csi.c @@ -1131,13 +1131,6 @@ static int csi_link_validate(struct v4l2_subdev *sd, * enters the CSI's however, they are treated internally * in the IPU as virtual channel 0. */ -#if 0 - mutex_unlock(&priv->lock); - vc_num = imx_media_find_mipi_csi2_channel(&priv->sd.entity); - if (vc_num < 0) - return vc_num; - mutex_lock(&priv->lock); -#endif ipu_csi_set_mipi_datatype(priv->csi, vc_num, &priv->format_mbus[CSI_SINK_PAD]); }
The virtual channel is always treated as 0 once the stream enters CSI's. Commented code in the "#if 0" can be safely removed as it will not serve any purpose in future. Issue reported by checkpatch script. Signed-off-by: Deepak R Varma <mh12gx2825@gmail.com> --- Changes since v4: - Rephrase patch description to make it exact and accurate as suggested by Julia. - Add linux-media to the recipient list. Suggested by Hans. Changes since v3: - Rephrase Patch description - Implement feedback from Vaishali & Helen to mention checkpatch in the patch description. Changes since v2: - None Changes since v1: - Implement feedback from Julia to be exact with the patch description drivers/staging/media/imx/imx-media-csi.c | 7 ------- 1 file changed, 7 deletions(-)