Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1411260 > unrolled thread

Re: [PATCH v3 0/4] Amlogic: Meson: Add reset controller

Started byPhilipp Zabel <p.zabel@pengutronix.de>
First post2016-06-01 16:10 +0200
Last post2016-06-01 19:50 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v3 0/4] Amlogic: Meson: Add reset controller Philipp Zabel <p.zabel@pengutronix.de> - 2016-06-01 16:10 +0200
    Re: [PATCH v3 0/4] Amlogic: Meson: Add reset controller Kevin Hilman <khilman@baylibre.com> - 2016-06-01 19:50 +0200

#1411260 — Re: [PATCH v3 0/4] Amlogic: Meson: Add reset controller

FromPhilipp Zabel <p.zabel@pengutronix.de>
Date2016-06-01 16:10 +0200
SubjectRe: [PATCH v3 0/4] Amlogic: Meson: Add reset controller
Message-ID<rFf0m-Xx-39@gated-at.bofh.it>
Hi Kevin,

Am Montag, den 30.05.2016, 15:27 +0200 schrieb Neil Armstrong:
> Patchset to add and enable the reset controller driver on Meson SoCs platforms.
> 
> This reset controller has up to 256 reset lines with reset pulse generation only,
> so the assert and deassert calls are not available.
> 
> Depends on the patch :
>  - "reset: add devm_reset_controller_register API" [1]
> 
> Changes since v2 at http://lkml.kernel.org/r/1464169758-26975-1-git-send-email-narmstrong@baylibre.com :
> - Get back __iomem in _reset() callback
> - Get rid of MODULE_ALIAS
> 
> Changes since v1 at http://lkml.kernel.org/r/1463732875-23141-1-git-send-email-narmstrong@baylibre.com :
> - Remove _gxbb_ in names
> - Depends build on ARCH_MESON only
> - Add meson8b compatible and bindings
> - Re-indent dt-bindings headers
> - Switch to devm_ reset controller register
> - Remove platform "remove" callback
> - Update Dual BSD/GPL file headers
> 
> Changes since the RFC at http://lkml.kernel.org/r/1463148012-25988-1-git-send-email-narmstrong@baylibre.com :
> - Fix register mapping and bit defines in bindings header
> - Remove assert and unassert calls
> - Fix missing __iomem
> 
> [1] https://patchwork.kernel.org/patch/8988471/
> 
> Neil Armstrong (4):
>   reset: Add support for the Amlogic Meson SoC Reset Controller
>   dt-bindings: reset: Add bindings for the Meson SoC Reset Controller
>   ARM64: dts: amlogic: Enable Reset Controller on GXBB-based platforms
>   ARM: dts: amlogic: Enable Reset Controller on Meson8b platforms

Should I take the two device tree patches into the reset tree?
Is resolving the reset/deassert situation a prerequisite for these
patches?

regards
Philipp

[toc] | [next] | [standalone]


#1411427

FromKevin Hilman <khilman@baylibre.com>
Date2016-06-01 19:50 +0200
Message-ID<rFirg-2Zd-13@gated-at.bofh.it>
In reply to#1411260
Philipp Zabel <p.zabel@pengutronix.de> writes:

> Hi Kevin,
>
> Am Montag, den 30.05.2016, 15:27 +0200 schrieb Neil Armstrong:
>> Patchset to add and enable the reset controller driver on Meson SoCs platforms.
>> 
>> This reset controller has up to 256 reset lines with reset pulse generation only,
>> so the assert and deassert calls are not available.
>> 
>> Depends on the patch :
>>  - "reset: add devm_reset_controller_register API" [1]
>> 
>> Changes since v2 at http://lkml.kernel.org/r/1464169758-26975-1-git-send-email-narmstrong@baylibre.com :
>> - Get back __iomem in _reset() callback
>> - Get rid of MODULE_ALIAS
>> 
>> Changes since v1 at http://lkml.kernel.org/r/1463732875-23141-1-git-send-email-narmstrong@baylibre.com :
>> - Remove _gxbb_ in names
>> - Depends build on ARCH_MESON only
>> - Add meson8b compatible and bindings
>> - Re-indent dt-bindings headers
>> - Switch to devm_ reset controller register
>> - Remove platform "remove" callback
>> - Update Dual BSD/GPL file headers
>> 
>> Changes since the RFC at http://lkml.kernel.org/r/1463148012-25988-1-git-send-email-narmstrong@baylibre.com :
>> - Fix register mapping and bit defines in bindings header
>> - Remove assert and unassert calls
>> - Fix missing __iomem
>> 
>> [1] https://patchwork.kernel.org/patch/8988471/
>> 
>> Neil Armstrong (4):
>>   reset: Add support for the Amlogic Meson SoC Reset Controller
>>   dt-bindings: reset: Add bindings for the Meson SoC Reset Controller
>>   ARM64: dts: amlogic: Enable Reset Controller on GXBB-based platforms
>>   ARM: dts: amlogic: Enable Reset Controller on Meson8b platforms
>
> Should I take the two device tree patches into the reset tree?

You can take the driver and the bindings. We will take the DT patches
through the amlogic tree, and submit via arm-soc.

> Is resolving the reset/deassert situation a prerequisite for these
> patches?

No, not a prerequisite.  Go ahead and merge the driver.

It just means that hardware reset doesn't actually happen when using
drivers/net/ethernet/stmicro/stmmac/* because it only uses
assert/reassert currently.

Kevin

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web