Re: [PATCH] media: dt-bindings: media: Fixup Allwinner A10 CSI binding
Hi Rob, On Tue, Oct 22, 2019 at 11:17:35AM -0500, Rob Herring wrote: > On Mon, Oct 7, 2019 at 5:48 AM Maxime Ripard wrote: > > > > On Mon, Oct 07, 2019 at 03:45:29PM +0530, Amit Kucheria wrote: > > > On Mon, Oct 7, 2019 at 3:35 PM Maxime Ripard wrote: > > > > > > > > On Sat, Oct 05, 2019 at 04:15:57PM +0530, Amit Kucheria wrote: > > > > > On Thu, Oct 3, 2019 at 5:22 PM Maxime Ripard > > > > > wrote: > > > > > > > > > > > > Hi, > > > > > > > > > > > > On Thu, Oct 03, 2019 at 04:52:24PM +0530, Amit Kucheria wrote: > > > > > > > This new binding fails dt_binding_check due to a typo. Fix it up. > > > > > > > > > > > > > > linux.git/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.yaml: > > > > > > > $id: path/filename 'arm/allwinner,sun4i-a10-csi.yaml' doesn't > > > > > > > match actual filename > > > > > > > linux.git/Documentation/devicetree/bindings/Makefile:12: recipe > > > > > > > for target > > > > > > > 'Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.example.dts' > > > > > > > failed > > > > > > > make[2]: *** > > > > > > > [Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.example.dts] > > > > > > > Error 1 > > > > > > > make[2]: *** Waiting for unfinished jobs > > > > > > > linux.git/Makefile:1284: recipe for target 'dt_binding_check' > > > > > > > failed > > > > > > > make[1]: *** [dt_binding_check] Error 2 > > > > > > > > > > > > > > Fixes: c5e8f4ccd7750 ("media: dt-bindings: media: Add Allwinner > > > > > > > A10 CSI binding") > > > > > > > Signed-off-by: Amit Kucheria > > > > > > > > > > > > Thanks for your patch. > > > > > > > > > > > > It has already been submitted though: > > > > > > https://lore.kernel.org/linux-arm-kernel/1568808060-17516-1-git-send-email-pragnesh.pa...@sifive.com/ > > > > > > > > > > > > I'm not sure why it hasn't been applied yet though :/ > > > > > > > > > > Perhaps a Fixes tag will allow more attention to it? > > > > > > > > I've added a fixes tag and merged it through the sunxi tree. > > > > > > > > Sorry for the time it took, and thanks for sending that fix! > > > > > > Will it get merged for -rc2? > > > > -rc2 was released yesterday, so we're a bit late for that, but it's > > going to be in next tomorrow. > > Is this going to Linus anytime soon? It is affecting anyone submitting > bindings against current -rc's. Sorry for the delay, I just sent a PR Maxime signature.asc Description: PGP signature
Re: [PATCH] media: dt-bindings: media: Fixup Allwinner A10 CSI binding
On Mon, Oct 7, 2019 at 5:48 AM Maxime Ripard wrote: > > On Mon, Oct 07, 2019 at 03:45:29PM +0530, Amit Kucheria wrote: > > On Mon, Oct 7, 2019 at 3:35 PM Maxime Ripard wrote: > > > > > > On Sat, Oct 05, 2019 at 04:15:57PM +0530, Amit Kucheria wrote: > > > > On Thu, Oct 3, 2019 at 5:22 PM Maxime Ripard wrote: > > > > > > > > > > Hi, > > > > > > > > > > On Thu, Oct 03, 2019 at 04:52:24PM +0530, Amit Kucheria wrote: > > > > > > This new binding fails dt_binding_check due to a typo. Fix it up. > > > > > > > > > > > > linux.git/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.yaml: > > > > > > $id: path/filename 'arm/allwinner,sun4i-a10-csi.yaml' doesn't > > > > > > match actual filename > > > > > > linux.git/Documentation/devicetree/bindings/Makefile:12: recipe for > > > > > > target > > > > > > 'Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.example.dts' > > > > > > failed > > > > > > make[2]: *** > > > > > > [Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.example.dts] > > > > > > Error 1 > > > > > > make[2]: *** Waiting for unfinished jobs > > > > > > linux.git/Makefile:1284: recipe for target 'dt_binding_check' failed > > > > > > make[1]: *** [dt_binding_check] Error 2 > > > > > > > > > > > > Fixes: c5e8f4ccd7750 ("media: dt-bindings: media: Add Allwinner A10 > > > > > > CSI binding") > > > > > > Signed-off-by: Amit Kucheria > > > > > > > > > > Thanks for your patch. > > > > > > > > > > It has already been submitted though: > > > > > https://lore.kernel.org/linux-arm-kernel/1568808060-17516-1-git-send-email-pragnesh.pa...@sifive.com/ > > > > > > > > > > I'm not sure why it hasn't been applied yet though :/ > > > > > > > > Perhaps a Fixes tag will allow more attention to it? > > > > > > I've added a fixes tag and merged it through the sunxi tree. > > > > > > Sorry for the time it took, and thanks for sending that fix! > > > > Will it get merged for -rc2? > > -rc2 was released yesterday, so we're a bit late for that, but it's > going to be in next tomorrow. Is this going to Linus anytime soon? It is affecting anyone submitting bindings against current -rc's. Rob
Re: [PATCH] media: dt-bindings: media: Fixup Allwinner A10 CSI binding
On Mon, Oct 07, 2019 at 03:45:29PM +0530, Amit Kucheria wrote: > On Mon, Oct 7, 2019 at 3:35 PM Maxime Ripard wrote: > > > > On Sat, Oct 05, 2019 at 04:15:57PM +0530, Amit Kucheria wrote: > > > On Thu, Oct 3, 2019 at 5:22 PM Maxime Ripard wrote: > > > > > > > > Hi, > > > > > > > > On Thu, Oct 03, 2019 at 04:52:24PM +0530, Amit Kucheria wrote: > > > > > This new binding fails dt_binding_check due to a typo. Fix it up. > > > > > > > > > > linux.git/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.yaml: > > > > > $id: path/filename 'arm/allwinner,sun4i-a10-csi.yaml' doesn't match > > > > > actual filename > > > > > linux.git/Documentation/devicetree/bindings/Makefile:12: recipe for > > > > > target > > > > > 'Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.example.dts' > > > > > failed > > > > > make[2]: *** > > > > > [Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.example.dts] > > > > > Error 1 > > > > > make[2]: *** Waiting for unfinished jobs > > > > > linux.git/Makefile:1284: recipe for target 'dt_binding_check' failed > > > > > make[1]: *** [dt_binding_check] Error 2 > > > > > > > > > > Fixes: c5e8f4ccd7750 ("media: dt-bindings: media: Add Allwinner A10 > > > > > CSI binding") > > > > > Signed-off-by: Amit Kucheria > > > > > > > > Thanks for your patch. > > > > > > > > It has already been submitted though: > > > > https://lore.kernel.org/linux-arm-kernel/1568808060-17516-1-git-send-email-pragnesh.pa...@sifive.com/ > > > > > > > > I'm not sure why it hasn't been applied yet though :/ > > > > > > Perhaps a Fixes tag will allow more attention to it? > > > > I've added a fixes tag and merged it through the sunxi tree. > > > > Sorry for the time it took, and thanks for sending that fix! > > Will it get merged for -rc2? -rc2 was released yesterday, so we're a bit late for that, but it's going to be in next tomorrow. Maxime signature.asc Description: PGP signature
Re: [PATCH] media: dt-bindings: media: Fixup Allwinner A10 CSI binding
On Mon, Oct 7, 2019 at 3:35 PM Maxime Ripard wrote: > > On Sat, Oct 05, 2019 at 04:15:57PM +0530, Amit Kucheria wrote: > > On Thu, Oct 3, 2019 at 5:22 PM Maxime Ripard wrote: > > > > > > Hi, > > > > > > On Thu, Oct 03, 2019 at 04:52:24PM +0530, Amit Kucheria wrote: > > > > This new binding fails dt_binding_check due to a typo. Fix it up. > > > > > > > > linux.git/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.yaml: > > > > $id: path/filename 'arm/allwinner,sun4i-a10-csi.yaml' doesn't match > > > > actual filename > > > > linux.git/Documentation/devicetree/bindings/Makefile:12: recipe for > > > > target > > > > 'Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.example.dts' > > > > failed > > > > make[2]: *** > > > > [Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.example.dts] > > > > Error 1 > > > > make[2]: *** Waiting for unfinished jobs > > > > linux.git/Makefile:1284: recipe for target 'dt_binding_check' failed > > > > make[1]: *** [dt_binding_check] Error 2 > > > > > > > > Fixes: c5e8f4ccd7750 ("media: dt-bindings: media: Add Allwinner A10 CSI > > > > binding") > > > > Signed-off-by: Amit Kucheria > > > > > > Thanks for your patch. > > > > > > It has already been submitted though: > > > https://lore.kernel.org/linux-arm-kernel/1568808060-17516-1-git-send-email-pragnesh.pa...@sifive.com/ > > > > > > I'm not sure why it hasn't been applied yet though :/ > > > > Perhaps a Fixes tag will allow more attention to it? > > I've added a fixes tag and merged it through the sunxi tree. > > Sorry for the time it took, and thanks for sending that fix! Will it get merged for -rc2?
Re: [PATCH] media: dt-bindings: media: Fixup Allwinner A10 CSI binding
On Sat, Oct 05, 2019 at 04:15:57PM +0530, Amit Kucheria wrote: > On Thu, Oct 3, 2019 at 5:22 PM Maxime Ripard wrote: > > > > Hi, > > > > On Thu, Oct 03, 2019 at 04:52:24PM +0530, Amit Kucheria wrote: > > > This new binding fails dt_binding_check due to a typo. Fix it up. > > > > > > linux.git/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.yaml: > > > $id: path/filename 'arm/allwinner,sun4i-a10-csi.yaml' doesn't match > > > actual filename > > > linux.git/Documentation/devicetree/bindings/Makefile:12: recipe for > > > target > > > 'Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.example.dts' > > > failed > > > make[2]: *** > > > [Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.example.dts] > > > Error 1 > > > make[2]: *** Waiting for unfinished jobs > > > linux.git/Makefile:1284: recipe for target 'dt_binding_check' failed > > > make[1]: *** [dt_binding_check] Error 2 > > > > > > Fixes: c5e8f4ccd7750 ("media: dt-bindings: media: Add Allwinner A10 CSI > > > binding") > > > Signed-off-by: Amit Kucheria > > > > Thanks for your patch. > > > > It has already been submitted though: > > https://lore.kernel.org/linux-arm-kernel/1568808060-17516-1-git-send-email-pragnesh.pa...@sifive.com/ > > > > I'm not sure why it hasn't been applied yet though :/ > > Perhaps a Fixes tag will allow more attention to it? I've added a fixes tag and merged it through the sunxi tree. Sorry for the time it took, and thanks for sending that fix! Maxime signature.asc Description: PGP signature
Re: [PATCH] media: dt-bindings: media: Fixup Allwinner A10 CSI binding
On Thu, Oct 3, 2019 at 5:22 PM Maxime Ripard wrote: > > Hi, > > On Thu, Oct 03, 2019 at 04:52:24PM +0530, Amit Kucheria wrote: > > This new binding fails dt_binding_check due to a typo. Fix it up. > > > > linux.git/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.yaml: > > $id: path/filename 'arm/allwinner,sun4i-a10-csi.yaml' doesn't match actual > > filename > > linux.git/Documentation/devicetree/bindings/Makefile:12: recipe for target > > 'Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.example.dts' > > failed > > make[2]: *** > > [Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.example.dts] > > Error 1 > > make[2]: *** Waiting for unfinished jobs > > linux.git/Makefile:1284: recipe for target 'dt_binding_check' failed > > make[1]: *** [dt_binding_check] Error 2 > > > > Fixes: c5e8f4ccd7750 ("media: dt-bindings: media: Add Allwinner A10 CSI > > binding") > > Signed-off-by: Amit Kucheria > > Thanks for your patch. > > It has already been submitted though: > https://lore.kernel.org/linux-arm-kernel/1568808060-17516-1-git-send-email-pragnesh.pa...@sifive.com/ > > I'm not sure why it hasn't been applied yet though :/ Perhaps a Fixes tag will allow more attention to it?
Re: [PATCH] media: dt-bindings: media: Fixup Allwinner A10 CSI binding
Hi, On Thu, Oct 03, 2019 at 04:52:24PM +0530, Amit Kucheria wrote: > This new binding fails dt_binding_check due to a typo. Fix it up. > > linux.git/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.yaml: > $id: path/filename 'arm/allwinner,sun4i-a10-csi.yaml' doesn't match actual > filename > linux.git/Documentation/devicetree/bindings/Makefile:12: recipe for target > 'Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.example.dts' > failed > make[2]: *** > [Documentation/devicetree/bindings/media/allwinner,sun4i-a10-csi.example.dts] > Error 1 > make[2]: *** Waiting for unfinished jobs > linux.git/Makefile:1284: recipe for target 'dt_binding_check' failed > make[1]: *** [dt_binding_check] Error 2 > > Fixes: c5e8f4ccd7750 ("media: dt-bindings: media: Add Allwinner A10 CSI > binding") > Signed-off-by: Amit Kucheria Thanks for your patch. It has already been submitted though: https://lore.kernel.org/linux-arm-kernel/1568808060-17516-1-git-send-email-pragnesh.pa...@sifive.com/ I'm not sure why it hasn't been applied yet though :/ Maxime > signature.asc Description: PGP signature