Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1474290
| From | Krzysztof Kozlowski <k.kozlowski@samsung.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH v2 01/10] pinctrl: dt-bindings: samsung: Add header with values used for configuration |
| Date | 2016-09-01 13:20 +0200 |
| Message-ID | <scych-16c-1@gated-at.bofh.it> (permalink) |
| References | <scuBH-63L-3@gated-at.bofh.it> <scuBI-63L-15@gated-at.bofh.it> <scy2C-11X-17@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On 09/01/2016 01:02 PM, Bartlomiej Zolnierkiewicz wrote: > > Hi, > > On Thursday, September 01, 2016 09:23:49 AM Krzysztof Kozlowski wrote: >> Hard-coded pinctrl configuration values are scattered through DTS files. >> The numbers are difficult to decode by human, especially without the >> datasheet. Additionally the drive strength differs between Exynos SoC >> families increasing the confusion. >> >> The header will help making this more readable and maintainable. >> >> Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> >> --- >> MAINTAINERS | 1 + >> include/dt-bindings/pinctrl/samsung.h | 57 +++++++++++++++++++++++++++++++++++ >> 2 files changed, 58 insertions(+) >> create mode 100644 include/dt-bindings/pinctrl/samsung.h >> >> diff --git a/MAINTAINERS b/MAINTAINERS >> index d8e81b1dde30..836bcd327ee4 100644 >> --- a/MAINTAINERS >> +++ b/MAINTAINERS >> @@ -9318,6 +9318,7 @@ L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers) >> L: linux-samsung-soc@vger.kernel.org (moderated for non-subscribers) >> S: Maintained >> F: drivers/pinctrl/samsung/ >> +F: include/dt-bindings/pinctrl/samsung.h >> >> PIN CONTROLLER - SINGLE >> M: Tony Lindgren <tony@atomide.com> >> diff --git a/include/dt-bindings/pinctrl/samsung.h b/include/dt-bindings/pinctrl/samsung.h >> new file mode 100644 >> index 000000000000..0f410ba5ec7e >> --- /dev/null >> +++ b/include/dt-bindings/pinctrl/samsung.h >> @@ -0,0 +1,57 @@ >> +/* >> + * Samsung's Exynos pinctrl bindings >> + * >> + * Copyright (c) 2016 Samsung Electronics Co., Ltd. >> + * http://www.samsung.com >> + * Author: Krzysztof Kozlowski <krzk@kernel.org> >> + * >> + * This program is free software; you can redistribute it and/or modify >> + * it under the terms of the GNU General Public License version 2 as >> + * published by the Free Software Foundation. >> +*/ >> + >> +#ifndef __DT_BINDINGS_PINCTRL_EXYNOS_H__ >> +#define __DT_BINDINGS_PINCTRL_EXYNOS_H__ > > Minor nit: > > shouldn't this be __DT_BINDINGS_PINCTRL_SAMSUNG_H__ (the header file > is named samsung.h and is going to be used also by s3c* SoCs)? Yes, it should. Thanks! Best regards, Krzysztof
Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread
[PATCH v2 01/10] pinctrl: dt-bindings: samsung: Add header with values used for configuration Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2016-09-01 09:30 +0200
[PATCH v2.1] pinctrl: dt-bindings: samsung: Add header with values used for configuration Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2016-09-01 11:10 +0200
Re: [PATCH v2 01/10] pinctrl: dt-bindings: samsung: Add header with values used for configuration Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2016-09-01 13:10 +0200
Re: [PATCH v2 01/10] pinctrl: dt-bindings: samsung: Add header with values used for configuration Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2016-09-01 13:20 +0200
csiph-web