Even if platform_driver does not set suppress_bind_attrs attribute,
when registering with platform_driver_probe,  the value of
suppress_bind_attrs is still true, see __platform_driver_probe()

Signed-off-by: lizhe <sensor1010@163.com>
---
 drivers/bus/brcmstb_gisb.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/bus/brcmstb_gisb.c b/drivers/bus/brcmstb_gisb.c
index 183d5cc37d42..b0c3704777e9 100644
--- a/drivers/bus/brcmstb_gisb.c
+++ b/drivers/bus/brcmstb_gisb.c
@@ -536,7 +536,6 @@ static struct platform_driver brcmstb_gisb_arb_driver = {
 		.name	= "brcm-gisb-arb",
 		.of_match_table = brcmstb_gisb_arb_of_match,
 		.pm	= &brcmstb_gisb_arb_pm_ops,
-		.suppress_bind_attrs = true,
 	},
 };
 
-- 
2.25.1

On 3/19/2022 11:45 PM, lizhe wrote:
> Even if platform_driver does not set suppress_bind_attrs attribute,
> when registering with platform_driver_probe,  the value of
> suppress_bind_attrs is still true, see __platform_driver_probe()
> 
> Signed-off-by: lizhe <sensor1010@163.com>

Applied with a revised subject, thanks!
-- 
Florian

On Tue, Mar 22, 2022 at 12:41 AM lizhe <sensor1010@163.com> wrote:
>
> Even if platform_driver does not set suppress_bind_attrs attribute,
> when registering with platform_driver_probe,  the value of
> suppress_bind_attrs is still true, see __platform_driver_probe()

Any reasons it can't be switched to builtin_platform_driver() macro
and hence constification the driver structure?

-- 
With Best Regards,
Andy Shevchenko

On Tue, Mar 22, 2022 at 4:52 PM Andy Shevchenko
<andy.shevchenko@gmail.com> wrote:
> On Tue, Mar 22, 2022 at 12:41 AM lizhe <sensor1010@163.com> wrote:
> >
> > Even if platform_driver does not set suppress_bind_attrs attribute,
> > when registering with platform_driver_probe,  the value of
> > suppress_bind_attrs is still true, see __platform_driver_probe()
>
> Any reasons it can't be switched to builtin_platform_driver() macro
> and hence constification of the driver structure?

Ah, the patch that brought that assignment actually allows to built
the driver as a module, so, the correct patch would be to switch to
module_platform_driver() I suppose.

-- 
With Best Regards,
Andy Shevchenko