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

Groups > linux.kernel > #1595299

Re: [PATCH 1/2] reset: add reset-simple to unify socfpga, stm32, and sunxi

Show all headers | View raw

Hi,

On 08/03/17 12:20, Philipp Zabel wrote:
> On Wed, 2017-03-08 at 12:05 +0100, Alexandre Torgue wrote:
>> Hi Philipp,
>>
>> On 03/08/2017 11:19 AM, Andre Przywara wrote:
>>> Hi,
>>>
>>> On 08/03/17 09:54, Philipp Zabel wrote:
>>>> Reset operations for simple reset controllers with reset lines that can
>>>> be controlled by toggling bits in (mostly) contiguous register ranges
>>>> using read-modify-write cycles under a spinlock. So far this covers the
>>>> socfpga, stm32, and sunxi drivers.
>>>
>>> Wow, that looks nice, thanks for that.
>>>
>>> But can't we go one step further and unify those driver into one file then?
>>> And either have different probe functions to cover the different DT
>>> requirements or to just have one unified probe checking for the super
>>> set of all properties?
>>
>> I agree with Andre. It looks nice and it should be a good thing to have 
>> a common probe inside reset-simple.c
>> Maybe only "nresets" and "inverted"  DT properties are needed.
> 
> Adding DT properties is not an option, as the driver would have to work
> with the existing bindings. We could merge them into one file, with a
> single probe function that configures different parameters depending on
> the of_device_id returned by of_match_device.

Yes, either that or we could just parse all possible properties, as
allowing additional properties on top of the documented binding doc
shouldn't hurt, I think. That's what I mean with super-set. I have to
check if this is acceptable, though.

And I think it would be worthwhile to create a generic binding (or at
least a generic compatible string) on the way, specifying all the
existing properties there, so new drivers could just use that without
having to add their own compatible string to the *driver* (but just to
the binding doc). This would have the advantage of new SoCs possibly
being supported without actual kernel changes.

Cheers,
Andre.

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 1/2] reset: add reset-simple to unify socfpga, stm32, and sunxi Philipp Zabel <p.zabel@pengutronix.de> - 2017-03-08 11:00 +0100
  [PATCH 2/2] reset: simple: read back to make sure changes are applied Philipp Zabel <p.zabel@pengutronix.de> - 2017-03-08 11:00 +0100
    Re: [PATCH 2/2] reset: simple: read back to make sure changes are  applied Andre Przywara <andre.przywara@arm.com> - 2017-03-08 15:10 +0100
  Re: [PATCH 1/2] reset: add reset-simple to unify socfpga, stm32, and  sunxi Andre Przywara <andre.przywara@arm.com> - 2017-03-08 11:40 +0100
    Re: [PATCH 1/2] reset: add reset-simple to unify socfpga, stm32,  and sunxi Philipp Zabel <p.zabel@pengutronix.de> - 2017-03-08 13:40 +0100
      Re: [PATCH 1/2] reset: add reset-simple to unify socfpga, stm32, and  sunxi Andre Przywara <andre.przywara@arm.com> - 2017-03-08 16:50 +0100
    Re: [PATCH 1/2] reset: add reset-simple to unify socfpga, stm32, and  sunxi Alexandre Torgue <alexandre.torgue@st.com> - 2017-03-08 14:00 +0100

csiph-web