| Message ID | 20230121033713.3535351-5-shravan.chippa@microchip.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | media: i2c: imx334: support lower bandwidth mode | expand |
On 21/01/2023 04:37, shravan kumar wrote: > From: Shravan Chippa <shravan.chippa@microchip.com> > > Add new supported link frequency in dt example. You got comment to fix you CC list. Why not following my feedback? Best regards, Krzysztof
On 23/01/2023 07:28, Shravan.Chippa@microchip.com wrote: > Hi, > >> -----Original Message----- >> From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >> Sent: 22 January 2023 07:37 PM >> To: shravan Chippa - I35088 <Shravan.Chippa@microchip.com>; >> paul.j.murphy@intel.com; daniele.alessandrelli@intel.com; >> mchehab@kernel.org; krzysztof.kozlowski+dt@linaro.org; >> shawnguo@kernel.org; s.hauer@pengutronix.de >> Cc: festevam@gmail.com; kernel@pengutronix.de; linux-imx@nxp.com; >> linux-media@vger.kernel.org; linux-kernel@vger.kernel.org; >> devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; Sakari >> Ailus <sakari.ailus@iki.fi> >> Subject: Re: [PATCH RESEND v10 4/5] dt-bindings: media: i2c: imx334 add >> new link_freq >> >> EXTERNAL EMAIL: Do not click links or open attachments unless you know the >> content is safe >> >> On 21/01/2023 04:37, shravan kumar wrote: >>> From: Shravan Chippa <shravan.chippa@microchip.com> >>> >>> Add new supported link frequency in dt example. >> >> You got a comment to fix you CC list. Why not follow my feedback? > > Based on your previous comment I ran the below script, rebased to the latest code and based on that output I have added a CC list > But I missed adding one name to the to-list which is "Rob Herring <robh+dt@kernel.org>", I will add it. Which is quite important... Best regards, Krzysztof
On Mon, Jan 23, 2023 at 09:00:30AM +0100, Krzysztof Kozlowski wrote: > On 23/01/2023 07:28, Shravan.Chippa@microchip.com wrote: > > Hi, > > > >> -----Original Message----- > >> From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > >> Sent: 22 January 2023 07:37 PM > >> To: shravan Chippa - I35088 <Shravan.Chippa@microchip.com>; > >> paul.j.murphy@intel.com; daniele.alessandrelli@intel.com; > >> mchehab@kernel.org; krzysztof.kozlowski+dt@linaro.org; > >> shawnguo@kernel.org; s.hauer@pengutronix.de > >> Cc: festevam@gmail.com; kernel@pengutronix.de; linux-imx@nxp.com; > >> linux-media@vger.kernel.org; linux-kernel@vger.kernel.org; > >> devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; Sakari > >> Ailus <sakari.ailus@iki.fi> > >> Subject: Re: [PATCH RESEND v10 4/5] dt-bindings: media: i2c: imx334 add > >> new link_freq > >> > >> EXTERNAL EMAIL: Do not click links or open attachments unless you know the > >> content is safe > >> > >> On 21/01/2023 04:37, shravan kumar wrote: > >>> From: Shravan Chippa <shravan.chippa@microchip.com> > >>> > >>> Add new supported link frequency in dt example. > >> > >> You got a comment to fix you CC list. Why not follow my feedback? > > > > Based on your previous comment I ran the below script, rebased to the latest code and based on that output I have added a CC list > > But I missed adding one name to the to-list which is "Rob Herring <robh+dt@kernel.org>", I will add it. > > Which is quite important... Not really. I've long since given up trying to filter based on that. The DT list vs. to me is about the same firehose. Expecting senders to get it right or figure out what's important to you or not will never happen. With lore now, there's little need as I control what I get or not. Rob
diff --git a/Documentation/devicetree/bindings/media/i2c/sony,imx334.yaml b/Documentation/devicetree/bindings/media/i2c/sony,imx334.yaml index f5055b9db693..09533496b20c 100644 --- a/Documentation/devicetree/bindings/media/i2c/sony,imx334.yaml +++ b/Documentation/devicetree/bindings/media/i2c/sony,imx334.yaml @@ -82,7 +82,7 @@ examples: imx334: endpoint { remote-endpoint = <&cam>; data-lanes = <1 2 3 4>; - link-frequencies = /bits/ 64 <891000000>; + link-frequencies = /bits/ 64 <891000000 445500000>; }; }; };