Messages in this thread | | | From | Sam Protsenko <> | Date | Fri, 30 Jul 2021 22:31:18 +0300 | Subject | Re: [PATCH 03/12] dt-bindings: pinctrl: samsung: Add Exynos850 doc |
| |
On Fri, 30 Jul 2021 at 18:25, Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> wrote: > > On 30/07/2021 16:49, Sam Protsenko wrote: > > Document compatible string for Exynos850 SoC. Nothing else is changed, > > as Exynos850 SoC uses already existing samsung pinctrl driver. > > > > Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org> > > --- > > Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt | 1 + > > 1 file changed, 1 insertion(+) > > > > The patch should be first in the series - dt-bindings go at beginning. > Although no need to resend just for this. > If the resend happens, the fix ("pinctrl: samsung: Fix pinctrl bank pin > count") should be sent separately (no mixing fixes and new features) > because they are independent and usually easier for review. >
Thanks, will be done in v2. Also, just re-sent ("pinctrl: samsung: Fix pinctrl bank pin count") fix separately, as you recommended.
> > Best regards, > Krzysztof
| |