After the declarations in a function, there should be a blank line, so
that the declaration part is visibly separated from the rest.
This refactoring makes the code more readable.
Signed-off-by: Fabian Krueger <fabian.krueger@fau.de>
Signed-off-by: Michael Scheiderer <michael.scheiderer@fau.de>
Cc: <linux-kernel@i4.cs.fau.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 kp_spi_write_reg(struct kp_spi_controller_state *cs, int idx, u64 val)
 {
        u64 __iomem *addr = cs->base;
+
        addr += idx;
        writeq(val, addr);
        if (idx == KP_SPI_REG_CONFIG)
                        unsigned long bit)
 {
        unsigned long timeout;
+
        timeout = jiffies + msecs_to_jiffies(1000);
        while (!(kp_spi_read_reg(cs, idx) & bit)) {
                if (time_after(jiffies, timeout)) {
 kp_spi_cleanup(struct spi_device *spidev)
 {
        struct kp_spi_controller_state *cs = spidev->controller_state;
+
        if (cs) {
                kfree(cs);
        }
 kp_spi_remove(struct platform_device *pldev)
 {
        struct spi_master *master = platform_get_drvdata(pldev);
+
        spi_unregister_master(master);
        return 0;
 }