Messages in this thread | | | Subject | Re: [PATCH 01/14] ata: ahci-platform: add reset control support and the flag to specify using reset | From | Sergei Shtylyov <> | Date | Wed, 22 Aug 2018 12:34:30 +0300 |
| |
Hello!
On 8/22/2018 10:36 AM, Kunihiko Hayashi wrote:
> Add support to get and control a list of resets for the device > as optional and shared. These resets must be kept de-asserted until > the device is enabled. > > This is specified as shared because some SoCs like UniPhier series > have common reset controls with all ahci controller instances. > > However, according to Thierry's view, > https://www.spinics.net/lists/linux-ide/msg55357.html > some hardware-specific drivers already use their own resets, > and the common reset make a path to occur double controls of resets. > > Now this add the flag to ahci_platform_get_resources() indicating > whether to use the resources, currently resets only, and existing > drivers set 0 to this flags. > > Suggested-by: Hans de Goede <hdegoede@redhat.com> > Cc: Thierry Reding <thierry.reding@gmail.com> > Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com> [...]
> diff --git a/include/linux/ahci_platform.h b/include/linux/ahci_platform.h > index 1b0a17b..eaedca5f 100644 > --- a/include/linux/ahci_platform.h > +++ b/include/linux/ahci_platform.h > @@ -30,7 +30,7 @@ void ahci_platform_disable_regulators(struct ahci_host_priv *hpriv); > int ahci_platform_enable_resources(struct ahci_host_priv *hpriv); > void ahci_platform_disable_resources(struct ahci_host_priv *hpriv); > struct ahci_host_priv *ahci_platform_get_resources( > - struct platform_device *pdev); > + struct platform_device *pdev, unsigned int flags);
That breaks all the users of this API. You should fix the callers in this same patch to avoid breakage.
[...]
MBR, Sergei
| |